Consistent docs and usage of env mod methods (#3351)

lib/spack/docs/packaging_guide.rst

@@ -1523,23 +1523,23 @@ properties to be used by dependents.
 
 The function declaration should look like this:
 
-.. code-block:: python
+.. literalinclude:: ../../../var/spack/repos/builtin/packages/qt/package.py
-
+   :pyobject: Qt.setup_dependent_environment
-   class Qt(Package):
+   :linenos:
-       ...
-       def setup_dependent_environment(self, module, spec, dep_spec):
-           """Dependencies of Qt find it using the QTDIR environment variable."""
-           os.environ['QTDIR'] = self.prefix
 
 Here, the Qt package sets the ``QTDIR`` environment variable so that
 packages that depend on a particular Qt installation will find it.
 
 The arguments to this function are:
 
-* **module**: the module of the dependent package, where global
+* **spack_env**: List of environment modifications to be applied when
-  properties can be assigned.
+  the dependent package is built within Spack.
-* **spec**: the spec of the *dependency package* (the one the function is called on).
+* **run_env**: List of environment modifications to be applied when
-* **dep_spec**: the spec of the dependent package (i.e. dep_spec depends on spec).
+  the dependent package is run outside of Spack. These are added to the
+  resulting module file.
+* **dependent_spec**: The spec of the dependent package about to be
+  built. This allows the extendee (self) to query the dependent's state.
+  Note that *this* package's spec is available as ``self.spec``.
 
 A good example of using these is in the Python package:
 
@@ -2805,11 +2805,8 @@ the one passed to install, only the MPI implementations all set some
 additional properties on it to help you out.  E.g., in mvapich2, you'll
 find this:
 
-.. code-block:: python
+.. literalinclude:: ../../../var/spack/repos/builtin/packages/mvapich2/package.py
-
+   :pyobject: Mvapich2.setup_dependent_package
-    def setup_dependent_package(self, module, dep_spec):
-        self.spec.mpicc  = join_path(self.prefix.bin, 'mpicc')
-        # … etc …
 
 That code allows the mvapich2 package to associate an ``mpicc`` property
 with the ``mvapich2`` node in the DAG, so that dependents can access it.

lib/spack/spack/package.py

@@ -1393,32 +1393,29 @@ def module(self):
     def setup_environment(self, spack_env, run_env):
         """Set up the compile and runtime environments for a package.
 
-        `spack_env` and `run_env` are `EnvironmentModifications`
+        ``spack_env`` and ``run_env`` are ``EnvironmentModifications``
         objects. Package authors can call methods on them to alter
         the environment within Spack and at runtime.
 
-        Both `spack_env` and `run_env` are applied within the build
+        Both ``spack_env`` and ``run_env`` are applied within the build
-        process, before this package's `install()` method is called.
+        process, before this package's ``install()`` method is called.
 
-        Modifications in `run_env` will *also* be added to the
+        Modifications in ``run_env`` will *also* be added to the
         generated environment modules for this package.
 
         Default implementation does nothing, but this can be
         overridden if the package needs a particular environment.
 
-        Examples:
+        Example:
 
-            1. Qt extensions need `QTDIR` set.
+        1. Qt extensions need ``QTDIR`` set.
 
-        Args:
+        :param EnvironmentModifications spack_env: List of environment
-            spack_env (EnvironmentModifications): list of
             modifications to be applied when this package is built
             within Spack.
-
+        :param EnvironmentModifications run_env: List of environment
-            run_env (EnvironmentModifications): list of environment
+            modifications to be applied when this package is run outside
-                changes to be applied when this package is run outside
+            of Spack. These are added to the resulting module file.
-                of Spack.
-
         """
         pass
 
@@ -1431,32 +1428,26 @@ def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         others that follow the extension model a way to implement
         common environment or compile-time settings for dependencies.
 
-        By default, this delegates to ``self.setup_environment()``
+        This is useful if there are some common steps to installing
+        all extensions for a certain package.
 
         Example:
 
-            1. Installing python modules generally requires
+        1. Installing python modules generally requires ``PYTHONPATH`` to point
-               `PYTHONPATH` to point to the lib/pythonX.Y/site-packages
+           to the ``lib/pythonX.Y/site-packages`` directory in the module's
-               directory in the module's install prefix.  This could
+           install prefix. This method could be used to set that variable.
-               set that variable.
 
-        Args:
+        :param EnvironmentModifications spack_env: List of environment
-
+            modifications to be applied when the dependent package is
-            spack_env (EnvironmentModifications): list of
+            built within Spack.
-                modifications to be applied when the dependent package
+        :param EnvironmentModifications run_env: List of environment
-                is bulit within Spack.
+            modifications to be applied when the dependent package is
-
+            run outside of Spack. These are added to the resulting
-            run_env (EnvironmentModifications): list of environment
+            module file.
-                changes to be applied when the dependent package is
+        :param Spec dependent_spec: The spec of the dependent package
-                run outside of Spack.
+            about to be built. This allows the extendee (self) to query
-
+            the dependent's state. Note that *this* package's spec is
-            dependent_spec (Spec): The spec of the dependent package
+            available as ``self.spec``.
-                about to be built. This allows the extendee (self) to
-                query the dependent's state. Note that *this*
-                package's spec is available as `self.spec`.
-
-        This is useful if there are some common steps to installing
-        all extensions for a certain package.
         """
         pass
 
@@ -1470,37 +1461,29 @@ def setup_dependent_package(self, module, dependent_spec):
         its extensions. This is useful if there are some common steps
         to installing all extensions for a certain package.
 
-        Example :
+        Examples:
 
-            1. Extensions often need to invoke the `python`
+        1. Extensions often need to invoke the ``python`` interpreter
-               interpreter from the Python installation being
+           from the Python installation being extended. This routine
-               extended.  This routine can put a 'python' Executable
+           can put a ``python()`` Executable object in the module scope
-               object in the module scope for the extension package to
+           for the extension package to simplify extension installs.
-               simplify extension installs.
 
-            2. MPI compilers could set some variables in the
+        2. MPI compilers could set some variables in the dependent's
-               dependent's scope that point to `mpicc`, `mpicxx`,
+           scope that point to ``mpicc``, ``mpicxx``, etc., allowing
-               etc., allowing them to be called by common names
+           them to be called by common name regardless of which MPI is used.
-               regardless of which MPI is used.
 
         3. BLAS/LAPACK implementations can set some variables
            indicating the path to their libraries, since these
            paths differ by BLAS/LAPACK implementation.
 
-        Args:
+        :param spack.package.PackageBase.module module: The Python ``module``
-
+            object of the dependent package. Packages can use this to set
-            module (module): The Python `module` object of the
-                dependent package. Packages can use this to set
             module-scope variables for the dependent to use.
 
-            dependent_spec (Spec): The spec of the dependent package
+        :param Spec dependent_spec: The spec of the dependent package
             about to be built. This allows the extendee (self) to
             query the dependent's state.  Note that *this*
-                package's spec is available as `self.spec`.
+            package's spec is available as ``self.spec``.
-
-        This is useful if there are some common steps to installing
-        all extensions for a certain package.
-
         """
         pass
 

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

@@ -53,7 +53,7 @@ def install(self, spec, prefix):
         mkdir(prefix.bin)
         install('output/bazel', prefix.bin)
 
-    def setup_dependent_package(self, module, dep_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         class BazelExecutable(Executable):
             """Special callable executable object for bazel so the user can
                specify parallel or not on a per-invocation basis.  Using
@@ -84,8 +84,8 @@ def __call__(self, *args, **kwargs):
                 return super(BazelExecutable, self).__call__(*args, **kwargs)
 
         jobs = cpu_count()
-        if not dep_spec.package.parallel:
+        if not dependent_spec.package.parallel:
             jobs = 1
-        elif dep_spec.package.make_jobs:
+        elif dependent_spec.package.make_jobs:
-            jobs = dep_spec.package.make_jobs
+            jobs = dependent_spec.package.make_jobs
         module.bazel = BazelExecutable('bazel', 'build', jobs)

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

@@ -287,5 +287,5 @@ def cmake_args(self):
 
         return options
 
-    def setup_environment(self, spack_env, env):
+    def setup_environment(self, spack_env, run_env):
-        env.set('DEAL_II_DIR', self.prefix)
+        run_env.set('DEAL_II_DIR', self.prefix)

var/spack/repos/builtin/packages/docbook-xml/package.py

@@ -42,7 +42,7 @@ def install(self, spec, prefix):
             else:
                 install(src, dst)
 
-    def setup_dependent_environment(self, spack_env, run_env, extension_spec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         catalog = os.path.join(self.spec.prefix, 'catalog.xml')
         spack_env.set('XML_CATALOG_FILES', catalog, separator=' ')
 

var/spack/repos/builtin/packages/docbook-xsl/package.py

@@ -44,7 +44,7 @@ def install(self, spec, prefix):
             else:
                 install(src, dst)
 
-    def setup_dependent_environment(self, spack_env, run_env, extension_spec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         catalog = os.path.join(self.spec.prefix, 'catalog.xml')
         spack_env.set('XML_CATALOG_FILES', catalog, separator=' ')
 

var/spack/repos/builtin/packages/everytrace-example/package.py

@@ -38,5 +38,5 @@ class EverytraceExample(CMakePackage):
     # Currently the only MPI this everytrace works with.
     depends_on('openmpi')
 
-    def setup_environment(self, spack_env, env):
+    def setup_environment(self, spack_env, run_env):
-        env.prepend_path('PATH', join_path(self.prefix, 'bin'))
+        run_env.prepend_path('PATH', join_path(self.prefix, 'bin'))

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

@@ -47,5 +47,5 @@ def cmake_args(self):
             '-DUSE_MPI=%s' % ('YES' if '+mpi' in spec else 'NO'),
             '-DUSE_FORTRAN=%s' % ('YES' if '+fortran' in spec else 'NO')]
 
-    def setup_environment(self, spack_env, env):
+    def setup_environment(self, spack_env, run_env):
-        env.prepend_path('PATH', join_path(self.prefix, 'bin'))
+        run_env.prepend_path('PATH', join_path(self.prefix, 'bin'))

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

@@ -53,8 +53,8 @@ def install(self, spec, prefix):
 
     # In theory the 'run' dependency on 'jdk' above should take
     # care of this for me. In practice, it does not.
-    def setup_environment(self, spack_env, env):
+    def setup_environment(self, spack_env, run_env):
         """Add <prefix> to the path; the package has a script at the
            top level.
         """
-        env.prepend_path('PATH', join_path(self.spec['jdk'].prefix, 'bin'))
+        run_env.prepend_path('PATH', join_path(self.spec['jdk'].prefix, 'bin'))

var/spack/repos/builtin/packages/go-bootstrap/package.py

@@ -87,7 +87,7 @@ def install(self, spec, prefix):
             else:
                 shutil.copy2(f, os.path.join(prefix, f))
 
-    def setup_dependent_environment(self, spack_env, run_env, dep_spec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         spack_env.set('GOROOT_BOOTSTRAP', self.spec.prefix)
 
     def setup_environment(self, spack_env, run_env):

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

@@ -114,7 +114,7 @@ def install(self, spec, prefix):
     def setup_environment(self, spack_env, run_env):
         spack_env.set('GOROOT_FINAL', self.spec.prefix)
 
-    def setup_dependent_package(self, module, ext_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         """Called before go modules' install() methods.
 
         In most cases, extensions will only need to set GOPATH and use go::
@@ -127,13 +127,13 @@ def setup_dependent_package(self, module, ext_spec):
         #  Add a go command/compiler for extensions
         module.go = Executable(join_path(self.spec.prefix.bin, 'go'))
 
-    def setup_dependent_environment(self, spack_env, run_env, ext_spec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         if os.environ.get('GOROOT', False):
             tty.warn('GOROOT is set, this is not recommended')
 
         path_components = []
         # Set GOPATH to include paths of dependencies
-        for d in ext_spec.traverse():
+        for d in dependent_spec.traverse():
             if d.package.extends(self.spec):
                 path_components.append(d.prefix)
 
@@ -142,4 +142,4 @@ def setup_dependent_environment(self, spack_env, run_env, ext_spec):
 
         # Allow packages to find this when using module or dotkit
         run_env.prepend_path('GOPATH', ':'.join(
-            [ext_spec.prefix] + path_components))
+            [dependent_spec.prefix] + path_components))

var/spack/repos/builtin/packages/intel-mkl/package.py

@@ -125,5 +125,5 @@ def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         # set up MKLROOT for everyone using MKL package
         spack_env.set('MKLROOT', self.prefix)
 
-    def setup_environment(self, spack_env, env):
+    def setup_environment(self, spack_env, run_env):
-        env.set('MKLROOT', self.prefix)
+        run_env.set('MKLROOT', self.prefix)

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

@@ -86,9 +86,9 @@ def append_paths(self, paths, cpaths, path):
         paths.append(os.path.join(path, '?', 'init.lua'))
         cpaths.append(os.path.join(path, '?.so'))
 
-    def setup_dependent_environment(self, spack_env, run_env, extension_spec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         lua_paths = []
-        for d in extension_spec.traverse(
+        for d in dependent_spec.traverse(
                 deptypes=('build', 'run'), deptype_query='run'):
             if d.package.extends(self.spec):
                 lua_paths.append(os.path.join(d.prefix, self.lua_lib_dir))
@@ -111,9 +111,9 @@ def setup_dependent_environment(self, spack_env, run_env, extension_spec):
         # Add LUA to PATH for dependent packages
         spack_env.prepend_path('PATH', self.prefix.bin)
 
-        # For run time environment set only the path for extension_spec and
+        # For run time environment set only the path for dependent_spec and
         # prepend it to LUAPATH
-        if extension_spec.package.extends(self.spec):
+        if dependent_spec.package.extends(self.spec):
             run_env.prepend_path('LUA_PATH', ';'.join(lua_patterns),
                                  separator=';')
             run_env.prepend_path('LUA_CPATH', ';'.join(lua_cpatterns),
@@ -149,7 +149,7 @@ def lua_lib_dir(self):
     def lua_share_dir(self):
         return os.path.join('share', 'lua', self.version.up_to(2))
 
-    def setup_dependent_package(self, module, ext_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         """
         Called before lua modules's install() methods.
 

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

@@ -69,7 +69,7 @@ def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         spack_env.set('MPICH_F90', spack_fc)
         spack_env.set('MPICH_FC', spack_fc)
 
-    def setup_dependent_package(self, module, dep_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         if 'platform=cray' in self.spec:
             self.spec.mpicc = spack_cc
             self.spec.mpicxx = spack_cxx

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

@@ -209,7 +209,7 @@ def setup_environment(self, spack_env, run_env):
            self.version > Version('2.0'):
             run_env.set('SLURM_MPI_TYPE', 'pmi2')
 
-    def setup_dependent_environment(self, spack_env, run_env, extension_spec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         spack_env.set('MPICC',  join_path(self.prefix.bin, 'mpicc'))
         spack_env.set('MPICXX', join_path(self.prefix.bin, 'mpicxx'))
         spack_env.set('MPIF77', join_path(self.prefix.bin, 'mpif77'))
@@ -221,7 +221,7 @@ def setup_dependent_environment(self, spack_env, run_env, extension_spec):
         spack_env.set('MPICH_F90', spack_fc)
         spack_env.set('MPICH_FC', spack_fc)
 
-    def setup_dependent_package(self, module, dep_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         self.spec.mpicc  = join_path(self.prefix.bin, 'mpicc')
         self.spec.mpicxx = join_path(self.prefix.bin, 'mpicxx')
         self.spec.mpifc  = join_path(self.prefix.bin, 'mpif90')

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

@@ -225,7 +225,7 @@ def configure_args(self):
     # Set up environment to make install easy for Octave extensions.
     # ========================================================================
 
-    def setup_dependent_package(self, module, ext_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         """Called before Octave modules' install() methods.
 
         In most cases, extensions will only need to have one line:

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

@@ -139,7 +139,7 @@ def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         spack_env.set('OMPI_FC', spack_fc)
         spack_env.set('OMPI_F77', spack_f77)
 
-    def setup_dependent_package(self, module, dep_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         self.spec.mpicc = join_path(self.prefix.bin, 'mpicc')
         self.spec.mpicxx = join_path(self.prefix.bin, 'mpic++')
         self.spec.mpifc = join_path(self.prefix.bin, 'mpif90')

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

@@ -99,7 +99,7 @@ def apply_patch(self, other):
         plumed.stdin.write(choice)
         plumed.wait()
 
-    def setup_dependent_package(self, module, ext_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         # Make plumed visible from dependent packages
         module.plumed = Executable(join_path(self.spec.prefix.bin, 'plumed'))
 

var/spack/repos/builtin/packages/py-numpy/package.py

@@ -53,7 +53,7 @@ class PyNumpy(PythonPackage):
     depends_on('blas',   when='+blas')
     depends_on('lapack', when='+lapack')
 
-    def setup_dependent_package(self, module, dep_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         python_version = self.spec['python'].version.up_to(2)
         arch = '{0}-{1}'.format(platform.system().lower(), platform.machine())
 

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

@@ -335,7 +335,7 @@ def python_include_dir(self):
     def site_packages_dir(self):
         return join_path(self.python_lib_dir, 'site-packages')
 
-    def setup_dependent_environment(self, spack_env, run_env, extension_spec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         """Set PYTHONPATH to include site-packages dir for the
         extension and any other python extensions it depends on."""
         # The python executable for version 3 may be python3 or python
@@ -361,7 +361,7 @@ def setup_dependent_environment(self, spack_env, run_env, extension_spec):
         spack_env.set('PYTHONHOME', prefix.strip('\n'))
 
         python_paths = []
-        for d in extension_spec.traverse(
+        for d in dependent_spec.traverse(
                 deptype=('build', 'run'), deptype_query='run'):
             if d.package.extends(self.spec):
                 python_paths.append(join_path(d.prefix,
@@ -371,12 +371,12 @@ def setup_dependent_environment(self, spack_env, run_env, extension_spec):
         spack_env.set('PYTHONPATH', pythonpath)
 
         # For run time environment set only the path for
-        # extension_spec and prepend it to PYTHONPATH
+        # dependent_spec and prepend it to PYTHONPATH
-        if extension_spec.package.extends(self.spec):
+        if dependent_spec.package.extends(self.spec):
             run_env.prepend_path('PYTHONPATH', join_path(
-                extension_spec.prefix, self.site_packages_dir))
+                dependent_spec.prefix, self.site_packages_dir))
 
-    def setup_dependent_package(self, module, ext_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         """Called before python modules' install() methods.
 
         In most cases, extensions will only need to have one line::
@@ -398,15 +398,15 @@ def setup_dependent_package(self, module, ext_spec):
                 module.setup_py.add_default_env(key, value)
 
         # Add variables for lib/pythonX.Y and lib/pythonX.Y/site-packages dirs.
-        module.python_lib_dir = join_path(ext_spec.prefix,
+        module.python_lib_dir = join_path(dependent_spec.prefix,
                                           self.python_lib_dir)
-        module.python_include_dir = join_path(ext_spec.prefix,
+        module.python_include_dir = join_path(dependent_spec.prefix,
                                               self.python_include_dir)
-        module.site_packages_dir = join_path(ext_spec.prefix,
+        module.site_packages_dir = join_path(dependent_spec.prefix,
                                              self.site_packages_dir)
 
         # Make the site packages directory for extensions
-        if ext_spec.package.is_extension:
+        if dependent_spec.package.is_extension:
             mkdirp(module.site_packages_dir)
 
     # ========================================================================

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

@@ -128,10 +128,10 @@ def url_for_version(self, version):
     def setup_environment(self, spack_env, run_env):
         run_env.set('QTDIR', self.prefix)
 
-    def setup_dependent_environment(self, spack_env, run_env, dspec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         spack_env.set('QTDIR', self.prefix)
 
-    def setup_dependent_package(self, module, ext_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         module.qmake = Executable(join_path(self.spec.prefix.bin, 'qmake'))
 
     def patch(self):

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

@@ -149,11 +149,11 @@ def filter_compilers(self):
     def r_lib_dir(self):
         return join_path('rlib', 'R', 'library')
 
-    def setup_dependent_environment(self, spack_env, run_env, extension_spec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         # Set R_LIBS to include the library dir for the
         # extension and any other R extensions it depends on.
         r_libs_path = []
-        for d in extension_spec.traverse(
+        for d in dependent_spec.traverse(
                 deptype=('build', 'run'), deptype_query='run'):
             if d.package.extends(self.spec):
                 r_libs_path.append(join_path(d.prefix, self.r_lib_dir))
@@ -167,11 +167,11 @@ def setup_dependent_environment(self, spack_env, run_env, extension_spec):
         # determine how many jobs can actually be started.
         spack_env.set('MAKEFLAGS', '-j{0}'.format(make_jobs))
 
-        # For run time environment set only the path for extension_spec and
+        # For run time environment set only the path for dependent_spec and
         # prepend it to R_LIBS
-        if extension_spec.package.extends(self.spec):
+        if dependent_spec.package.extends(self.spec):
             run_env.prepend_path('R_LIBS', join_path(
-                extension_spec.prefix, self.r_lib_dir))
+                dependent_spec.prefix, self.r_lib_dir))
 
     def setup_environment(self, spack_env, run_env):
         run_env.prepend_path('LIBRARY_PATH',
@@ -181,7 +181,7 @@ def setup_environment(self, spack_env, run_env):
         run_env.prepend_path('CPATH',
                              join_path(self.prefix, 'rlib', 'R', 'include'))
 
-    def setup_dependent_package(self, module, ext_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         """Called before R modules' install() methods. In most cases,
         extensions will only need to have one line:
             R('CMD', 'INSTALL', '--library={0}'.format(self.module.r_lib_dir),
@@ -191,9 +191,9 @@ def setup_dependent_package(self, module, ext_spec):
         module.R = Executable(join_path(self.spec.prefix.bin, 'R'))
 
         # Add variable for library directry
-        module.r_lib_dir = join_path(ext_spec.prefix, self.r_lib_dir)
+        module.r_lib_dir = join_path(dependent_spec.prefix, self.r_lib_dir)
 
         # Make the site packages directory for extensions, if it does not exist
         # already.
-        if ext_spec.package.is_extension:
+        if dependent_spec.package.is_extension:
             mkdirp(module.r_lib_dir)

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

@@ -79,7 +79,7 @@ def install(self, spec, prefix):
             make()
             make("install")
 
-    def setup_dependent_environment(self, spack_env, run_env, dspec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         spack_env.set('ROOTSYS', self.prefix)
         spack_env.set('ROOT_VERSION', 'v6')
         spack_env.prepend_path('PYTHONPATH', self.prefix.lib)

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

@@ -52,20 +52,20 @@ def install(self, spec, prefix):
         make()
         make("install")
 
-    def setup_dependent_environment(self, spack_env, run_env, extension_spec):
+    def setup_dependent_environment(self, spack_env, run_env, dependent_spec):
         # TODO: do this only for actual extensions.
         # Set GEM_PATH to include dependent gem directories
         ruby_paths = []
-        for d in extension_spec.traverse():
+        for d in dependent_spec.traverse():
             if d.package.extends(self.spec):
                 ruby_paths.append(d.prefix)
 
         spack_env.set_path('GEM_PATH', ruby_paths)
 
         # The actual installation path for this gem
-        spack_env.set('GEM_HOME', extension_spec.prefix)
+        spack_env.set('GEM_HOME', dependent_spec.prefix)
 
-    def setup_dependent_package(self, module, ext_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         """Called before ruby modules' install() methods.  Sets GEM_HOME
         and GEM_PATH to values appropriate for the package being built.
 

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

@@ -71,7 +71,7 @@ def install(self, spec, prefix):
             make()
             make("install")
 
-    def setup_dependent_package(self, module, ext_spec):
+    def setup_dependent_package(self, module, dependent_spec):
         """
         Called before python modules' install() methods.
 

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

@@ -49,10 +49,10 @@ def url_for_version(self, version):
         base_url = 'http://prdownloads.sourceforge.net/tcl'
         return '{0}/tcl{1}-src.tar.gz'.format(base_url, version)
 
-    def setup_environment(self, spack_env, env):
+    def setup_environment(self, spack_env, run_env):
         # When using Tkinter from within spack provided python+tk, python
         # will not be able to find Tcl/Tk unless TCL_LIBRARY is set.
-        env.set('TCL_LIBRARY', join_path(self.prefix.lib, 'tcl{0}'.format(
+        run_env.set('TCL_LIBRARY', join_path(self.prefix.lib, 'tcl{0}'.format(
             self.spec.version.up_to(2))))
 
     @run_after('install')