Skip to content

The pin moves to the interpreter upstream publishes now - #26

Merged
fabeat merged 1 commit into
mainfrom
fix/interpreter-pins
Oct 30, 2026
Merged

fabeat merged 1 commit into
mainfrom
fix/interpreter-pins

Conversation

@fabeat

@fabeat fabeat commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

Upstream rebuilt php-8.4.27 for all four platforms without changing the version, so the file the name points at holds different bytes than it held when the pin was taken.

The untagged pipeline run found it. Three of the four prebuilt platforms stopped at the pin — macos-arm64, macos-x86_64, linux-x86_64 and linux-aarch64 all failing at Fetch the published interpreter — and the Windows build, which compiles its own interpreter, was unaffected.

The pin is doing its job: the build stops and says which bytes it found and which it expected, rather than absorbing the change. But the release could not have gone out.

The four pins are the digests upstream publishes now, cross-checked against the .sha256 files beside the downloads, and one of them (linux-x86_64) against the download itself.

The comment above the pin is already right

It says the pin is on a build rather than on a version, and that it has to be moved by hand when upstream rebuilds one. That is what happened, three days after the last move (91089e8). Nothing in the comment needed changing.

What this has to land before

The release. The tag waits for this to merge, and then the pipeline runs without a tag once more, because a release waits for every binary.

Upstream rebuilt php-8.4.27 for all four platforms without changing the version, so the file the name points at holds different bytes than it held when the pin was taken. The build stopped at the pin and said which bytes it found and which it expected — the pin doing its job — but the release could not have gone out.

The four pins are the digests upstream publishes now, cross-checked against the .sha256 files beside the downloads, and one of them against the download itself.
@sonarqubecloud

Copy link
Copy Markdown

@fabeat
fabeat merged commit c011867 into main Oct 30, 2026
7 checks passed
@fabeat
fabeat deleted the fix/interpreter-pins branch October 30, 2026 15:53
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.

1 participant