hlrs-software-stack/spack
7
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

TensorFlow: don't override HOME env var (#14054)

Browse source
This commit is contained in:
Adam J. Stewart 2019-12-09 11:22:57 -06:00 committed by GitHub
parent 919a5421b7
commit 23faffa2d0
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
var/spack/repos/builtin/packages/py-tensorflow/package.py
View file

@ -487,9 +487,6 @@ def setup_build_environment(self, env):
tmp_path = '/tmp/spack/tf'
mkdirp(tmp_path)
env.set('TEST_TMPDIR', tmp_path)
# TODO: Is setting this necessary? It breaks `spack build-env`
# because Bash can't find my .bashrc
env.set('HOME', tmp_path)
def configure(self, spec, prefix):
# NOTE: configure script is interactive. If you set the appropriate
@ -592,11 +589,15 @@ def post_configure_fixes(self):
'.tf_configure.bazelrc')
def build(self, spec, prefix):
tmp_path = env['TEST_TMPDIR']
# https://docs.bazel.build/versions/master/command-line-reference.html
args = [
# Don't allow user or system .bazelrc to override build settings
'--nohome_rc',
'--nosystem_rc',
# Bazel does not work properly on NFS, switch to /tmp
'--output_user_root=' + tmp_path,
'build',
# Spack logs don't handle colored output well
'--color=no',
@ -665,7 +666,6 @@ def build(self, spec, prefix):
build_pip_package = Executable(
'bazel-bin/tensorflow/tools/pip_package/build_pip_package')
tmp_path = env['TEST_TMPDIR']
build_pip_package(tmp_path)
def install(self, spec, prefix):