[node-patches] Change in ovirt-node[master]: system. Fix which()

fabiand at fedoraproject.org fabiand at fedoraproject.org
Wed Aug 7 12:16:03 UTC 2013


Fabian Deutsch has uploaded a new change for review.

Change subject: system. Fix which()
......................................................................

system. Fix which()

abspath was used instead of isabs, this broke the logic.

Change-Id: Ida05d44b057056ffbd6300ab4393e143a0ba79b3
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=991408
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/utils/system.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/70/17770/1

diff --git a/src/ovirt/node/utils/system.py b/src/ovirt/node/utils/system.py
index ead2e68..08dce48 100644
--- a/src/ovirt/node/utils/system.py
+++ b/src/ovirt/node/utils/system.py
@@ -92,7 +92,7 @@
         $PATH. Otherwise None (if not found in any path in $PATHS).
     """
     ret = None
-    if os.path.abspath(cmd):
+    if os.path.isabs(cmd):
         if File(cmd).exists():
             ret = cmd
     else:


-- 
To view, visit http://gerrit.ovirt.org/17770
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ida05d44b057056ffbd6300ab4393e143a0ba79b3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>



More information about the node-patches mailing list