-
Notifications
You must be signed in to change notification settings - Fork 20
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 JasPer for Icontools #1041
base: main
Are you sure you want to change the base?
Fix JasPer for Icontools #1041
Conversation
launch jenkins |
thanks for your contribution, I'll have a look as soon as we are done with the major upgrade to spack v0.22. |
@efmkoene The uenv currently available on Santis provides jasper, and it only uses at max spack v0.22. |
OK, that's very confusing. My user environment has JasPer
Is that not the case for you? |
When installing with the |
Hm that should not be, I'll have a look tomorrow |
On the plus side, after more than an hour, it did compile everything correctly (without needing to do anything twice, or fix the JasPer compilation)! |
So finally I have something for you:
All with branch |
Some context for you: This is the new uenv based on spackl v0.22 that we will soon use for ICON on Santis. Please let me know if this reduces builds times on your side AND if icontools runs for your usecase. |
I did
It compiles quickly (only builds netcdf-fortran and icontools, the rest is pulled from the However, unfortunately it returns a segmentation fault on this simple input file,
which is run using the command The icontools that worked (i.e., this pull request branch) returns
Whereas the icontools/main branch returns
Any suggestion? Thanks for going through this with me... |
From the log I see that you dont specify the version in the spec. |
You can run spack info icontools, this shows all available versions of the package |
Unfortunately, it performs no better. Can you confirm that on your end, too?
|
I am off today, will be back on. Tuesday. This might be an issue for Ben to fix |
This change adds the JasPer library into the
spack-c2sm
project, merely to add a variant for building onaarch64
which, at least for me, didn't work out of the box.As you can see below, for reasons unclear to me, we need a double pass of
spack install icontools
because otherwiseopenmpi
fails to build. But that's not the topic of this PR.I've made the
aarch64
variant the default version, which should suffice on Alps for now.Please verify that you can reproduce the issue that this PR solves, too.