If the user specified their own CFLAGS_EXTRA these should not be
overwritten by `-e`.

Signed-off-by: Thomas Weißschuh <li...@weissschuh.net>
---
 tools/testing/selftests/nolibc/run-tests.sh | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/nolibc/run-tests.sh 
b/tools/testing/selftests/nolibc/run-tests.sh
index 0446e6326a40..324509b99e2c 100755
--- a/tools/testing/selftests/nolibc/run-tests.sh
+++ b/tools/testing/selftests/nolibc/run-tests.sh
@@ -15,7 +15,7 @@ download_location="${cache_dir}/crosstools/"
 build_location="$(realpath "${cache_dir}"/nolibc-tests/)"
 perform_download=0
 test_mode=system
-CFLAGS_EXTRA="-Werror"
+werror=1
 archs="i386 x86_64 arm64 arm mips32le mips32be ppc ppc64 ppc64le riscv s390 
loongarch"
 
 TEMP=$(getopt -o 'j:d:c:b:a:m:peh' -n "$0" -- "$@")
@@ -69,7 +69,7 @@ while true; do
                        test_mode="$2"
                        shift 2; continue ;;
                '-e')
-                       CFLAGS_EXTRA=""
+                       werror=0
                        shift; continue ;;
                '-h')
                        print_usage
@@ -140,6 +140,9 @@ test_arch() {
        ct_abi=$(crosstool_abi "$1")
        cross_compile=$(realpath 
"${download_location}gcc-${crosstool_version}-nolibc/${ct_arch}-${ct_abi}/bin/${ct_arch}-${ct_abi}-")
        build_dir="${build_location}/${arch}"
+       if [ "$werror" -ne 0 ]; then
+               CFLAGS_EXTRA="$CFLAGS_EXTRA -Werror"
+       fi
        MAKE=(make -j"${nproc}" XARCH="${arch}" 
CROSS_COMPILE="${cross_compile}" O="${build_dir}")
 
        mkdir -p "$build_dir"

-- 
2.45.2


Reply via email to