From 5375d2c5ec60a5fd9921d611e743928b46362fe6 Mon Sep 17 00:00:00 2001
From: Marcel Loose <loose@astron.nl>
Date: Thu, 17 Jan 2013 10:23:52 +0000
Subject: [PATCH] Task #2699: vdsmaker doesn't need to set
self.outputs['result'] anymore, because the framework now sets
self.outputs['returncode'].
---
CEP/Pipeline/recipes/sip/nodes/vdsmaker.py | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/CEP/Pipeline/recipes/sip/nodes/vdsmaker.py b/CEP/Pipeline/recipes/sip/nodes/vdsmaker.py
index 63d827f7334..bcaabac3bc2 100644
--- a/CEP/Pipeline/recipes/sip/nodes/vdsmaker.py
+++ b/CEP/Pipeline/recipes/sip/nodes/vdsmaker.py
@@ -33,8 +33,7 @@ class vdsmaker(LOFARnodeTCP):
if not os.access(executable, os.X_OK):
raise ExecutableMissing(executable)
cmd = [executable, clusterdesc, infile, outfile]
- result = catch_segfaults(cmd, None, None, self.logger).returncode
- self.outputs["result"] = result
+ return catch_segfaults(cmd, None, None, self.logger).returncode
except ExecutableMissing, e:
self.logger.error("%s not found" % (e.args[0]))
return 1
--
GitLab