Skip to content

Conversation

@TheSecurityDev
Copy link
Contributor

Unlink and relink openssl@3 in mac_setup.sh. Otherwise, it's hit or miss with my PRs whether the macOS build will succeed.

Failure example: https://github.com/commaai/openpilot/actions/runs/18534765840/job/52826803925?pr=36342

Copilot AI review requested due to automatic review settings October 15, 2025 19:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes flaky macOS builds by adding OpenSSL relinking steps to the setup script. The change addresses inconsistent build failures where OpenSSL@3 dependencies weren't properly linked.

  • Adds OpenSSL@3 unlinking and relinking steps to stabilize macOS builds
  • Uses || true to prevent failure if OpenSSL@3 isn't currently linked
  • Forces overwrite during relinking to resolve potential conflicts

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link
Contributor

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

@github-actions github-actions bot added the stale label Oct 25, 2025
@github-actions
Copy link
Contributor

This PR has been automatically closed due to inactivity. Feel free to re-open once activity resumes.

1 similar comment
@github-actions
Copy link
Contributor

This PR has been automatically closed due to inactivity. Feel free to re-open once activity resumes.

@github-actions github-actions bot closed this Oct 27, 2025
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.

1 participant