Skip to content

make delattr aware of environment variables in ModuleLoadEnvironment#4784

Merged
boegel merged 9 commits intoeasybuilders:5.0.xfrom
lexming:fix-delattr-modloa
Mar 12, 2025
Merged

make delattr aware of environment variables in ModuleLoadEnvironment#4784
boegel merged 9 commits intoeasybuilders:5.0.xfrom
lexming:fix-delattr-modloa

Conversation

@lexming
Copy link
Copy Markdown
Contributor

@lexming lexming commented Mar 3, 2025

There are two ways to remove attributes from ModuleLoadEnvironment:

  • ModuleLoadEnvironment.remove("ENV_VAR") will ignore missing attributes
  • delattr(ModuleLoadEnvironment, "ENV_VAR") will error on missing attributes

@lexming lexming added bug fix EasyBuild-5.0 EasyBuild 5.0 labels Mar 3, 2025
@lexming lexming added this to the 5.0 milestone Mar 3, 2025
Comment thread easybuild/tools/modules.py
@Micket Micket moved this to Blockers in EasyBuild v5.0 Mar 3, 2025
Comment thread easybuild/tools/modules.py Outdated
Comment thread easybuild/tools/modules.py Outdated
@boegel boegel changed the title make delattr aware of environment variables in ModuleLoadEnvironment make delattr aware of environment variables in ModuleLoadEnvironment Mar 12, 2025
@boegel boegel changed the title make delattr aware of environment variables in ModuleLoadEnvironment make delattr aware of environment variables in ModuleLoadEnvironment Mar 12, 2025
@boegel boegel merged commit fc15f5e into easybuilders:5.0.x Mar 12, 2025
37 checks passed
@lexming lexming deleted the fix-delattr-modloa branch March 12, 2025 17:16
@branfosj branfosj removed the status in EasyBuild v5.0 Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

5 participants