Alon Bar-Lev has uploaded a new change for review.

Change subject: core: executePipe: call callback only if set
......................................................................

core: executePipe: call callback only if set

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1214724
Change-Id: I4a8e12876aebf4e8700e94444ebe14063e63ad68
Signed-off-by: Alon Bar-Lev <alo...@redhat.com>
---
M ChangeLog
M src/otopi/plugin.py
2 files changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/otopi refs/changes/80/40180/1

diff --git a/ChangeLog b/ChangeLog
index 07e8d22..d602bce 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -3,6 +3,7 @@
 ????-??-?? - Version 1.4.0
 
  * core: support different exit codes.
+ * core: executePipe: call callback only if set rhbz#1214724.
 
 2015-01-15 - Version 1.3.1
 
diff --git a/src/otopi/plugin.py b/src/otopi/plugin.py
index bb3af02..b763579 100644
--- a/src/otopi/plugin.py
+++ b/src/otopi/plugin.py
@@ -391,7 +391,8 @@
                 ) < now
             ):
                 _callCallback.next = datetime.datetime.now()
-                callback(state=popens)
+                if callback:
+                    callback(state=popens)
 
         def _isString(s):
             return (


-- 
To view, visit https://gerrit.ovirt.org/40180
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4a8e12876aebf4e8700e94444ebe14063e63ad68
Gerrit-PatchSet: 1
Gerrit-Project: otopi
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to