Skip to content

Bump package versions to latest upstream releases - #49

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
bump-versions-20260905
Open

github-actions[bot] wants to merge 1 commit into
mainfrom
bump-versions-20260905

Conversation

@github-actions

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

Copy link
Copy Markdown
Contributor

Automated bump from check-releases.yml. Merging rebuilds and republishes the apt repository.

@gafferongames

Copy link
Copy Markdown
Contributor

Review of PR #49: Bump package versions to latest upstream releases

  • Head SHA: 2a566a83952b0c8801a583e6e37979f00be4a180
  • Target / Base: origin/main (8ba166a51f3a0d7761d73f3e64d285f3c8ab256d)
  • Score: 2/10
  • Verdict: CHANGES REQUESTED (HOLD / Broken CI)

Verification Summary

  1. Git 3-Way Merge Simulation:

    • Ran git merge-tree --write-tree origin/main 2a566a83952b0c8801a583e6e37979f00be4a180.
    • Result: Clean merge, exit code 0, write-tree 5a5522236b723aa2cc27c8b11cdc2234271e9ff7.
  2. Diff Inspection (versions.env):

    • SERIALIZE_VERSION: 1.15.0 -> 1.16.0
    • RELIABLE_VERSION: 1.4.0 -> 1.4.2
    • NETCODE_VERSION: 1.4.3 -> 1.4.5
    • YOJIMBO_VERSION: 1.10.1 -> 1.12.1
    • SCHEMA_VERSION: 2.0.0 -> 2.4.0
  3. Release Check (./scripts/check-releases.sh):

    • Exited with code 1 (stale).
    • Upstream releases have advanced since this automated bump was opened on 2026-09-05:
      • serialize: pinned 1.16.0, latest 1.17.0
      • reliable: pinned 1.4.2, latest 1.4.5
      • netcode: pinned 1.4.5, latest 1.4.8
      • yojimbo: pinned 1.12.1, latest 1.13.5
      • schema: pinned 2.4.0, latest 2.5.0
  4. CI Build Implications & Remote CI Evidence:

    • Remote workflow run ID 33957900112 (build.yml, event workflow_dispatch on bump-versions-20260905) failed across all 10 matrix build targets (Debian 12/13, Ubuntu 22.04/24.04/26.04 on amd64 & arm64).
    • Root cause in packages/yojimbo:
      packages/yojimbo/debian/rules contains:
      override_dh_auto_test:
      ifeq (,$(filter nocheck,$(DEB_BUILD_OPTIONS)))
      	$(CURDIR)/bin/test
      endif
      With yojimbo 1.12.1, cmake builds test binaries in the build directory (obj-$(DEB_HOST_GNU_TYPE)/bin/test), so $(CURDIR)/bin/test does not exist:
      make[1]: /__w/apt/apt/build/yojimbo/yojimbo-1.12.1/bin/test: No such file or directory
      make[1]: *** [debian/rules:20: override_dh_auto_test] Error 127
      
      This causes dpkg-buildpackage to exit 2 on every architecture.
  5. Controls / Sensitivity Verification:

    • Negative Control: Mutated SCHEMA_VERSION=99.99.99 in versions.env; ./scripts/check-releases.sh exited 1 and reported schema: pinned 99.99.99, latest 2.5.0.
    • Positive Control: Updated versions.env to current upstream releases with ./scripts/check-releases.sh --update; subsequent check exited 0 (all pinned versions are current).
    • Negative Control 2: Mutated single package SERIALIZE_VERSION=1.16.0; ./scripts/check-releases.sh cleanly failed on only that package.

Conclusion & Recommendation

Merging this PR will break the repository packaging build. The packaging rules in packages/yojimbo/debian/rules must be updated to locate bin/test in the build directory, and the version pins should be refreshed to the latest upstream releases.

READ-COST: reader=Emma model=gemini-2.5-pro seconds=192 tokens=unknown plan=max

This branch has not been deployed

No deployments
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