[ARVADOS] updated: 46f3700db55bfd5dec7e555a43ba769947813b92

[Git user]

Summary of changes:
 sdk/cwl/arvados_cwl/__init__.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

       via  46f3700db55bfd5dec7e555a43ba769947813b92 (commit)
      from  11eae1a76ec028044b2beaa3fdc53daac4c26f5e (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 46f3700db55bfd5dec7e555a43ba769947813b92
Author: Jiayong Li <jiayong at math.mit.edu>
Date:   Tue Dec 6 11:06:21 2016 -0500

    Verify if each individual tag is empty in make_output_collection of __init__.py of arvados-cwl-runner

diff --git a/sdk/cwl/arvados_cwl/__init__.py b/sdk/cwl/arvados_cwl/__init__.py
index 5078b63..754abc8 100644
--- a/sdk/cwl/arvados_cwl/__init__.py
+++ b/sdk/cwl/arvados_cwl/__init__.py
@@ -262,13 +262,14 @@ class ArvCwlRunner(object):
                     final.api_response()["name"],
                     final.manifest_locator())
 
-        if tagsString is not None:
+        if tagsString:
             final_uuid = final.manifest_locator()
             tags = tagsString.split(',')
             for tag in tags:
-                 self.api.links().create(body={
-                    "head_uuid": final_uuid, "link_class": "tag", "name": tag
-                    }).execute(num_retries=self.num_retries)
+                if tag:
+                    self.api.links().create(body={
+                      "head_uuid": final_uuid, "link_class": "tag", "name": tag
+                      }).execute(num_retries=self.num_retries)
 
         def finalcollection(fileobj):
             fileobj["location"] = "keep:%s/%s" % (final.portable_data_hash(), fileobj["location"])

-----------------------------------------------------------------------


hooks/post-receive
--