Skip to content

ci: abort if build requirements are missing - #26368

Merged
ggerganov merged 1 commit into
ggml-org:masterfrom
nikwen:git-lfs-error-message
Aug 7, 2026
Merged

ci: abort if build requirements are missing#26368
ggerganov merged 1 commit into
ggml-org:masterfrom
nikwen:git-lfs-error-message

Conversation

@nikwen

@nikwen nikwen commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Overview

Running CI locally was failing for me and it wasn't obvious from the error message why.

This PR makes it easier for contributors to figure out what's going on:

  1. Abort CI if build requirements are missing.
  2. Check that Git LFS has been configured.
  3. Fix formatting by adding trailing newlines to log messages. (Otherwise, the output is harder to read.)

Additional information

Error message that I got

16/44 Test #16: test-tokenizers-ggml-vocabs .......***Failed    0.66 sec
Already up to date.
/Users/nikwen/Projects/ML/llama.cpp/tests/test-tokenizers-repo.sh: line 35: shopt: globstar: invalid shell option name
main : reading vocab from: '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/HybridDNA/ggml-vocab-carbon.gguf'
register_backend: registered backend CPU (1 devices)
register_device: registered device CPU (Apple M4 Pro)
gguf_init_from_reader: invalid magic characters: 'vers', expected 'GGUF'
llama_model_load: error loading model: llama_model_loader: failed to load model from /Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/HybridDNA/ggml-vocab-carbon.gguf
llama_model_load_from_file_impl: failed to load model
main: error: failed to load vocab '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/HybridDNA/ggml-vocab-carbon.gguf'
main : reading vocab from: '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/PLaMo2/ggml-vocab-plamo2.gguf'
register_backend: registered backend CPU (1 devices)
register_device: registered device CPU (Apple M4 Pro)
gguf_init_from_reader: invalid magic characters: 'vers', expected 'GGUF'
llama_model_load: error loading model: llama_model_loader: failed to load model from /Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/PLaMo2/ggml-vocab-plamo2.gguf
llama_model_load_from_file_impl: failed to load model
main: error: failed to load vocab '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/PLaMo2/ggml-vocab-plamo2.gguf'
main : reading vocab from: '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/RWKV/ggml-vocab-rwkv-7-world.gguf'
register_backend: registered backend CPU (1 devices)
register_device: registered device CPU (Apple M4 Pro)
gguf_init_from_reader: invalid magic characters: 'vers', expected 'GGUF'
llama_model_load: error loading model: llama_model_loader: failed to load model from /Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/RWKV/ggml-vocab-rwkv-7-world.gguf
llama_model_load_from_file_impl: failed to load model
main: error: failed to load vocab '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/RWKV/ggml-vocab-rwkv-7-world.gguf'
main : reading vocab from: '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/SPM/ggml-vocab-gemma-3.gguf'
register_backend: registered backend CPU (1 devices)
register_device: registered device CPU (Apple M4 Pro)
gguf_init_from_reader: invalid magic characters: 'vers', expected 'GGUF'
llama_model_load: error loading model: llama_model_loader: failed to load model from /Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/SPM/ggml-vocab-gemma-3.gguf
llama_model_load_from_file_impl: failed to load model
main: error: failed to load vocab '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/SPM/ggml-vocab-gemma-3.gguf'
main : reading vocab from: '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/UGM/ggml-vocab-nomic-bert-moe.gguf'
register_backend: registered backend CPU (1 devices)
register_device: registered device CPU (Apple M4 Pro)
gguf_init_from_reader: invalid magic characters: 'vers', expected 'GGUF'
llama_model_load: error loading model: llama_model_loader: failed to load model from /Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/UGM/ggml-vocab-nomic-bert-moe.gguf
llama_model_load_from_file_impl: failed to load model
main: error: failed to load vocab '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/UGM/ggml-vocab-nomic-bert-moe.gguf'
main : reading vocab from: '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/WPM/ggml-vocab-jina-v2-en.gguf'
register_backend: registered backend CPU (1 devices)
register_device: registered device CPU (Apple M4 Pro)
gguf_init_from_reader: invalid magic characters: 'vers', expected 'GGUF'
llama_model_load: error loading model: llama_model_loader: failed to load model from /Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/WPM/ggml-vocab-jina-v2-en.gguf
llama_model_load_from_file_impl: failed to load model
main: error: failed to load vocab '/Users/nikwen/Projects/ML/llama.cpp/models/ggml-vocabs/WPM/ggml-vocab-jina-v2-en.gguf'

Turns out you need to have Git LFS installed and configured, which I didn't have.

The existing check for this didn't abort the run. Thus, I missed its short message in a huge wall of text.

Requirements

  • I have read and agree with the contributing guidelines
  • AI usage disclosure: YES. Changes made manually. Opus 5 did a review pass.

@nikwen
nikwen requested a review from ggerganov as a code owner July 31, 2026 12:01
@github-actions github-actions Bot added the devops improvements to build systems and github actions label Jul 31, 2026
1. Abort CI if build requirements are missing.
2. Add check to make sure Git LFS has been configured.
3. Add trailing newlines to log messages.
@nikwen
nikwen force-pushed the git-lfs-error-message branch from c493dcf to 5615eec Compare August 6, 2026 09:48
@nikwen

nikwen commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator Author

@taronaeo Would you have time to take a look at another CI PR? Thank you so much in advance! :)

Comment thread ci/run.sh
Comment on lines +655 to 658
if ! git config --get filter.lfs.clean &> /dev/null; then
gg_printf 'git-lfs not initialized, please run `git lfs install`\n'
exit 1
fi

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! This is easily missed when setting up CI :)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I missed it locally. Thanks for your review! :)

@taronaeo taronaeo added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label Aug 6, 2026
@ggerganov
ggerganov merged commit d8d9887 into ggml-org:master Aug 7, 2026
3 checks passed
miltos22 pushed a commit to miltos22/llama.cpp-wackMall-merge-request that referenced this pull request Aug 10, 2026
1. Abort CI if build requirements are missing.
2. Add check to make sure Git LFS has been configured.
3. Add trailing newlines to log messages.
satindergrewal pushed a commit to satindergrewal/llama.cpp that referenced this pull request Aug 11, 2026
1. Abort CI if build requirements are missing.
2. Add check to make sure Git LFS has been configured.
3. Add trailing newlines to log messages.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants