Skip to content

cmake: fix Mach-O current version number#17877

Merged
ggerganov merged 1 commit intoggml-org:masterfrom
Rhys-T:fix-macho-current-version
Dec 9, 2025
Merged

cmake: fix Mach-O current version number#17877
ggerganov merged 1 commit intoggml-org:masterfrom
Rhys-T:fix-macho-current-version

Conversation

@Rhys-T
Copy link
Copy Markdown
Contributor

@Rhys-T Rhys-T commented Dec 9, 2025

PR #17091 set the VERSION of various libraries to 0.0.abcd, where abcd is the LLAMA_BUILD_NUMBER. That build number is too large to fit in the Mach-O 'current version' field's 'micro' part, which only goes up to 255. This just sets the Mach-O current version to 0 to get it building properly again.

Fixes #17258.

I tried running the CPU-only CI command (bash ./ci/run.sh ./tmp/results ./tmp/mnt). It built successfully, but test 32 of 38 (test-backend-ops) sat there for a long time and eventually timed out. Everything else ran fine. For what it's worth, I've been using a slightly fancier version of this same patch via Nix, and it seems to be working properly.

If it would be better to keep the full build number in there, I can push an alternate patch that sets the Mach-O version to 0.ab.cd instead. Wasn't sure if that would conflict with any other version numbers you were planning though, so for now I went with just setting it to 0 as suggested in #17258 (comment).

PR ggml-org#17091 set the VERSION of various libraries to 0.0.abcd, where abcd
is the LLAMA_BUILD_NUMBER. That build number is too large to fit in the
Mach-O 'current version' field's 'micro' part, which only goes up to
255. This just sets the Mach-O current version to 0 to get it building
properly again.

Fixes ggml-org#17258.
@ggerganov ggerganov merged commit 63908b6 into ggml-org:master Dec 9, 2025
78 checks passed
booxter added a commit to booxter/nixpkgs that referenced this pull request Dec 10, 2025
This version includes a build fix for darwin:
ggml-org/llama.cpp#17877
Ethan-a2 pushed a commit to Ethan-a2/llama.cpp that referenced this pull request Dec 12, 2025
PR ggml-org#17091 set the VERSION of various libraries to 0.0.abcd, where abcd
is the LLAMA_BUILD_NUMBER. That build number is too large to fit in the
Mach-O 'current version' field's 'micro' part, which only goes up to
255. This just sets the Mach-O current version to 0 to get it building
properly again.

Fixes ggml-org#17258.
peterhoeg pushed a commit to peterhoeg/nixpkgs that referenced this pull request Dec 12, 2025
This version includes a build fix for darwin:
ggml-org/llama.cpp#17877
0Marble pushed a commit to 0Marble/llama.cpp that referenced this pull request Dec 18, 2025
PR ggml-org#17091 set the VERSION of various libraries to 0.0.abcd, where abcd
is the LLAMA_BUILD_NUMBER. That build number is too large to fit in the
Mach-O 'current version' field's 'micro' part, which only goes up to
255. This just sets the Mach-O current version to 0 to get it building
properly again.

Fixes ggml-org#17258.
mkg20001 pushed a commit to mkg20001/nixpkgs that referenced this pull request Dec 20, 2025
This version includes a build fix for darwin:
ggml-org/llama.cpp#17877
Anico2 added a commit to Anico2/llama.cpp that referenced this pull request Jan 15, 2026
PR ggml-org#17091 set the VERSION of various libraries to 0.0.abcd, where abcd
is the LLAMA_BUILD_NUMBER. That build number is too large to fit in the
Mach-O 'current version' field's 'micro' part, which only goes up to
255. This just sets the Mach-O current version to 0 to get it building
properly again.

Fixes ggml-org#17258.
blime4 referenced this pull request in blime4/llama.cpp Feb 5, 2026
PR #17091 set the VERSION of various libraries to 0.0.abcd, where abcd
is the LLAMA_BUILD_NUMBER. That build number is too large to fit in the
Mach-O 'current version' field's 'micro' part, which only goes up to
255. This just sets the Mach-O current version to 0 to get it building
properly again.

Fixes #17258.
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
PR ggml-org#17091 set the VERSION of various libraries to 0.0.abcd, where abcd
is the LLAMA_BUILD_NUMBER. That build number is too large to fit in the
Mach-O 'current version' field's 'micro' part, which only goes up to
255. This just sets the Mach-O current version to 0 to get it building
properly again.

Fixes ggml-org#17258.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compile bug: compile fail

2 participants