brewsci-mumps: fix macOS build on Xcode 16+ and update dead source URL#88
Open
kdunn926 wants to merge 1 commit into
Open
brewsci-mumps: fix macOS build on Xcode 16+ and update dead source URL#88kdunn926 wants to merge 1 commit into
kdunn926 wants to merge 1 commit into
Conversation
Two independent fixes so the formula builds on current macOS: * Source URL: http://mumps.enseeiht.fr no longer responds. Point url at the official https://mumps-solver.org host, which serves the identical MUMPS_5.3.5 tarball (unchanged sha256). * Shared-lib link: the macOS branch wrapped each static archive in "-Wl,-all_load <archive> ... -Wl,-noall_load", but the linker shipped with Xcode 16 removed -noall_load, failing with "ld: unknown options: -noall_load". Use -Wl,-force_load,<archive> instead, which force-loads a single archive and needs no undo flag. Linux keeps the --whole-archive / --no-whole-archive wrap unchanged. Verified: brew install brewsci-mumps builds from source on macOS 15 (arm64, Xcode CLT 16.4); the resulting libraries link and pass a full Elmer FEM MUMPS test suite (435/435).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two independent fixes so
brewsci-mumpsbuilds from source on current macOS.1. Dead source URL
http://mumps.enseeiht.fr/MUMPS_5.3.5.tar.gzno longer responds (connection times out), sobrew install brewsci-mumpsfails at the download step. The officialhttps://mumps-solver.orghost serves the identical tarball, so only theurlchanges — thesha256is unchanged (verified:e5d665fdb7043043f0799ae3dbe3b37e5b200d1ab7a6f7b2a4e463fd89507fa4).2.
-noall_loadrejected by the Xcode 16+ linkerThe macOS shared-library link wrapped each static archive in
-Wl,-all_load <archive> ... -Wl,-noall_load. The linker shipped with Xcode 16 removed-noall_load:Switched to
-Wl,-force_load,<archive>, which force-loads a single archive and needs no matching "undo" flag. This is safe because-all_load/-force_loadonly affect static.ainputs and every other input on those link lines is a dynamic-l. Linux behavior is unchanged (still--whole-archive/--no-whole-archive).Test plan
brew install --build-from-source brewsci-mumpson macOS 15.3 (arm64), Xcode CLT 16.4 — builds cleanly (both fixes required).libdmumps/libzmumps/… link and run correctly: a full Elmer FEM build with--with-mumpspasses its MUMPS quick-test suite 435/435 (parallel MUMPS via ScaLAPACK + ParMetis).Notes
revisionbump to trigger new bottles.