ComparativeGenomicsToolkit / cactus

Official home of genome aligner based upon notion of Cactus graphs
Other
526 stars 111 forks source link

apply local TMPDIR redirect to all singularity commands #1518

Closed glennhickey closed 2 weeks ago

glennhickey commented 2 weeks ago

singularity exec takes in the environment from the caller, which in turn takes its TMPDIR from --workDir etc. But what's a valid directory outside the container, may not be, apparently, valid inside the container.

This had evidently come up before, as there's code to reset TMPDIR to . for cactus_consoldiated invocations. This PR applies this to any command run via singularity.

I think this may be a safer, more general patch than #1517 which addresses the same issue but for bcftools sort in particular.

This does not seem to be an issue with docker as docker run doesn't import the environment.