Skip to content

Conversation

@amontoison
Copy link
Contributor

Some packages like PETSc can be compiled with the 64-bit integer version.
I propose to add it in MUMPS_seq_jll.jl.

name = "MUMPS_seq"
upstream_version = v"5.6.2"
version_offset = v"0.0.1" # reset to 0.0.0 once the upstream version changes
version_offset = v"0.0.2" # reset to 0.0.0 once the upstream version changes
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Won't Yggdrasil just publish new versions as +1, +2, etc? Unless we are patching the package, I thought we wouldn't need to change the version_offset.

Copy link
Contributor Author

@amontoison amontoison Feb 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but for Coin packages and its dependencies, we multiply all version numbers by 100:
https://github.com/JuliaPackaging/Yggdrasil/blob/master/M/MUMPS/MUMPS_seq%405/build_tarballs.jl#L19-L21
We often recompile the same version of a package (like Ipopt) with updated dependencies (MUMPS, SPRAL, ...).
Without this trick, if the new version of the dependency breaks the package, all the Coin ecosystem is broken... (cc @odow 😛).

@amontoison
Copy link
Contributor Author

@ViralBShah The issue is with GCC 7, you can't use another compiler because it's the only one with libgfortran4.
It's a compiler bug with the flag -ffixed-line-length-none...

@amontoison
Copy link
Contributor Author

Superseded by #8548

@amontoison amontoison closed this May 5, 2024
@amontoison amontoison deleted the mumps64 branch May 5, 2024 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants