packages : moved decorators into AutotoolsPackage and CMakePackage

lib/spack/spack/package.py

@@ -1733,6 +1733,16 @@ def build(self, spec, prefix):
     def install(self, spec, prefix):
         inspect.getmodule(self).make('install')
 
+    @PackageBase.sanity_check('build')
+    @PackageBase.on_package_attributes(run_tests=True)
+    def _run_default_function(self):
+        try:
+            fn = getattr(self, 'check')
+            tty.msg('Trying default sanity checks [check]')
+            fn()
+        except AttributeError:
+            tty.msg('Skipping default sanity checks [method `check` not implemented]')  # NOQA: ignore=E501
+
     # This will be used as a registration decorator in user
     # packages, if need be
     PackageBase.sanity_check('install')(PackageBase.sanity_check_prefix)
@@ -1792,6 +1802,16 @@ def install(self, spec, prefix):
         with working_dir(self.wdir()):
             inspect.getmodule(self).make('install')
 
+    @PackageBase.sanity_check('build')
+    @PackageBase.on_package_attributes(run_tests=True)
+    def _run_default_function(self):
+        try:
+            fn = getattr(self, 'check')
+            tty.msg('Trying default sanity checks [check]')
+            fn()
+        except AttributeError:
+            tty.msg('Skipping default sanity checks [method `check` not implemented]')  # NOQA: ignore=E501
+
     PackageBase.sanity_check('install')(PackageBase.sanity_check_prefix)
 
 

var/spack/repos/builtin/packages/autoconf/package.py

@@ -25,10 +25,8 @@
 from spack import *
 
 
-class Autoconf(Package):
+class Autoconf(AutotoolsPackage):
-    """
+    """Autoconf -- system configuration part of autotools"""
-    Autoconf -- system configuration part of autotools
-    """
     homepage = 'https://www.gnu.org/software/autoconf/'
     url = 'http://ftp.gnu.org/gnu/autoconf/autoconf-2.69.tar.gz'
 
@@ -54,8 +52,3 @@ def setup_dependent_package(self, module, dependent_spec):
                        'ifnames']
         for name in executables:
             setattr(module, name, self._make_executable(name))
-
-    def install(self, spec, prefix):
-        configure("--prefix=%s" % prefix)
-        make()
-        make("install")

var/spack/repos/builtin/packages/hdf5/package.py

@@ -143,9 +143,7 @@ def configure_args(self):
 
         return ["--with-zlib=%s" % spec['zlib'].prefix] + extra_args
 
-    @AutotoolsPackage.sanity_check('install')
+    def check(self):
-    @AutotoolsPackage.on_package_attributes(run_tests=True)
-    def check_install(self):
         "Build and run a small program to test the installed HDF5 library"
         spec = self.spec
         print("Checking HDF5 installation...")

var/spack/repos/builtin/packages/lzo/package.py

@@ -43,8 +43,6 @@ def configure_args(self):
             '--enable-shared'
         ]
 
-    @AutotoolsPackage.sanity_check('build')
-    @AutotoolsPackage.on_package_attributes(run_tests=True)
     def check(self):
         make('check')
         make('test')

var/spack/repos/builtin/packages/qhull/package.py

@@ -45,7 +45,5 @@ class Qhull(CMakePackage):
 
     depends_on('cmake@2.6:', type='build')
 
-    @CMakePackage.sanity_check('build')
-    @CMakePackage.on_package_attributes(run_tests=True)
     def check(self):
         make('test')