Skip to content

Commit 5a032d7

Browse files
committed
Help future-proof GCC/Clang unified build options
The YAML uses a single `CXXFLAGS` for GCC and Clang flags, which is mostly okay because they tend to support the same flags... but sometimes GCC and Clang differ So added `-Wno-unknown-warning -Wno-unknown-warning-option`... the first is GCC's spelling and seems to ignore only `-Wno-xxx` negative flags, the second is Clang's spelling and seems to also ignore `-Wxxx` flags, and using them both seems to work to make GCC and Clang ignore also each other's no-warning options (which is either meta or a sweet irony) HT: @DyXel thanks for #916 (comment)
1 parent fc5a354 commit 5a032d7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/build-cppfront.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ jobs:
4242
runs-on: ${{ matrix.runs-on }}
4343
env:
4444
CXX: ${{ matrix.compiler }}
45-
CXXFLAGS: -std=${{ matrix.cxx-std }} -Wall -Wextra -Wold-style-cast -Wunused-parameter -Wpedantic -Werror -pthread
45+
CXXFLAGS: -std=${{ matrix.cxx-std }} -Wall -Wextra -Wold-style-cast -Wunused-parameter -Wpedantic -Werror -pthread -Wno-unknown-warning -Wno-unknown-warning-option
4646
steps:
4747
- uses: actions/checkout@v3
4848
- name: Install compiler

0 commit comments

Comments
 (0)