Skip to content

add --module-cache-suffix configuration setting to allow multiple (Lmod) caches#4403

Merged
boegel merged 1 commit intoeasybuilders:developfrom
Flamefire:module_cache_suffix
Dec 20, 2023
Merged

add --module-cache-suffix configuration setting to allow multiple (Lmod) caches#4403
boegel merged 1 commit intoeasybuilders:developfrom
Flamefire:module_cache_suffix

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

@Flamefire Flamefire commented Dec 14, 2023

This is useful when having multiple module trees (i.e. different --installprefix) to not overwrite the cache file of one with the other when using --update-modules-tool-cache. Inspired by how LMod itself creates the cache, e.g. ~/.cache/lmod/spiderT.rapids_x86_64_Linux.lua

Actually the name is created by the default "groupName" hook

The logic is actually: spiderT.${LMOD_SYSTEM_NAME}_.$(uname -m)_$(uname -s).lua, see https://github.com/TACC/Lmod/blob/55d353b3a3a4649cbaaef18b2f00cdc017d7cc92/src/StandardPackage.lua#L88

This is useful when having multiple module trees (i.e. different `--installprefix`)
to not overwrite the cache file of one with the other when using `--update-modules-tool-cache`.
Inspired by how LMod itself creates the cache, e.g.
`~/.cache/lmod/spiderT.rapids_x86_64_Linux.lua`
@boegel boegel added this to the next release (4.9.0?) milestone Dec 20, 2023
@boegel boegel changed the title Introduce --module-cache-suffix to allow multiple caches add --module-cache-suffix configuration setting to allow multiple (Lmod) caches Dec 20, 2023
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@boegel boegel merged commit 89ac755 into easybuilders:develop Dec 20, 2023
@Flamefire Flamefire deleted the module_cache_suffix branch December 21, 2023 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants