-
Notifications
You must be signed in to change notification settings - Fork 763
{numlib}[foss/2024a] DOLFINx-0.9.0, FFCx-0.9.0, KaHIP-3.18, spdlog-1.12.0, UFL-2024.2.0, scikit-build-core-0.11.0 #22700
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
base: develop
Are you sure you want to change the base?
Conversation
Updated software
|
| sources = ['v%(version)s.tar.gz'] | ||
| checksums = ['4dccf2d10f410c1e2feaff89966bfc49a1abb29ef6f08246335b110e001e09a9'] | ||
|
|
||
| configopts = '-DCMAKE_POSITION_INDEPENDENT_CODE=ON' |
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.
This is usually done by toolchainopts = { pic:True }
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.
this is fixed in 3efe419.
|
Some suggestions:
better: That way, if that PR is merged, we can tick it and it makes it a bit easier. Just have a look at other PRs and I am sure you can see what I mean. |
|
It looks like |
|
One way to make life easier is to use the EasyBuild built-in GitHub integration like this: which will sort out the title for you. Not sure if you are already doing that though. :-) |
|
Thanks for the tips; I applied both changes. |
Added in f32a1d5. |
Unfortunately, I never got to fully setup my github integration to start new PRs. That's why I do it old-school ;-) |
|
This is blocked by merge conflicts & failing CI, can we revive this so it's eligible for merging? |
I have already been revisiting this PR, but building compatible version of PETSc (i.e. 3.23.5) is giving me some trouble, as reported in the issue #23717 . Once that is fixed, I will update the dependencies of building DOLFINx to be compatible with the versions upstream and retry to build it. |
This PR adds new or updated easyconfigs for building
DOLFINxwithfoss/2024a. I also have to mention that this PR is inspired by PR #22422. Here are the list of included dependencies:DOLFINx-0.9.0.post1-foss-2024a.ebFFCx-0.9.0-gfbf-2024a.ebUFL-2024.2.0-gfbf-2024a.ebRequires: