There is no point in forcing warning-free compilation when building
an ABI reference. It is only preventing from compiling ABI reference
of old releases with recent compilers.

Note: DPDK 20.02 is built (with warnings) by GCC 10 if using -fcommon.

Signed-off-by: Thomas Monjalon <tho...@monjalon.net>
---
 devtools/test-build.sh        | 6 ++----
 devtools/test-meson-builds.sh | 3 +--
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/devtools/test-build.sh b/devtools/test-build.sh
index 6e53f86fc8..f013656024 100755
--- a/devtools/test-build.sh
+++ b/devtools/test-build.sh
@@ -68,8 +68,6 @@ J=$DPDK_MAKE_JOBS
 builds_dir=${DPDK_BUILD_TEST_DIR:-.}
 short=false
 unset verbose
-# for ABI checks, we need debuginfo
-test_cflags="-Wfatal-errors -g"
 while getopts hj:sv ARG ; do
        case $ARG in
                j ) J=$OPTARG ;;
@@ -248,7 +246,7 @@ for conf in $configs ; do
        config $dir $target $options
 
        echo "================== Build $conf"
-       ${MAKE} -j$J EXTRA_CFLAGS="$test_cflags $DPDK_DEP_CFLAGS" \
+       ${MAKE} -j$J EXTRA_CFLAGS="-Wfatal-errors -g $DPDK_DEP_CFLAGS" \
                EXTRA_LDFLAGS="$DPDK_DEP_LDFLAGS" $verbose O=$dir
        ! $short || break
        export RTE_TARGET=$target
@@ -282,7 +280,7 @@ for conf in $configs ; do
                        echo -n "================== Build $conf "
                        echo "($DPDK_ABI_REF_VERSION)"
                        ${MAKE} -j$J \
-                               EXTRA_CFLAGS="$test_cflags $DPDK_DEP_CFLAGS" \
+                               EXTRA_CFLAGS="-Wno-error -g $DPDK_DEP_CFLAGS" \
                                EXTRA_LDFLAGS="$DPDK_DEP_LDFLAGS" $verbose \
                                O=$abirefdir/build
                        export RTE_TARGET=$target
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index e8df017596..18b874fac5 100755
--- a/devtools/test-meson-builds.sh
+++ b/devtools/test-meson-builds.sh
@@ -74,7 +74,6 @@ config () # <dir> <builddir> <meson options>
                return
        fi
        options=
-       options="$options --werror"
        if echo $* | grep -qw -- '--default-library=shared' ; then
                options="$options -Dexamples=all"
        else
@@ -127,7 +126,7 @@ build () # <directory> <target compiler> <meson options>
        # skip build if compiler not available
        command -v ${CC##* } >/dev/null 2>&1 || return 0
        load_env $targetcc || return 0
-       config $srcdir $builds_dir/$targetdir $*
+       config $srcdir $builds_dir/$targetdir --werror $*
        compile $builds_dir/$targetdir
        if [ -n "$DPDK_ABI_REF_VERSION" ]; then
                abirefdir=${DPDK_ABI_REF_DIR:-reference}/$DPDK_ABI_REF_VERSION
-- 
2.26.2

Reply via email to