Skip to content

Commit 707f072

Browse files
committed
Further fix docker paths on mac
1 parent 7538c66 commit 707f072

File tree

2 files changed

+7
-11
lines changed

2 files changed

+7
-11
lines changed

cwltool/docker.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,7 +257,7 @@ def add_volumes(self, pathmapper, runtime, secret_store=None):
257257
f.write(contents.encode("utf-8"))
258258
runtime.append(u"--volume=%s:%s:rw" % (
259259
docker_windows_path_adjust(os.path.realpath(createtmp)),
260-
docker_windows_path_adjust(os.path.realpath(vol.target))))
260+
vol.target))
261261

262262
def create_runtime(self, env, runtimeContext):
263263
# type: (MutableMapping[Text, Text], RuntimeContext) -> List
@@ -309,7 +309,7 @@ def create_runtime(self, env, runtimeContext):
309309
if runtimeContext.rm_container:
310310
runtime.append(u"--rm")
311311

312-
runtime.append(u"--env=TMPDIR=%s" % self.builder.tmpdir)
312+
runtime.append(u"--env=TMPDIR=/tmp")
313313

314314
# spec currently says "HOME must be set to the designated output
315315
# directory." but spec might change to designated temp directory.

cwltool/process.py

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -623,16 +623,12 @@ def _init_job(self, joborder, runtimeContext):
623623
dockerReq.get("dockerOutputDirectory").startswith('/'):
624624
outdir = dockerReq.get("dockerOutputDirectory")
625625
else:
626-
outdir = fs_access.docker_compatible_realpath(
627-
dockerReq.get("dockerOutputDirectory") or
628-
runtimeContext.docker_outdir or "/var/spool/cwl")
626+
outdir = dockerReq.get("dockerOutputDirectory") or \
627+
runtimeContext.docker_outdir or "/var/spool/cwl"
629628
elif defaultDocker:
630-
outdir = fs_access.docker_compatible_realpath(
631-
runtimeContext.docker_outdir or "/var/spool/cwl")
632-
tmpdir = fs_access.docker_compatible_realpath(
633-
runtimeContext.docker_tmpdir or "/tmp")
634-
stagedir = fs_access.docker_compatible_realpath(
635-
runtimeContext.docker_stagedir or "/var/lib/cwl")
629+
outdir = runtimeContext.docker_outdir or "/var/spool/cwl"
630+
tmpdir = runtimeContext.docker_tmpdir or "/tmp"
631+
stagedir = runtimeContext.docker_stagedir or "/var/lib/cwl"
636632
else:
637633
outdir = fs_access.realpath(runtimeContext.outdir or
638634
tempfile.mkdtemp(prefix=getdefault(runtimeContext.tmp_outdir_prefix,

0 commit comments

Comments
 (0)