summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--lib/spack/spack/hooks/module_file_generation.py20
1 files changed, 12 insertions, 8 deletions
diff --git a/lib/spack/spack/hooks/module_file_generation.py b/lib/spack/spack/hooks/module_file_generation.py
index e448cbc550..ba30561d86 100644
--- a/lib/spack/spack/hooks/module_file_generation.py
+++ b/lib/spack/spack/hooks/module_file_generation.py
@@ -3,19 +3,19 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.config
import spack.modules
import spack.modules.common
import llnl.util.tty as tty
-try:
- enabled = spack.config.get('modules:enable')
-except KeyError:
- tty.debug('NO MODULE WRITTEN: list of enabled module files is empty')
- enabled = []
-
def _for_each_enabled(spec, method_name):
"""Calls a method for each enabled module"""
+ enabled = spack.config.get('modules:enable')
+ if not enabled:
+ tty.debug('NO MODULE WRITTEN: list of enabled module files is empty')
+ return
+
for name in enabled:
generator = spack.modules.module_types[name](spec)
try:
@@ -26,5 +26,9 @@ def _for_each_enabled(spec, method_name):
tty.warn(msg.format(method_name, str(e)))
-post_install = lambda spec: _for_each_enabled(spec, 'write')
-post_uninstall = lambda spec: _for_each_enabled(spec, 'remove')
+def post_install(spec):
+ _for_each_enabled(spec, 'write')
+
+
+def post_uninstall(spec):
+ _for_each_enabled(spec, 'remove')