Skip to content

chore: bump llama.cpp (https://github.com/ggml-org/llama.cpp/releases/b11403) - #1078

Merged
ericcurtin merged 2 commits into
mainfrom
bump-llamacpp-b11403
Oct 8, 2026
Merged

ericcurtin merged 2 commits into
mainfrom
bump-llamacpp-b11403

Conversation

@github-actions

@github-actions github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Bumps llama.cpp submodule.

@github-actions github-actions Bot added the dependencies Pull requests that update a dependency file label Oct 5, 2026

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@ericcurtin
ericcurtin requested a balanced review from Copilot October 5, 2026 07:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@ericcurtin

Copy link
Copy Markdown
Contributor

model-runner is being deprecated in favour of https://github.com/llmmanorg/llmman, so please open PRs there instead.

The e2e-test (ubuntu-latest) build is red (also on main): .versions pins LLAMA_SERVER_VERSION=b9879 but server-vulkan-b9879 no longer exists upstream. Worth fixing before this can merge.

@ericcurtin
ericcurtin requested a balanced review from Copilot October 5, 2026 19:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@ericcurtin
ericcurtin requested a balanced review from Copilot October 7, 2026 14:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@ericcurtin
ericcurtin requested a balanced review from Copilot October 7, 2026 19:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@ericcurtin

Copy link
Copy Markdown
Contributor

Correction: the red e2e was a SIGPIPE bug in resolve-llama-upstream-image.sh (fixed in #1080), not a missing tag. Branch updated, all checks green, only the required approval is missing. Follow-on: move this bump workflow to https://github.com/llmmanorg/llmman.

@ericcurtin
ericcurtin requested a balanced review from Copilot October 8, 2026 07:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@ericcurtin ericcurtin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Routine submodule bump, checks green. LGTM.

@ericcurtin
ericcurtin merged commit 359f6dd into main Oct 8, 2026
14 checks passed
@ericcurtin
ericcurtin deleted the bump-llamacpp-b11403 branch October 8, 2026 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants