Skip to content

add sanitycheck_dependencies easyconfig parameter #4916

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

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Crivella
Copy link
Contributor

Allow having a separate set of dependencies loaded only at sanity check.

NOTE:

Thinking about this more, this might not be as straight-forward, as I think the idea of sanity-checking is to test whether all commands work properly with only the build module loaded.

Introducing extra modules could make a check pass that supposedly should not...

@boegel boegel added this to the 5.x milestone Jun 18, 2025
@boegel
Copy link
Member

boegel commented Jun 18, 2025

Using this would indeed sort of defeat the purpose of the sanity check, but I can imagine there are cases where it would make sense, for example if you want to use an extra command in a sanity check command that is not available by default, but which is not required to use the software in the normal way.

@Crivella What use case did you have for this exactly?

@boegel boegel changed the title Added sanitycheck_dependencies Easyconfig parameter add sanitycheck_dependencies easyconfig parameter Jun 18, 2025
@Crivella
Copy link
Contributor Author

This starting idea for this was easybuilders/easybuild-easyconfigs#22930 where I wanted to use pkg-config to check that the patch we apply to the .pc files to add libtiff as a required dependency actually does not break the command.

Still i think for this to be robust we would need not only sanity_check_deps but also a separate sets of commands that are supposed to be ran with these extra dependencies loaded, so that all the other sanity checks can be still performed in the normal way

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