Skip to content

Commit d4287d7

Browse files
authored
Set $HOME and $TMPDIR correctly for non-Docker command line tools. (#149)
1 parent e6843b9 commit d4287d7

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

cwltool/job.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -72,11 +72,6 @@ def run(self, dry_run=False, pull_image=True, rm_container=True,
7272

7373
runtime = [] # type: List[unicode]
7474

75-
# spec currently says "HOME must be set to the designated output
76-
# directory." but spec might change to designated temp directory.
77-
# env = {"TMPDIR": self.tmpdir, "HOME": self.tmpdir} # type: Mapping[str,str]
78-
env = {"TMPDIR": self.tmpdir, "HOME": self.outdir} # type: Mapping[str,str]
79-
8075
(docker_req, docker_is_req) = get_feature(self, "DockerRequirement")
8176

8277
for knownfile in self.pathmapper.files():
@@ -139,12 +134,13 @@ def run(self, dry_run=False, pull_image=True, rm_container=True,
139134
env = self.environment
140135
if not os.path.exists(self.tmpdir):
141136
os.makedirs(self.tmpdir)
142-
env["TMPDIR"] = self.tmpdir
143137
vars_to_preserve = kwargs.get("preserve_environment")
144138
if vars_to_preserve is not None:
145139
for key, value in os.environ.items():
146140
if key in vars_to_preserve and key not in env:
147141
env[key] = value
142+
env["HOME"] = self.outdir
143+
env["TMPDIR"] = self.tmpdir
148144

149145
stageFiles(self.pathmapper, os.symlink)
150146

0 commit comments

Comments
 (0)