]> git.ipfire.org Git - thirdparty/openssl.git/commitdiff
Disable default case checks on clang 18
authorNeil Horman <nhorman@openssl.org>
Wed, 26 Jun 2024 20:23:04 +0000 (16:23 -0400)
committerTomas Mraz <tomas@openssl.org>
Fri, 28 Jun 2024 09:17:59 +0000 (11:17 +0200)
Recent updates in CI have upgraded clang to clang-18, which gripes when
it finds a switch statement without a default case.  We should add those
cases in, but since we have a lot of those, and CI is currently failing,
disable the check until we get them fixed up

Fixes #24739

Reviewed-by: Bernd Edlinger <bernd.edlinger@hotmail.de>
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/24744)

.github/workflows/os-zoo.yml

index b031ce2253a65bc9e2fde1765649d8d018e60761..53e90215d507157b6ef418e7d65cd6af33ae5888 100644 (file)
@@ -27,7 +27,7 @@ jobs:
       image: docker.io/library/alpine:${{ matrix.tag }}
     env:
       # https://www.openwall.com/lists/musl/2022/02/16/14
-      EXTRA_CFLAGS: ${{ matrix.cc == 'clang' && '-Wno-sign-compare' || '' }}
+      EXTRA_CFLAGS: ${{ matrix.cc == 'clang' && '-Wno-sign-compare -Wno-switch-default' || '' }}
       CC: ${{ matrix.cc }}
     steps:
     - name: install packages