[dpdk-dev] mk: fix message when test application is not built

Message ID 1489770555-25936-1-git-send-email-olivier.matz@6wind.com
State Accepted
Headers show

Checks

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

Commit Message

Olivier Matz March 17, 2017, 5:09 p.m.
To build the tests, we should use "make test-build".

Fixes: 64592d97c1ae ("mk: do not build tests by default")

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
 mk/rte.sdktest.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Ferruh Yigit March 20, 2017, 11:35 a.m.
On 3/17/2017 5:09 PM, Olivier Matz wrote:
> To build the tests, we should use "make test-build".
> 
> Fixes: 64592d97c1ae ("mk: do not build tests by default")
> 
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
Thomas Monjalon March 27, 2017, 8:42 a.m.
2017-03-20 11:35, Ferruh Yigit:
> On 3/17/2017 5:09 PM, Olivier Matz wrote:
> > To build the tests, we should use "make test-build".
> > 
> > Fixes: 64592d97c1ae ("mk: do not build tests by default")
> > 
> > Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

Applied, thanks

Patch hide | download patch | download mbox

diff --git a/mk/rte.sdktest.mk b/mk/rte.sdktest.mk
index 9cc7d59..daeea90 100644
--- a/mk/rte.sdktest.mk
+++ b/mk/rte.sdktest.mk
@@ -62,7 +62,7 @@  test test-basic test-fast test-perf:
 			$(RTE_TARGET) \
 			$(BLACKLIST) $(WHITELIST); \
 	else \
-		echo "No test found, please do a 'make build' first, or specify O=" ; \
+		echo "No test found, please do a 'make test-build' first, or specify O=" ; \
 	fi
 
 # this is a special target to ease the pain of running coverage tests
@@ -80,5 +80,5 @@  coverage:
 			$(BLACKLIST) $(WHITELIST) ; \
 		$(RTE_OUTPUT)/app/dpdk-procinfo --file-prefix=ring_perf -- -m; \
 	else \
-		echo "No test found, please do a 'make build' first, or specify O=" ;\
+		echo "No test found, please do a 'make test-build' first, or specify O=" ;\
 	fi