[dpdk-dev] devtools: add test build to default build

Message ID 20170327143502.9109-1-ferruh.yigit@intel.com (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Ferruh Yigit March 27, 2017, 2:35 p.m. UTC
  Test build removed from default build, it needs to be added explicitly
to the test-build script.

Fixes: 7d3b1ec47fae ("test: move unit tests to separate directory")

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 devtools/test-build.sh | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
  

Comments

Thomas Monjalon April 6, 2017, 7:34 p.m. UTC | #1
2017-03-27 15:35, Ferruh Yigit:
> Test build removed from default build, it needs to be added explicitly
> to the test-build script.
> 
> Fixes: 7d3b1ec47fae ("test: move unit tests to separate directory")
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

Bruce's version has been picked:
	http://dpdk.org/patch/22750
  

Patch

diff --git a/devtools/test-build.sh b/devtools/test-build.sh
index 84d3165..fc00bf3 100755
--- a/devtools/test-build.sh
+++ b/devtools/test-build.sh
@@ -230,9 +230,13 @@  for conf in $configs ; do
 	make -j$J EXTRA_CFLAGS="$maxerr $DPDK_DEP_CFLAGS" \
 		EXTRA_LDFLAGS="$DPDK_DEP_LDFLAGS" $verbose O=$dir
 	! $short || break
-	echo "================== Build examples for $dir"
 	export RTE_SDK=$(pwd)
 	export RTE_TARGET=$dir
+	echo "================== Build tests for $dir"
+	make -j$J EXTRA_CFLAGS="$maxerr $DPDK_DEP_CFLAGS" \
+		EXTRA_LDFLAGS="$DPDK_DEP_LDFLAGS" $verbose O=$dir \
+		test-build
+	echo "================== Build examples for $dir"
 	make -j$J -sC examples \
 		EXTRA_LDFLAGS="$DPDK_DEP_LDFLAGS" $verbose \
 		O=$(readlink -m $dir/examples)