Martin Polednik has posted comments on this change. Change subject: static: move vdsm bonding defaults under static ......................................................................
Patch Set 6: (1 comment) No path was hardcoded during the creation of this patch. :) https://gerrit.ovirt.org/#/c/62537/6/static/Makefile.am File static/Makefile.am: PS6, Line 24: /usr/share > Don't we have a predefined variable for this? It's referring to a source path, not the target. Target is vdsmdir. (see configure.ac) AC_SUBST([vdsmdir], ['${datarootdir}/vdsm']) -- To view, visit https://gerrit.ovirt.org/62537 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9cbd81e59aec49876e15337d7cd617e51316a604 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Ondřej Svoboda <osvob...@redhat.com> Gerrit-Reviewer: Petr Horáček <phora...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
