Change in vdsm[master]: Moving import of zombierepear

ybronhei at redhat.com ybronhei at redhat.com
Sun Jan 12 10:33:26 UTC 2014


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 <ybronhei at redhat.com>
---
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 <ybronhei at redhat.com>


More information about the vdsm-patches mailing list