Skip to content

fix(aztec): use perl -i for portable in-place edit in add_crate.sh#23335

Merged
nchamo merged 1 commit into
merge-train/fairiesfrom
nc/fix-add-crate-bsd-sed
May 18, 2026
Merged

fix(aztec): use perl -i for portable in-place edit in add_crate.sh#23335
nchamo merged 1 commit into
merge-train/fairiesfrom
nc/fix-add-crate-bsd-sed

Conversation

@nchamo

@nchamo nchamo commented May 16, 2026

Copy link
Copy Markdown
Contributor

Problem

aztec init fails on macOS with sed: -e: No such file or directory. The sed -i invocations in add_crate.sh rely on GNU sed's optional argument to -i; BSD sed (default on macOS) requires an explicit extension argument and otherwise treats the next token as the backup extension.

Surfaced by the new macOS leg of the CLI acceptance test matrix being explored in branch nchamo/f-566-explore-macos-ci-coverage-for-the-full-dev-onboarding-path.

Fix

Replace the three sed -i calls with perl -i -pe, which behaves identically across implementations. Regex syntax is unchanged except for the capture group switching from \(...\) to (...).

@nchamo nchamo self-assigned this May 16, 2026
@nchamo nchamo requested a review from mverzilli May 18, 2026 11:24
@nchamo nchamo merged commit 236bcbb into merge-train/fairies May 18, 2026
14 checks passed
@nchamo nchamo deleted the nc/fix-add-crate-bsd-sed branch May 18, 2026 11:24
@AztecBot

Copy link
Copy Markdown
Collaborator

✅ Successfully backported to backport-to-v4-next-staging #23386.

nchamo added a commit that referenced this pull request May 19, 2026
BEGIN_COMMIT_OVERRIDE
fix(aztec-up): fall back to no timeout when /usr/bin/timeout absent
(macOS) (#23310)
fix(aztec): use perl -i for portable in-place edit in add_crate.sh
(#23335)
END_COMMIT_OVERRIDE
alexghr added a commit that referenced this pull request May 19, 2026
## Summary

Brings the accumulated v4-next fixes onto v4 ahead of the next 4.3.0
stable release. Sibling pattern to #23306 (which merged v4-next into v4
for 4.3.0-rc.1).

### Commits coming in from v4-next

- `5b61b2d5d3` chore: Accumulated backports to v4-next (#23386)
- `38ae87a965` fix(aztec): use perl -i for portable in-place edit in
add_crate.sh (#23335)
- `f463d6a3cb` fix(aztec-up): fall back to no timeout when
/usr/bin/timeout absent (macOS) (#23310)
- `1a40b92d98` chore: Accumulated backports to v4-next (#23321)
- `84e5bce90c` chore: reduce compat e2e timeout (#23318)
@aminsammara aminsammara mentioned this pull request May 19, 2026
2 tasks
aminsammara added a commit that referenced this pull request May 20, 2026
Promote v4.3.0-rc.1 to stable. Single commit prepending a `## [4.3.0]`
entry to `CHANGELOG.md`; no other files touched.

## What's in this PR

The 4.3.0 changelog entry is a superset of the existing `4.3.0-rc.1`
entry plus four lines for the commits added between rc.1 and now:

- **Bug Fixes**
- `aztec:` use perl -i for portable in-place edit in add_crate.sh
(#23335)
- `aztec-up:` fall back to no timeout when /usr/bin/timeout absent on
macOS (#23310)
- `release-image:` stamp stdlib/package.json with release version
(#23393)
- **Miscellaneous**
  - reduce compat e2e timeout (#23318)

The `## [4.3.0-rc.1]` section is preserved below the new entry for
historical continuity.

## Filtered from the changelog

Eight commits in `v4.3.0-rc.1..HEAD` are dropped as noise: the two
`chore: merge v4-next into v4` merge commits, the pre-PR dupe
`75fd494a43` of #23393, the two umbrella `chore: Accumulated backports`
PRs (#23386, #23321 — their contents are #23310/#23335/#23318 listed
above), the internal `Merge branch 'v4-next' into
backport-to-v4-next-staging` commit, and the v4-next manifest bump to
4.4.0 (release machinery, doesn't apply to v4).

## After merging

1. Tag `v4.3.0` on the merge commit and push the tag — that kicks off
the release pipeline.
2. The manifest stays at `4.3.0` for the 4.3.X patch cycle; bump only if
you're cutting 4.4.0 from this branch.

## Test plan

- [ ] CI green
- [ ] Visual check that the 4.3.0 section reads correctly in
`CHANGELOG.md`
critesjosh added a commit that referenced this pull request May 20, 2026
…or items

Backfills v4.3.0 docs with items from the official release notes that
weren't carried into the migration notes, framework docs, tutorials, or
operator changelog:

- migration_notes: add '[CLI] aztec init now scaffolds a Counter example
  template' under v4.3.0 (#22751). aztec new continues to scaffold blank.
- contract_structure: rewrite Directory structure for the 2-crate
  workspace; drop the stale --name reference removed in v4.3.0.
- token_contract tutorial: fix 'aztec init created...' wording to
  'aztec new' (the command shown is aztec new bob_token); update live
  copy from single-crate to 2-crate layout.
- recursive_verification tutorial: replace removed 'aztec new --name X Y'
  with 'aztec new ValueNotEqual'; fix the workspace layout, package name
  (ValueNotEqual_contract), ccc script, and artifact paths.
- v4.3 operator changelog: add 'Other notable changes' for the three
  non-breaking release-notes items (sequencer last-block signing #22117,
  stdlib package.json stamping #23393, macOS aztec-up install fixes
  #23310 / #23335).
danielntmd pushed a commit to danielntmd/aztec-packages that referenced this pull request Jun 4, 2026
BEGIN_COMMIT_OVERRIDE
fix(aztec): use perl -i for portable in-place edit in add_crate.sh
(AztecProtocol#23335)
END_COMMIT_OVERRIDE
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