Commit b4250450 authored by Konrad Sztyber's avatar Konrad Sztyber Committed by Jim Harris
Browse files

autopackage: disable unit tests 



It doesn't make sense to include them as part of a release build and
they introduce bogus compiler warnings with LTO enabled:

nvme_pcie_common_ut.c: In function ‘test_nvme_pcie_poll_group_get_stats’:
nvme_pcie_common_ut.c:575:9: error: ‘__builtin_memcmp_eq’ specified bound 72 exceeds source size 0 [-Werror=stringop-overread]
  575 |         CU_ASSERT(memcmp(&tgroup_stat->pcie, &pgroup->stats, sizeof(struct spdk_nvme_pcie_stat)) == 0);
      |         ^

Fixes #3274

Signed-off-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I021367f021647a018aa734f6d0bcd9721824a4a7
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22059


Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Reviewed-by: default avatarMichal Berger <michal.berger@intel.com>
parent 581e7134

autopackage.sh

+1 −1
Original line number Diff line number Diff line
@@ -34,7 +34,7 @@ if [[ $CC == *clang* ]]; then 
fi



config_params="$(get_config_params | sed 's/--enable-debug//g')"

"$rootdir/configure" $config_params --enable-lto

"$rootdir/configure" $config_params --enable-lto --disable-unit-tests



$MAKE ${MAKEFLAGS}

$MAKE ${MAKEFLAGS} clean