Skip to content

Fix exporting of module function in Bash#829

Merged
rtmclay merged 1 commit into
TACC:mainfrom
Flamefire:patch-1
Jun 16, 2026
Merged

Fix exporting of module function in Bash#829
rtmclay merged 1 commit into
TACC:mainfrom
Flamefire:patch-1

Conversation

@Flamefire

Copy link
Copy Markdown
Contributor

15b5772 introduced a helper method __lmod_internal_module_cmd, however this is not exported. Hence on subshells module and ml is defined but call the non-existing __lmod_internal_module_cmd function and fail.

15b5772 introduced a helper method `__lmod_internal_module_cmd`, however this is not exported.
Hence on subshells `module` and `ml` is defined but call the non-existing `__lmod_internal_module_cmd` function and fail.
@rtmclay rtmclay merged commit 70a8c55 into TACC:main Jun 16, 2026
9 checks passed
@rtmclay

rtmclay commented Jun 16, 2026

Copy link
Copy Markdown
Member

Thanks very much for the patch!

@boegel

boegel commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

@rtmclay Would be nice to have a quick new release with this fix included...

@Flamefire Flamefire deleted the patch-1 branch June 16, 2026 06:01
@rtmclay

rtmclay commented Jun 16, 2026

Copy link
Copy Markdown
Member

Lmod 9.2.4 is now released with this PR.

gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Jun 16, 2026
This version contained a bug on cluster login nodes.

See-also: TACC/Lmod#829
Signed-off-by: Alexander Puck Neuwirth <apn-pucky@gentoo.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants