-
Notifications
You must be signed in to change notification settings - Fork 683
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
CFITSIO sanity check not --module-only compatible #19970
Comments
As discussed, |
We can also run To answer's @akesandgren's question in Slack: EasyBuild sets |
IMO a sanity check should be a single simple command. Copying files or changing dirs should not be done in it. |
What about software where e.g. additional compilation is done in the install step? Introduce a new |
EB already has support for an optional |
I respectfully disagree. There's a lot of value in |
The sanity_check_command for CFITSIO is not --module-only compatible.
The above requires write access to %(installdir)s/share, which you may not have when running --module-only
Something like this will work better
The text was updated successfully, but these errors were encountered: