Yaniv Bronhaim has posted comments on this change. Change subject: hooks: Relocate hooking (api) module under lib ......................................................................
Patch Set 4: (1 comment) basically im fine with that.. however, softlink might look better :\ not sure https://gerrit.ovirt.org/#/c/55544/4/vdsm.spec.in File vdsm.spec.in: Line 1097: %{python_sitelib}/%{vdsm_name}/dmidecodeUtil.py* Line 1098: %{python_sitelib}/%{vdsm_name}/exception.py* Line 1099: %{python_sitelib}/%{vdsm_name}/executor.py* Line 1100: %{python_sitelib}/%{vdsm_name}/health.py* Line 1101: %{python_sitelib}/%{vdsm_name}/hooking.py* please add it to debian/vdsm-python.install as well Line 1102: %{python_sitelib}/%{vdsm_name}/hooks.py* Line 1103: %{python_sitelib}/%{vdsm_name}/host.py* Line 1104: %{python_sitelib}/%{vdsm_name}/hostdev.py* Line 1105: %{python_sitelib}/%{vdsm_name}/ipwrapper.py* -- To view, visit https://gerrit.ovirt.org/55544 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5f2e0ed06d5f17885b5892bb8659c52f4542a9d9 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Edward Haas <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Edward Haas <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
