Skip to content

Fix the arm64 CI build failing to link openssl - #764

Merged
hannesa2 merged 1 commit into
masterfrom
fix-ci-openssl-link
Oct 10, 2026
Merged

hannesa2 merged 1 commit into
masterfrom
fix-ci-openssl-link

Conversation

@goneng

@goneng goneng commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator

Keep the arm64 build from failing in "Show openssl version" since
Homebrew's openssl rename (seen on #763 since 2026-10-08).

Homebrew now names openssl@4 as plain openssl and treats openssl@3 as keg-only.
On the arm64 runner, uninstalling openssl@3 leaves
/opt/homebrew/bin/openssl -> /opt/homebrew/opt/openssl@3/bin/openssl behind;
the step then removes opt/openssl@3, and the reinstall stops at "Could not symlink bin/openssl".
The x86_64 image has an older Homebrew and still passes.

  • Run brew cleanup --prune-prefix before reinstalling, which drops the dangling link
  • Link openssl@3 with --force, since keg-only kegs are not linked, so openssl on PATH
    is still openssl@3 for the later steps, including the existing "Unlink openssl 3"

Test plan

Homebrew now names openssl@4 as plain `openssl` and treats openssl@3
as keg-only, so uninstalling openssl@3 on the arm64 runner leaves its
/opt/homebrew/bin/openssl link behind, dangling once the step removes
the opt folder, and the reinstall stops at "Could not symlink
bin/openssl".

- Prune dangling links from the Homebrew prefix before reinstalling
- Link openssl@3 explicitly, since keg-only kegs are not linked, so
  `openssl` on PATH stays the version the later steps expect
@goneng
goneng requested a review from hannesa2 October 10, 2026 01:12
@goneng
goneng marked this pull request as ready for review October 10, 2026 01:23
@hannesa2 hannesa2 added the pipeline CI pipeline label Oct 10, 2026
@hannesa2
hannesa2 merged commit 646aabc into master Oct 10, 2026
3 checks passed
@hannesa2
hannesa2 deleted the fix-ci-openssl-link branch October 10, 2026 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pipeline CI pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants