Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

Flamefire
Copy link
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
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
40 checks passed
@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