Edward Haas has posted comments on this change.

Change subject: hooks: Relocate hooking (api) module under lib
......................................................................


Patch Set 4:

(1 comment)

I tried to use softlink, unfortunately it does not work: Unable to import a 
softlink without having the target on the search path.

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
Ok, I will
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

Reply via email to