Skip to content

Commit

Permalink
Resolved issue: gatk-launch appends as needed to GATK_GCS_STAGING #1338
Browse files Browse the repository at this point in the history
… (#5452)

* Resolved the issue of adding gs:// to the beginning and / to the end of the environment variable GATK_GCS_STAGING. Tested it locally.

* Changed a line of code in gatk to a better style (as suggested by David Benjamin -- thanks).
  • Loading branch information
MartonKN authored Nov 29, 2018
1 parent e258888 commit 65a681f
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions gatk
Original file line number Diff line number Diff line change
Expand Up @@ -283,6 +283,13 @@ def md5(file):

def cacheJarOnGCS(jar, dryRun):
staging = os.environ.get("GATK_GCS_STAGING")

if staging is not None:
staging = staging.strip('/')
staging = staging + '/'
if not staging.startswith("gs://"):
staging = "gs://" + staging

if dryRun is True:
return jar
elif staging is None:
Expand Down

0 comments on commit 65a681f

Please sign in to comment.