Skip to content

Commit 948203a

Browse files
committed
feat: add the flag "--fatal-meson-warnings" to every meson setup command
1 parent 9160473 commit 948203a

File tree

10 files changed

+14
-9
lines changed

10 files changed

+14
-9
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,7 @@ jobs:
199199
brew install sdl2 sdl2_ttf sdl2_mixer sdl2_image
200200
201201
- name: Configure
202-
run: meson setup build -Dbuildtype=release -Ddefault_library=${{( matrix.config.os == 'windows' && matrix.config.environment == 'msvc' && matrix.config.static ) && 'static' ||'shared' }} -Dclang_libcpp=${{ ( ( matrix.config.os == 'ubuntu' && matrix.config.use-clang == true && matrix.config.use-clang_stdlib ) || matrix.config.os == 'macos' ) && 'enabled' || 'disabled' }} -Drun_in_ci=true
202+
run: meson setup build -Dbuildtype=release -Ddefault_library=${{( matrix.config.os == 'windows' && matrix.config.environment == 'msvc' && matrix.config.static ) && 'static' ||'shared' }} -Dclang_libcpp=${{ ( ( matrix.config.os == 'ubuntu' && matrix.config.use-clang == true && matrix.config.use-clang_stdlib ) || matrix.config.os == 'macos' ) && 'enabled' || 'disabled' }} -Drun_in_ci=true --fatal-meson-warnings
203203

204204
- name: Build
205205
run: meson compile -C build

.github/workflows/lint.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ jobs:
3434
sudo apt-get update
3535
sudo apt-get install ninja-build libsdl2-2.0-0 libsdl2-dev libsdl2-ttf* libsdl2-mixer* libsdl2-image* desktop-file-utils -y --no-install-recommends
3636
37-
meson setup build -Dbuildtype=release -Dclang_libcpp=disabled -Dtests=true -Drun_in_ci=true
37+
meson setup build -Dbuildtype=release -Dclang_libcpp=disabled -Dtests=true -Drun_in_ci=true --fatal-meson-warnings
3838
meson compile -C build git_version.hpp
3939
4040
- uses: cpp-linter/cpp-linter-action@v2

.github/workflows/musl.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ jobs:
4141
fetch-depth: '0'
4242

4343
- name: Configure
44-
run: meson setup build -Dbuildtype=release -Ddefault_library=shared -Dclang_libcpp=${{ ( matrix.config.use-clang == true && matrix.config.use-clang_stdlib ) && 'enabled' || 'disabled' }} -Drun_in_ci=true
44+
run: meson setup build -Dbuildtype=release -Ddefault_library=shared -Dclang_libcpp=${{ ( matrix.config.use-clang == true && matrix.config.use-clang_stdlib ) && 'enabled' || 'disabled' }} -Drun_in_ci=true --fatal-meson-warnings
4545

4646
- name: Build
4747
run: meson compile -C build

.github/workflows/test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ jobs:
3535
sudo apt-get install ninja-build libsdl2-2.0-0 libsdl2-dev libsdl2-ttf* libsdl2-mixer* libsdl2-image* gcovr desktop-file-utils -y --no-install-recommends
3636
3737
- name: Configure
38-
run: meson setup build -Dbuildtype=debug -Db_coverage=true -Dtests=true -Dclang_libcpp=enabled -Drun_in_ci=true
38+
run: meson setup build -Dbuildtype=debug -Db_coverage=true -Dtests=true -Dclang_libcpp=enabled -Drun_in_ci=true --fatal-meson-warnings
3939

4040
- name: Run tests
4141
run: meson test -C build --verbose

.github/workflows/windows_installer.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ jobs:
4141
run: 7z x "-oC:/Program Files (x86)/NSIS" "${{ github.workspace }}/envar_plugin.zip"
4242

4343
- name: Configure
44-
run: meson setup build -Dbuildtype=release -Ddefault_library=shared -Dbuild_installer=true -Drun_in_ci=true
44+
run: meson setup build -Dbuildtype=release -Ddefault_library=shared -Dbuild_installer=true -Drun_in_ci=true --fatal-meson-warnings
4545

4646
- name: Build installer
4747
run: |

io.github.openbrickprotocolfoundation.oopetris.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ modules:
1212
- -Dbuild_installer=true
1313
- --libdir=lib
1414
- -Dtests=true
15+
- --fatal-meson-warnings
1516
- -Drun_in_ci=true # note we always want the CI checks in the case we built the flatpak
1617
- --force-fallback-for=fmt # note, the freedesktop sdk has this installed, but it is not copiable into the runtime, so we need to build it ourself, to be able to install it correctly
1718
builddir: true

platforms/build-3ds.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -272,7 +272,8 @@ if [ "$COMPILE_TYPE" == "complete_rebuild" ] || [ ! -e "$BUILD_DIR" ]; then
272272
-Dcurl:bearer-auth=enabled \
273273
-Dcurl:brotli=enabled \
274274
-Dcurl:libz=enabled \
275-
"-Drun_in_ci=$RUN_IN_CI"
275+
"-Drun_in_ci=$RUN_IN_CI" \
276+
--fatal-meson-warnings
276277

277278
fi
278279

platforms/build-android.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -448,7 +448,8 @@ EOF
448448
"-Dbuildtype=$BUILDTYPE" \
449449
-Dsdl2:use_hidapi=enabled \
450450
-Dclang_libcpp=disabled \
451-
"-Drun_in_ci=$RUN_IN_CI"
451+
"-Drun_in_ci=$RUN_IN_CI" \
452+
--fatal-meson-warnings
452453

453454
fi
454455

platforms/build-switch.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,8 @@ if [ "$COMPILE_TYPE" == "complete_rebuild" ] || [ ! -e "$BUILD_DIR" ]; then
169169
-Dcurl:bearer-auth=enabled \
170170
-Dcurl:brotli=enabled \
171171
-Dcurl:libz=enabled \
172-
"-Drun_in_ci=$RUN_IN_CI"
172+
"-Drun_in_ci=$RUN_IN_CI" \
173+
--fatal-meson-warnings
173174

174175
fi
175176

platforms/build-web.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,8 @@ if [ "$COMPILE_TYPE" == "complete_rebuild" ] || [ ! -e "$BUILD_DIR" ]; then
166166
"-Dbuildtype=$BUILDTYPE" \
167167
-Ddefault_library=static \
168168
-Dtests=false \
169-
"-Drun_in_ci=$RUN_IN_CI"
169+
"-Drun_in_ci=$RUN_IN_CI" \
170+
--fatal-meson-warnings
170171

171172
fi
172173

0 commit comments

Comments
 (0)