[dpdk-dev] [PATCH] devtools: fix examples external compilation for x86-default

Subsystems: the rest

2 messages, 2 authors, 2021-02-02 · open the first message on its own page

[dpdk-dev] [PATCH] devtools: fix examples external compilation for x86-default

From: David Marchand <hidden>
Date: 2021-02-02 10:59:07

Since we don't check ABI on the x86-default target anymore, installation
of the target must always happen for examples external compilation check
to work.

Fixes: 6a426d733ec0 ("devtools: reduce ABI checks and static binaries")

Signed-off-by: David Marchand <redacted>
---
 devtools/test-meson-builds.sh | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index a99a177e0c..c11ae87e0d 100755
--- a/devtools/test-meson-builds.sh
+++ b/devtools/test-meson-builds.sh
@@ -276,10 +276,7 @@ done
 load_env cc
 build_path=$(readlink -f $builds_dir/build-x86-default)
 export DESTDIR=$build_path/install
-# No need to reinstall if ABI checks are enabled
-if [ -z "$DPDK_ABI_REF_VERSION" ]; then
-	install_target $build_path $DESTDIR
-fi
+install_target $build_path $DESTDIR
 pc_file=$(find $DESTDIR -name libdpdk.pc)
 export PKG_CONFIG_PATH=$(dirname $pc_file):$PKG_CONFIG_PATH
 libdir=$(dirname $(find $DESTDIR -name librte_eal.so))
-- 
2.23.0

Re: [dpdk-dev] [PATCH] devtools: fix examples external compilation for x86-default

From: Thomas Monjalon <hidden>
Date: 2021-02-02 22:54:53

02/02/2021 11:54, David Marchand:
Since we don't check ABI on the x86-default target anymore, installation
of the target must always happen for examples external compilation check
to work.
Otherwise the error is:
find: ‘build-x86-default/install’: No such file or directory
Fixes: 6a426d733ec0 ("devtools: reduce ABI checks and static binaries")

Signed-off-by: David Marchand <redacted>
Acked-by: Thomas Monjalon <redacted>

Applied, thanks


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help