Skip to content

test: accept matching runtime ABIs in the cross-version matrix - #3470

Merged
paul-hammant merged 1 commit into
mainfrom
fix/abi-matrix-matching-runtime
Oct 29, 2026
Merged

paul-hammant merged 1 commit into
mainfrom
fix/abi-matrix-matching-runtime

Conversation

@paul-hammant

Copy link
Copy Markdown
Collaborator

CI can build the same runtime ABI as the latest published release. The ABI matrix unconditionally expected the shared-runtime library and importer to reject that pairing, failing PR #3468 after 0.860.0 was published.

Read the runtime ABI IDs from the built library catalogs. Matching IDs must pass the host calls and Aether import; differing IDs retain the existing rejection checks. Compare full IDs so ABI revisions and layout flags remain significant. Update the ABI policy and add a changelog fragment.

Validation:

  • Fails on main with “this library on the 0.860.0 shared runtime is not refused” and “this importer took the 0.860.0 shared-runtime library”; passes with the fix.
  • Matrix against published 0.859.0 passes, including rejection of the incompatible shared runtime.
  • Same-ABI 0.860.0 matrix passes using the released SDK with locally built static/shared runtimes. The unmodified Linux release runtime hits SIGILL at incsspq in panic handling on this host, so it could not complete this check here.
  • sh tests/integration/abi_policy/test_abi_policy.sh passes.
  • build/ae fmt std examples tests, shell syntax and git diff --check pass.
  • make check-docs passes outside the sandbox (the sandbox prevented the UDP example from binding).

@paul-hammant
paul-hammant marked this pull request as ready for review October 29, 2026 22:14
@paul-hammant
paul-hammant merged commit 09bc967 into main Oct 29, 2026
44 checks passed
@paul-hammant
paul-hammant deleted the fix/abi-matrix-matching-runtime branch October 29, 2026 22:16
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