From bf640ea9fb006e5ea7639039dcd5d4447fe94964 Mon Sep 17 00:00:00 2001 From: Jonathan Rainer Date: Fri, 25 Oct 2024 12:44:01 +0100 Subject: [PATCH] Fix macOS CI (#2238) Fixes: #2237 CI has been broken for Mac since Homebrew pulled support for OpenSSL at the specific version we're requiring. However this appears to be a holdover from older versions of MacOS where Rosetta etc. were less well supported, so in moving this forward we _should_ be able to upgrade as well. Green Smoke Test run: https://github.com/apollographql/rover/actions/runs/11515349122/job/32056173251 --- .circleci/config.yml | 8 ++++---- .github/workflows/smoke-test.yml | 4 ++-- .github/workflows/tests-mac-x86.yml | 2 +- xtask/src/target.rs | 6 +++--- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index 53efb8aac..50803b055 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -62,7 +62,7 @@ executors: amd_macos: &amd_macos_executor macos: - xcode: "14.2" + xcode: "15.0.0" resource_class: macos.m1.large.gen1 environment: XTASK_TARGET: "x86_64-apple-darwin" @@ -73,7 +73,7 @@ executors: # until there is official support for building on real ARM Macs arm_macos: &arm_macos_executor macos: - xcode: "14.2" + xcode: "15.0.0" resource_class: macos.m1.large.gen1 environment: XTASK_TARGET: "aarch64-apple-darwin" @@ -530,8 +530,8 @@ commands: - equal: [*arm_macos_executor, << parameters.platform >>] steps: - run: - name: Install OpenSSL@1.1 - command: brew install openssl@1.1 + name: Install OpenSSL + command: brew install openssl@3 - run: name: Install p7zip command: brew install p7zip diff --git a/.github/workflows/smoke-test.yml b/.github/workflows/smoke-test.yml index 6c1e6bd46..18ee22714 100644 --- a/.github/workflows/smoke-test.yml +++ b/.github/workflows/smoke-test.yml @@ -19,9 +19,9 @@ jobs: matrix: compile_target: - target: x86_64-apple-darwin - compiles_on: macos-14 + compiles_on: macos-13-xlarge - target: aarch64-apple-darwin - compiles_on: macos-14 + compiles_on: macos-13-xlarge - target: x86_64-pc-windows-msvc compiles_on: windows-2022 - target: x86_64-unknown-linux-gnu diff --git a/.github/workflows/tests-mac-x86.yml b/.github/workflows/tests-mac-x86.yml index b82566013..dffc93abb 100644 --- a/.github/workflows/tests-mac-x86.yml +++ b/.github/workflows/tests-mac-x86.yml @@ -5,7 +5,7 @@ jobs: build-rover: name: Build Rover for macOS x86-64 # Arm64 runner for cross-compilation - runs-on: macos-14 + runs-on: macos-13-xlarge steps: - uses: actions/checkout@v4 name: "Checkout rover repo" diff --git a/xtask/src/target.rs b/xtask/src/target.rs index 2e6403d11..aec6a4a13 100644 --- a/xtask/src/target.rs +++ b/xtask/src/target.rs @@ -72,12 +72,12 @@ impl Target { } else if self.is_macos() { let openssl_path = BREW_OPT .iter() - .map(|x| Utf8Path::new(x).join("openssl@1.1")) + .map(|x| Utf8Path::new(x).join("openssl@3")) .find(|x| x.exists()) .ok_or_else(|| { anyhow!( - "OpenSSL v1.1 is not installed. Please install with `brew install \ - openssl@1.1`" + "OpenSSL v3 is not installed. Please install with `brew install \ + openssl@3`" ) })?;