[X2Go-Commits] [x2gobroker] 01/02: x2gobroker/agent.py: Add FIXME about bogus code passage. Needs rechecking...

git-admin at x2go.org git-admin at x2go.org
Thu Sep 13 16:33:15 CEST 2018


This is an automated email from the git hooks/post-receive script.

x2go pushed a commit to branch master
in repository x2gobroker.

commit 32245e36db0a912142ff97f9e23bd2fa8980c35a
Author: Mike Gabriel <mike.gabriel at das-netzwerkteam.de>
Date:   Thu Sep 13 16:31:26 2018 +0200

    x2gobroker/agent.py: Add FIXME about bogus code passage. Needs rechecking...
---
 x2gobroker/agent.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/x2gobroker/agent.py b/x2gobroker/agent.py
index 14620de..3563da2 100644
--- a/x2gobroker/agent.py
+++ b/x2gobroker/agent.py
@@ -150,6 +150,9 @@ def _call_local_broker_agent(username, task, cmdline_args=[], logger=None):
         cmd_line.append('{arg}'.format(arg=cmdline_arg))
 
     logger.info('Executing agent command locally: {cmd}'.format(cmd=" ".join(cmd_line)))
+
+    ### FIXME: why do we set result to ['FAILED'] here and override it later???
+
     result = ['FAILED']
     try:
         agent_process = subprocess.Popen(cmd_line,

--
Alioth's /home/x2go-admin/maintenancescripts/git/hooks/post-receive-email on /srv/git/code.x2go.org/x2gobroker.git


More information about the x2go-commits mailing list