From 9cd7090345e8744f001b5fef354ff65e090c5393 Mon Sep 17 00:00:00 2001 From: Marc Herbert Date: Wed, 12 Apr 2023 21:20:39 +0000 Subject: [PATCH] .github/zephyr: add -Werror to AFLAGS, not just CFLAGS Past Zephyr experience showed that CFLAGS is not enough to cover all cases: https://github.com/zephyrproject-rtos/zephyr/commit/8a603da6cde63 Signed-off-by: Marc Herbert --- .github/workflows/zephyr.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/zephyr.yml b/.github/workflows/zephyr.yml index 59aa94dcf..40deff8eb 100644 --- a/.github/workflows/zephyr.yml +++ b/.github/workflows/zephyr.yml @@ -138,6 +138,7 @@ jobs: - name: build run: cd workspace && ./sof/zephyr/docker-run.sh ./sof/zephyr/docker-build.sh --cmake-args=-DEXTRA_CFLAGS=-Werror + --cmake-args=-DEXTRA_AFLAGS='-Werror -Wa,--fatal-warnings' --cmake-args=--warn-uninitialized ${{ matrix.IPC_platforms }} - name: Upload build artifacts @@ -282,6 +283,7 @@ jobs: run: python sof/scripts/xtensa-build-zephyr.py --no-interactive --cmake-args=-DEXTRA_CFLAGS=-Werror + --cmake-args=-DEXTRA_AFLAGS='-Werror -Wa,--fatal-warnings' --cmake-args=--warn-uninitialized ${{ matrix.platforms }} - name: Upload build artifacts