Dan Kenigsberg has posted comments on this change.

Change subject: vdsm.spec.in/vdsm.rwtab: create /rhev repository
......................................................................


Patch Set 3: (1 inline comment)

I do not think that /rhev/ matters to Vdsm much without the data-center after 
that. please consider using them together.

....................................................
File configure.ac
Line 54: AC_SUBST([vdsmrepo], ['rhev'])
both places should be fixed - there's nothing in /rhev proper but data-center; 
I think that keeping /rhev/data-center as a single string, fits nicer with the 
plans to move to /var/run/vdsm/bla in some point in the future. 

Would putting /rhev/data-center in vdsm.rwtab work for us?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I971a7b47e834d7b6b95874ff6437e66b1928fc82
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to