Repository: spack/spack

Issue:
build_environment.py: 'module' object has no attribute 'PackageBase'
Not sure how to deal with the error whose message I appended below ...

Adding the base class to "spack.package import Package" does the trick, but I have no idea what I am doing ...

`1 file changed, 1 insertion(+), 1 deletion(-)
lib/spack/spack/__init__.py | 2 +-

modified   lib/spack/spack/__init__.py
@@ -160,7 +160,7 @@ dirty = _config.get('dirty', False)
 #-----------------------------------------------------------------------------
 __all__ = []
 
-from spack.package import Package
+from spack.package import Package, PackageBase
 from spack.build_systems.makefile import MakefilePackage
 from spack.build_systems.autotools import AutotoolsPackage
 from spack.build_systems.cmake import CMakePackage
`
-------------------- error ----------------------

`==> Executing phase : 'install'
==> WRITE LOCK: /u/proj2/proj/trm/spack-emacs/opt/spack/.spack-db/prefix_lock[2843439391484420822:1] [Released]
==> WRITE LOCK: /u/proj2/proj/trm/spack-emacs/var/spack/stage/.lock[7968242201785533596:1] [Released]
Process Process-1:
Traceback (most recent call last):
  File "/usr/lib64/python2.6/multiprocessing/process.py", line 232, in _bootstrap
    self.run()
  File "/usr/lib64/python2.6/multiprocessing/process.py", line 88, in run
    self._target(*self._args, **self._kwargs)
  File "/u/proj2/proj/trm/spack-emacs/lib/spack/spack/build_environment.py", line 564, in child_execution
    package_context = get_package_context(tb)
  File "/u/proj2/proj/trm/spack-emacs/lib/spack/spack/build_environment.py", line 627, in get_package_context
    if isinstance(obj, spack.PackageBase):
AttributeError: 'module' object has no attribute 'PackageBase'
^CSignal received : entering python shell.
Traceback:
  File "/home/trm/spack/bin/spack", line 212, in <module>
    main(sys.argv)
  File "/home/trm/spack/bin/spack", line 208, in main
    _main(args, unknown)
  File "/home/trm/spack/bin/spack", line 174, in _main
    return_val = command(parser, args)
  File "/u/proj2/proj/trm/spack-emacs/lib/spack/spack/cmd/install.py", line 346, in install
    package.do_install(**kwargs)
  File "/u/proj2/proj/trm/spack-emacs/lib/spack/spack/package.py", line 1185, in do_install
    **kwargs
  File "/u/proj2/proj/trm/spack-emacs/lib/spack/spack/package.py", line 1185, in do_install
    **kwargs
  File "/u/proj2/proj/trm/spack-emacs/lib/spack/spack/package.py", line 1185, in do_install
    **kwargs
  File "/u/proj2/proj/trm/spack-emacs/lib/spack/spack/package.py", line 1185, in do_install
    **kwargs
  File "/u/proj2/proj/trm/spack-emacs/lib/spack/spack/package.py", line 1283, in do_install
    spack.build_environment.fork(self, build_process, dirty=dirty)
  File "/u/proj2/proj/trm/spack-emacs/lib/spack/spack/build_environment.py", line 592, in fork
    child_exc = parent_connection.recv()

`


Discussion:

Resolving patch:
diff --git a/lib/spack/spack/build_environment.py b/lib/spack/spack/build_environment.py
--- a/lib/spack/spack/build_environment.py
+++ b/lib/spack/spack/build_environment.py
@@ -60,9 +60,10 @@
 
 import llnl.util.lang as lang
 import llnl.util.tty as tty
+from llnl.util.filesystem import *
+
 import spack
 import spack.store
-from llnl.util.filesystem import *
 from spack.environment import EnvironmentModifications, validate
 from spack.util.environment import *
 from spack.util.executable import Executable, which
@@ -450,7 +451,8 @@ def parent_class_modules(cls):
     """
     Get list of super class modules that are all descend from spack.Package
     """
-    if not issubclass(cls, spack.Package) or issubclass(spack.Package, cls):
+    if (not issubclass(cls, spack.package.Package) or
+        issubclass(spack.package.Package, cls)):
         return []
     result = []
     module = sys.modules.get(cls.__module__)
@@ -622,9 +624,9 @@ def make_stack(tb, stack=None):
     for tb in stack:
         frame = tb.tb_frame
         if 'self' in frame.f_locals:
-            # Find the first proper subclass of spack.PackageBase.
+            # Find the first proper subclass of PackageBase.
             obj = frame.f_locals['self']
-            if isinstance(obj, spack.PackageBase):
+            if isinstance(obj, spack.package.PackageBase):
                 break
 
     # we found obj, the Package implementation we care about.
diff --git a/var/spack/repos/builtin.mock/packages/failing-build/package.py b/var/spack/repos/builtin.mock/packages/failing-build/package.py
new file mode 100644
--- /dev/null
+++ b/var/spack/repos/builtin.mock/packages/failing-build/package.py
@@ -0,0 +1,37 @@
+##############################################################################
+# Copyright (c) 2013-2016, Lawrence Livermore National Security, LLC.
+# Produced at the Lawrence Livermore National Laboratory.
+#
+# This file is part of Spack.
+# Created by Todd Gamblin, tgamblin@llnl.gov, All rights reserved.
+# LLNL-CODE-647188
+#
+# For details, see https://github.com/llnl/spack
+# Please also see the LICENSE file for our notice and the LGPL.
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU Lesser General Public License (as
+# published by the Free Software Foundation) version 2.1, February 1999.
+#
+# This program is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the IMPLIED WARRANTY OF
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the terms and
+# conditions of the GNU Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public
+# License along with this program; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+##############################################################################
+from spack import *
+
+
+class FailingBuild(Package):
+    """This package has a trivial install method that fails."""
+
+    homepage = "http://www.example.com/trivial_install"
+    url      = "http://www.unit-test-should-replace-this-url/trivial_install-1.0.tar.gz"
+
+    version('1.0', 'foobarbaz')
+
+    def install(self, spec, prefix):
+        raise InstallError("Expected failure.")


Test patch:
diff --git a/lib/spack/spack/test/install.py b/lib/spack/spack/test/install.py
--- a/lib/spack/spack/test/install.py
+++ b/lib/spack/spack/test/install.py
@@ -90,3 +90,15 @@ def test_store(mock_archive):
     except Exception:
         pkg.remove_prefix()
         raise
+
+
+@pytest.mark.usefixtures('install_mockery')
+def test_failing_build(mock_archive):
+    spec = Spec('failing-build').concretized()
+
+    for s in spec.traverse():
+        fake_fetchify(mock_archive.url, s.package)
+
+    pkg = spec.package
+    with pytest.raises(spack.build_environment.ChildError):
+        pkg.do_install()