Federico Simoncelli has posted comments on this change.

Change subject: BZ#745510 Additional rpm fixes required by fedora
......................................................................


Patch Set 1: (3 inline comments)

....................................................
File vdsm.spec.in
Line 21: Url:            http://www.ovirt.org/project/subprojects
Dan, do you have a better url? The official page is currently misleading 
(http://www.ovirt.org/wiki/Vdsm), it's still pointing to the old git repository.

Line 275: # Install the configuration sample
It's probably safer to postpone the introduction of the vdsm.conf file in rhel? 
The only side-effect we have now is that when the package is removed (rpm -e) 
the file gets renamed to vdsm.conf.rpmsave.

Line 416: %doc COPYING README vdsm/vdsm.conf.sample
Always ship the sample config file.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7bfeddd1558d391762425ad07ecd3942b802f6c0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to