-
Notifications
You must be signed in to change notification settings - Fork 706
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
fix configure step for ELSI 2.5.0+ (and add missing foss versions) #17288
fix configure step for ELSI 2.5.0+ (and add missing foss versions) #17288
Conversation
Test report by @Flamefire |
@boegelbot please test @ generoso |
@boegel: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1424876569 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegel I see the same (or similar, not sure) errors during the tests so I disabled testing for my reports as the build is what was broken and manual investigation shows the correct compilers were used as far as I could tell. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Going through everything marked for 4.7.2
Two of the 2019b failures comes from missing MUMPS module, which, i'm really not sure what happened. Something must be broken in the module tree on generoso, or someone was rebuilding those ancient MUMPS packages at the same time, which seems unlikely. The last failure is the PEXSI variant which had a lot of segfaults.
2019b is to old for my own test tree, and I really don't think we want to hold back on the fixes for the more recent easyconfigs as a result of those old broken ones, which almost certainly were broken before this PR as far as i can tell.
I'm willing to accept this PR as is.
Test report by @Micket |
Going in, thanks @Flamefire! |
(created using
eb --new-pr
)fixes #17259