e4s ci: add umpire +rocm#32504
Conversation
|
Do you know what is going on @davidbeckingsale ? |
|
Yup, the hip_repair_options method in camp ( ) is not written to work with a CachedCMakePackage.CC @trws |
|
Dang, yes, I see that. It needs to add the appropriate set command I suppose, very strange gitlab didn't catch that. It should be as simple as either making a second copy of the function or adding an option and using |
I must admit I don't understand the issue. Is there an example I could reference? We could test the proposed change out in this PR here. |
61efb04 to
cb78268
Compare
|
I've updated the umpire package in this PR (see 1aaf096) and will see how that works out here. Based on my offline trial of this change, this gets us passed the cmake error and onto some bona fide errors? |
c9f18ef to
1aaf096
Compare
1aaf096 to
d7d78a3
Compare
|
@trws Can you advise, did I do this correct? Forgetting whether this is how we would ultimately want to do it, I'm trying to figure out whether the error I get after doing this is because my change is wrong, or whether it is a bona fide error that we need to deal with even if I did this right? Change I made: Resulting error:
|
|
This change is fine, though it would probably be better to have a function for it since it's going to end up getting used in multiple packages. That error is from umpire's cmake mis-detecting the availability of std::filesystem though I think, which should be unrelated to this. @mcfadden8, any idea? |
I agree that the AFAIK, we are not seeing these in the Umpire CI, but that may be due to a missing configuration to be tested. Can someone provide me the spack spec being used so that I may see if I can reproduce this locally? |
The spec is The CI build environment:
Thank you! |
Any luck with this @mcfadden8 ? |
|
Sorry @eugeneswalker, I have not yet had a chance to look. I'll try to take a look at this early next week. |
d7d78a3 to
f168d11
Compare
I'm able to reproduce this with a simpler setup using a gcc11 container. I believe that @davidbeckingsale also reported seeing this a earlier. I'll look at it more closely and talk with David about it. |
|
Update: (@trws, @davidbeckingsale, and @eugeneswalker) The So what's left to figure out is why We have documented this in our internal bug tracking system and have begun working on a fix. This is being worked on here: llnl/blt#599 |
Manually patching in LLNL/blt#599 resolves the See: |
8859d52 to
60284ed
Compare
60284ed to
09bbf2c
Compare
Hoping that the merge of #32469 has fixed the issue building
umpire +rocmFYI @wspear @davidbeckingsale