Skip to content

Conversation

@josh-tracey
Copy link
Contributor

This commit provides additional refinements to the OpenSSL dependency handling for macOS builds in the GitHub Actions release workflow. The changes focus on ensuring the correct Homebrew instance is invoked, especially when cross-compiling or running on a different architecture than the target (e.g., x86_64 target on an ARM64 runner).

Key improvements:

  • Corrected the use of arch -x86_64 to prevent duplicate invocations.
  • Implemented more robust detection of the Homebrew executable by checking multiple standard paths for both x86_64 (under Rosetta) and native ARM64 Homebrew installations.
  • Added error handling if a suitable Homebrew installation cannot be found.
  • Ensured that brew update is attempted but failures are handled gracefully before proceeding with OpenSSL installation.

These adjustments aim to fix errors like "arch: Can't find any plists for arch" and improve the overall reliability of OpenSSL-dependent builds on macOS.

This commit provides additional refinements to the OpenSSL dependency
handling for macOS builds in the GitHub Actions release workflow.
The changes focus on ensuring the correct Homebrew instance is invoked,
especially when cross-compiling or running on a different architecture
than the target (e.g., x86_64 target on an ARM64 runner).

Key improvements:
- Corrected the use of `arch -x86_64` to prevent duplicate invocations.
- Implemented more robust detection of the Homebrew executable by checking
  multiple standard paths for both x86_64 (under Rosetta) and native
  ARM64 Homebrew installations.
- Added error handling if a suitable Homebrew installation cannot be found.
- Ensured that `brew update` is attempted but failures are handled gracefully
  before proceeding with OpenSSL installation.

These adjustments aim to fix errors like "arch: Can't find any plists for arch"
and improve the overall reliability of OpenSSL-dependent builds on macOS.
@vercel
Copy link

vercel bot commented May 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sailr ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 25, 2025 11:35am

@josh-tracey josh-tracey merged commit 00139a2 into main May 25, 2025
2 of 5 checks passed
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.

2 participants