Synchronize RADIUSS packages with all local additions #2856
Conversation
spackbot-triage
bot
added
new-version
dependencies
patch
update-package
conflicts
new-variant
labels
| cuda_flags.append("-Xcompiler {}".format(using_toolchain[0])) | ||
|
|
||
| # ppc64le workaround | ||
| if spec.satisfies("target=ppc64le %gcc@8.1:"): |
There was a problem hiding this comment.
With Sierra being retired, do we need this anymore?
|
|
||
| if spec.satisfies("+mpi"): | ||
| # MPI configuration from mpi_for_radiuss_projects | ||
| if spec["mpi"].name == "spectrum-mpi" and spec.satisfies("^blt"): |
There was a problem hiding this comment.
I also don't think any spectrum-mpi sections are needed any longer.
| sys_type = env["SYS_TYPE"] | ||
|
|
||
| # Replace /usr/bin/srun path with srun flux wrapper path on TOSS 4 | ||
| if "toss_4" in sys_type: |
There was a problem hiding this comment.
Is this still needed after the update to the CachedCMakePackage that @tdrwenski did?
There was a problem hiding this comment.
I agree, I think this block can be removed here (and for care, chai, raja-perf, umpire), it looks like the workaround we deleted here for axom: spack/spack#49033
There was a problem hiding this comment.
@white238 @tdrwenski thank you for the review !
I am testing your suggested changes in #3067 and in llnl/Caliper#712.
There was a problem hiding this comment.
(because I can't apply them here)
4 participants
No description provided.