Juan Hernandez has posted comments on this change.

Change subject: packaging: Fix ownership of deployments and contents
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File packaging/fedora/spec/ovirt-engine.spec.in
Line 690
Line 691
Line 692
Line 693
Line 694
The directory needs to exist because the application server will create 
subdirectories inside, and it needs to be owned by the user ovirt.

Any directory or file packaged by RPM needs to exist in the build root, it is 
not enough to have the %dir directive.

We can't change the ownership in the %install section because that will force 
us to use root for every build.


--
To view, visit http://gerrit.ovirt.org/9369
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb4745cfc9898b43a164781fd5866cf2b1fa4f82
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Ohad Basan <oba...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to