summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGreg Becker <becker33@llnl.gov>2020-03-19 15:11:50 -0700
committerTodd Gamblin <tgamblin@llnl.gov>2020-03-20 11:47:10 -0700
commit30b4704522316263fff07fc75146f0a7a6dc7952 (patch)
treed8c1dba87853c075b334bc301ea16667acafaf99
parent09e13cf7cf0327bb35a9b3d4e7c7ad64ea4a3076 (diff)
downloadspack-30b4704522316263fff07fc75146f0a7a6dc7952.tar.gz
spack-30b4704522316263fff07fc75146f0a7a6dc7952.tar.bz2
spack-30b4704522316263fff07fc75146f0a7a6dc7952.tar.xz
spack-30b4704522316263fff07fc75146f0a7a6dc7952.zip
Cray bugfix: TERM missing while reading default target (#15381)
Bug: Spack hangs on some Cray machines Reason: The TERM environment variable is necessary to run bash -lc "echo $CRAY_CPU_TARGET", but we run that command within env -i, which wipes the environment. Fix: Manually forward the TERM environment variable to env -i /bin/bash -lc "echo $CRAY_CPU_TARGET"
-rw-r--r--lib/spack/spack/platforms/cray.py14
1 files changed, 8 insertions, 6 deletions
diff --git a/lib/spack/spack/platforms/cray.py b/lib/spack/spack/platforms/cray.py
index cbeba96461..8eae51e66a 100644
--- a/lib/spack/spack/platforms/cray.py
+++ b/lib/spack/spack/platforms/cray.py
@@ -7,7 +7,7 @@ import os
import re
import llnl.util.tty as tty
from spack.paths import build_env_path
-from spack.util.executable import which
+from spack.util.executable import Executable
from spack.architecture import Platform, Target, NoPlatformError
from spack.operating_systems.cray_frontend import CrayFrontend
from spack.operating_systems.cnl import Cnl
@@ -117,11 +117,13 @@ class Cray(Platform):
'''
# env -i /bin/bash -lc echo $CRAY_CPU_TARGET 2> /dev/null
if getattr(self, 'default', None) is None:
- env = which('env')
- output = env("-i", "/bin/bash", "-lc", "echo $CRAY_CPU_TARGET",
- output=str, error=os.devnull)
- self.default = output.strip()
- tty.debug("Found default module:%s" % self.default)
+ output = Executable('/bin/bash')('-lc', 'echo $CRAY_CPU_TARGET',
+ env={'TERM': os.environ['TERM']},
+ output=str, error=os.devnull)
+ output = ''.join(output.split()) # remove all whitespace
+ if output:
+ self.default = output
+ tty.debug("Found default module:%s" % self.default)
return self.default
def _avail_targets(self):