Skip to content

Commit

Permalink
use macOS 12 for GitHub CI builds
Browse files Browse the repository at this point in the history
  • Loading branch information
reneme committed Dec 29, 2022
1 parent c535266 commit 9e5d476
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ jobs:
# see TODOs here: https://github.com/randombit/botan/pull/3007
- compiler: gcc

runs-on: macos-latest
runs-on: macos-12

steps:
- uses: actions/checkout@v3
Expand Down Expand Up @@ -142,7 +142,7 @@ jobs:
host_os: ubuntu-22.04
- target: emscripten
compiler: emcc
host_os: macos-latest
host_os: macos-12
- target: baremetal
compiler: gcc
host_os: ubuntu-22.04
Expand Down Expand Up @@ -221,7 +221,7 @@ jobs:
host_os: ubuntu-22.04
- target: cross-ios-arm64
compiler: clang
host_os: macos-latest
host_os: macos-12

runs-on: ${{ matrix.host_os }}

Expand Down
6 changes: 6 additions & 0 deletions src/lib/x509/certstor_system_macos/certstor_macos.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -266,10 +266,16 @@ class Certificate_Store_MacOS_Impl
m_system_chain(nullptr),
m_keychains(nullptr)
{
BOTAN_DIAGNOSTIC_PUSH
BOTAN_DIAGNOSTIC_IGNORE_DEPRECATED
// macOS 12.0 deprecates 'Custom keychain management', though the API still works.
// Ideas for a replacement can be found in the discussion of GH #3122:
// https://github.com/randombit/botan/pull/3122
check_success(SecKeychainOpen(system_roots, &m_system_roots.get()),
"open system root certificates");
check_success(SecKeychainOpen(system_keychain, &m_system_chain.get()),
"open system keychain");
BOTAN_DIAGNOSTIC_POP
check_notnull(m_system_roots, "open system root certificate chain");
check_notnull(m_system_chain, "open system certificate chain");

Expand Down

0 comments on commit 9e5d476

Please sign in to comment.