zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tcl module template: automatically unload automatically loaded modules (#32853)

Browse Source 
Remove `module-info mode load` condition that prevents auto-unloading when autoloading is enabled. It looks like this condition was added to work around an issue in environment-modules that is no longer necessary.

Add quotes to make is-loaded happy
This commit is contained in:
Sergey Kosukhin 2022-10-06 20:15:28 +02:00 committed by GitHub
parent c3018f95ee
commit a520a7ef28
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/spack/spack/test/modules/tcl.py
View File

@ -113,7 +113,7 @@ def test_alter_environment(self, modulefile_content, module_configuration, confi
assert len([x for x in content if x.startswith("prepend-path CMAKE_PREFIX_PATH")]) == 0
assert len([x for x in content if 'setenv FOO "foo"' in x]) == 0
assert len([x for x in content if "unsetenv BAR" in x]) == 0
assert len([x for x in content if "is-loaded foo/bar" in x]) == 1
assert len([x for x in content if "is-loaded 'foo/bar'" in x]) == 1
assert len([x for x in content if "module load foo/bar" in x]) == 1
assert len([x for x in content if "setenv LIBDWARF_ROOT" in x]) == 1

2
share/spack/templates/modules/modulefile.tcl
View File

@ -23,7 +23,7 @@ proc ModulesHelp { } {
{% block autoloads %}
{% for module in autoload %}
if {{ '{' }} [ module-info mode load ] && ![ is-loaded {{ module }} ] {{ '}' }} {{ '{' }}
if {{ '{' }} ![ is-loaded '{{ module }}' ] {{ '}' }} {{ '{' }}
{% if verbose %}
puts stderr "Autoloading {{ module }}"
{% endif %}