From 7504e9997db3ce355ab0380f316ca7427b2a5c9a Mon Sep 17 00:00:00 2001 From: Gregory Lee Date: Wed, 21 Mar 2018 14:34:27 -0700 Subject: make py-setuptools a run-time-only dep for py-basemap and patch pytho… (#6989) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * make py-setuptools a run-time-only dep for py-basemap and patch python package to only apply setuptools flag for build deps * py-qtconsole does not require setuptools --- lib/spack/spack/build_systems/python.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'lib') diff --git a/lib/spack/spack/build_systems/python.py b/lib/spack/spack/build_systems/python.py index fcfa68e4f4..837ebd65ae 100644 --- a/lib/spack/spack/build_systems/python.py +++ b/lib/spack/spack/build_systems/python.py @@ -237,9 +237,15 @@ class PythonPackage(PackageBase): # Spack manages the package directory on its own by symlinking # extensions into the site-packages directory, so we don't really # need the .pth files or egg directories, anyway. + # + # We need to make sure this is only for build dependencies. A package + # such as py-basemap will not build properly with this flag since + # it does not use setuptools to build and those does not recognize + # the --single-version-externally-managed flag if ('py-setuptools' == spec.name or # this is setuptools, or - 'py-setuptools' in spec._dependencies): # it's an immediate dep - args += ['--single-version-externally-managed', '--root=/'] + 'py-setuptools' in spec._dependencies and # it's an immediate dep + 'build' in spec._dependencies['py-setuptools'].deptypes): + args += ['--single-version-externally-managed', '--root=/'] return args -- cgit v1.2.3-70-g09d2