Skip to content

chore: upgrade llama.cpp from b11236 to b11237 - #465

Merged
bernardladenthin merged 1 commit into
mainfrom
claude/busy-archimedes-lo9uyy
Sep 28, 2026
Merged

bernardladenthin merged 1 commit into
mainfrom
claude/busy-archimedes-lo9uyy

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • Bumps the pinned llama.cpp from b11236 to b11237 (llama/CMakeLists.txt, README.md, CLAUDE.md, LlamaCppVersion.java).
  • The range contains one upstream commit, #29603 (ggml/src/ggml-openvino/ggml-openvino.cpp, +22 lines). The OpenVINO backend now reports views with an unaligned batch stride as unsupported, so the scheduler falls back instead of computing them wrong.
  • This project's side needs only the version change: no API change, no patch-target file in the range, and no upstream workflow change.

Test plan

  • No patch-target file changes in b11236...b11237, so all nine patches apply unchanged
  • CI is green on this branch
  • Docs updated (docs/history/llama-cpp-breaking-changes.md)

Related issues / PRs

Follows #464 (b11222 → b11236).

Checklist

  • My commits follow Conventional Commits
  • No security-sensitive changes

🤖 Generated with Claude Code

https://claude.ai/code/session_01QSXAwbXqx9u5xsMArMvgT8


Generated by Claude Code

One upstream commit (#29603): the OpenVINO backend marks unaligned
batch-stride views unsupported. Backend-internal; no API change, no patch
touches the file, no workflow change upstream.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QSXAwbXqx9u5xsMArMvgT8

Copy link
Copy Markdown
Owner Author

Heads-up: the two Verify GPG signing key checks ("… (no secrets printed)" and "… — Gradle/BouncyCastle path …") will fail on this PR. They read the signing key from the maven-central environment, and those secrets are not available on PR branches. They fail the same way on every PR (see #464, #461). This PR changes only the llama.cpp pin and docs, and touches no signing or workflow code. There is no fix to port; they pass on main / release dispatches.


Generated by Claude Code

Copy link
Copy Markdown
Owner Author

claude-review failed here, but not because of this PR. The action's Claude call returned is_error: true after 286 ms with total_cost_usd: 0 and empty modelUsage, so no model turn ran and nothing was reviewed. It is the same action/API-side failure seen on #461, and it is independent of this diff, which changes only the llama.cpp pin and docs. There is no fix to port from here. I don't have the means to re-run the job; a manual re-run of claude-review should pass once the action's API call succeeds.


Generated by Claude Code

@sonarqubecloud

Copy link
Copy Markdown

@bernardladenthin
bernardladenthin merged commit 9e4b77c into main Sep 28, 2026
12 of 17 checks passed
@bernardladenthin
bernardladenthin deleted the claude/busy-archimedes-lo9uyy branch September 28, 2026 21:17

This branch had an error being deployed

1 failed deployment
startgate — b8a1d36e Deployed Sep 28, 2026 by bernardladenthin via Start gate (abort window) #1062
maven-central — b8a1d36e Deployed Sep 28, 2026 by bernardladenthin via Verify GPG signing key (no secrets printed) #1062
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