From 81daaddfe91a1684e9d80c9fd42ca6ba2586ee33 Mon Sep 17 00:00:00 2001 From: Glenn Johnson Date: Mon, 11 Jul 2016 20:08:30 -0500 Subject: [PATCH 1/2] Use the site_packages_dir from the Python class It is not necessary to set site_packages_dir in this package as it is already provided by the Python package class. --- var/spack/repos/builtin/packages/libxml2/package.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/var/spack/repos/builtin/packages/libxml2/package.py b/var/spack/repos/builtin/packages/libxml2/package.py index 360386669a2..fe4bafe2e7d 100644 --- a/var/spack/repos/builtin/packages/libxml2/package.py +++ b/var/spack/repos/builtin/packages/libxml2/package.py @@ -43,9 +43,6 @@ class Libxml2(Package): def install(self, spec, prefix): if '+python' in spec: - site_packages_dir = os.path.join(prefix, - 'lib/python%s/site-packages' % - (spec['python'].version.up_to(2))) python_args = ["--with-python=%s" % spec['python'].prefix, "--with-python-install-dir=%s" % site_packages_dir] else: From 5e323b052aaa991b49b7ad6eeae7b3b10fdd0360 Mon Sep 17 00:00:00 2001 From: Glenn Johnson Date: Mon, 11 Jul 2016 20:21:25 -0500 Subject: [PATCH 2/2] Remove import of os The import of `os` is no longer needed. --- var/spack/repos/builtin/packages/libxml2/package.py | 1 - 1 file changed, 1 deletion(-) diff --git a/var/spack/repos/builtin/packages/libxml2/package.py b/var/spack/repos/builtin/packages/libxml2/package.py index fe4bafe2e7d..a6dabd2c05d 100644 --- a/var/spack/repos/builtin/packages/libxml2/package.py +++ b/var/spack/repos/builtin/packages/libxml2/package.py @@ -23,7 +23,6 @@ # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA ############################################################################## from spack import * -import os class Libxml2(Package):