]> git.ipfire.org Git - thirdparty/squid.git/commitdiff
CI: Use packages from GitHub Actions MacOS runner base-image (#1850)
authorFrancesco Chemolli <5175948+kinkie@users.noreply.github.com>
Tue, 25 Jun 2024 23:03:16 +0000 (23:03 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Wed, 26 Jun 2024 13:35:43 +0000 (13:35 +0000)
    openldap 2.6.8 is already installed and up-to-date.
    openssl@3 3.3.1 is already installed and up-to-date.

Do not "brew install" openldap and openssl packages to avoid GitHub
Actions warnings. If GitHub stops pre-installing these packages, then
MacOS tests will fail because Squid is not compatible with MacOS's
native frameworks for LDAP and OpenSSL.

Also fix an indentation error.

.github/workflows/default.yaml

index f038c03375943cfca605e5ff3a3ffe2b9dbd36dd..d9db83abb29d77f69f336971812e5c851fdf3386 100644 (file)
@@ -105,8 +105,8 @@ jobs:
           - { name: layer-01-minimal, nick: minimal }
           - { name: layer-02-maximus, nick: maximus }
         exclude:
-            # Non-clang testing on MacOS is too much work for very little gain
-            - { os: macos-14, compiler: { CC: gcc, CXX: g++ } }
+          # Non-clang testing on MacOS is too much work for very little gain
+          - { os: macos-14, compiler: { CC: gcc, CXX: g++ } }
 
     runs-on: ${{ matrix.os }}
 
@@ -133,7 +133,8 @@ jobs:
           brew install \
             automake coreutils cppunit gawk \
             gnu-getopt gnu-sed grep libtool \
-            make openldap openssl cyrus-sasl
+            make cyrus-sasl
+            # openldap openssl # already provided by github workers base-image
 
       - name: Checkout sources
         uses: actions/checkout@v4