Yaniv Bronhaim has uploaded a new change for review. Change subject: Moving import of zombierepear ......................................................................
Moving import of zombierepear The import caused build error while zombierepear package was not installed when running make rpm. Although the package is installed as requirement for vdsm rpm and is not needed for the build. During normal run the error won't occur. This patch moves the import to the only necessary location in utils.py. Change-Id: I906b4f4212dd455cdf3ffc2b2409f2e57c9845e6 Signed-off-by: Yaniv Bronhaim <[email protected]> --- M lib/vdsm/utils.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/80/23180/1 diff --git a/lib/vdsm/utils.py b/lib/vdsm/utils.py index f4c8ceb..f8c6c40 100644 --- a/lib/vdsm/utils.py +++ b/lib/vdsm/utils.py @@ -52,7 +52,6 @@ import subprocess import threading import time -import zombiereaper from cpopen import CPopen from . import constants @@ -1005,6 +1004,7 @@ def __del__(self): if self._proc.returncode is None: + import zombiereaper zombiereaper.autoReapPID(self._proc.pid) -- To view, visit http://gerrit.ovirt.org/23180 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I906b4f4212dd455cdf3ffc2b2409f2e57c9845e6 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
