Skip to content

feat: merge-train/barretenberg#21924

Merged
AztecBot merged 2 commits into
nextfrom
merge-train/barretenberg
Mar 24, 2026
Merged

feat: merge-train/barretenberg#21924
AztecBot merged 2 commits into
nextfrom
merge-train/barretenberg

Conversation

@AztecBot

@AztecBot AztecBot commented Mar 24, 2026

Copy link
Copy Markdown
Collaborator

BEGIN_COMMIT_OVERRIDE
fix!: dsl inf (#21865)
fix: use llvm-objdump-20 for Mach-O re-signing in version injection (#21953)
END_COMMIT_OVERRIDE

Explicitly connect the infinity flags for grumpkin points generated in
acir to their counterparts in cycle group in the DSL module. Credit:
@suyash67

@ludamad ludamad left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🤖 Auto-approved

@AztecBot AztecBot enabled auto-merge March 24, 2026 08:13
@AztecBot

Copy link
Copy Markdown
Collaborator Author

🤖 Auto-merge enabled after 4 hours of inactivity. This PR will be merged automatically once all checks pass.

…21953)

## Summary

- macOS release binaries were broken (SIGKILL/exit 137) because
`inject_version` in `bootstrap.sh` used `llvm-objdump` (unversioned) to
detect Mach-O binaries for re-signing, but the CI environment only has
`llvm-objdump-20`
- The detection silently failed (`&>/dev/null`), skipping `ldid -S`
re-signing after version injection, leaving binaries with an invalidated
linker-signed code signature
- Fix: `llvm-objdump` → `llvm-objdump-20`
@AztecBot AztecBot added this pull request to the merge queue Mar 24, 2026
Merged via the queue into next with commit 232f82d Mar 24, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants