[dpdk-dev] app: don't try to build test-crypto-perf if cryptodev disabled

Message ID 1491812059-2028-1-git-send-email-i.maximets@samsung.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers

Checks

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

Commit Message

Ilya Maximets April 10, 2017, 8:14 a.m. UTC
  This fixes build in following configuration:

	CONFIG_RTE_LIBRTE_CRYPTODEV=n
	CONFIG_RTE_APP_CRYPTO_PERF=y

Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test application")

Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
---
 app/Makefile | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
  

Comments

De Lara Guarch, Pablo April 10, 2017, 9:51 a.m. UTC | #1
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ilya Maximets
> Sent: Monday, April 10, 2017 9:14 AM
> To: dev@dpdk.org; Doherty, Declan
> Cc: Heetae Ahn; Thomas Monjalon; Ilya Maximets
> Subject: [dpdk-dev] [PATCH] app: don't try to build test-crypto-perf if
> cryptodev disabled
> 
> This fixes build in following configuration:
> 
> 	CONFIG_RTE_LIBRTE_CRYPTODEV=n
> 	CONFIG_RTE_APP_CRYPTO_PERF=y
> 
> Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test
> application")
> 
> Signed-off-by: Ilya Maximets <i.maximets@samsung.com>

Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
  
Thomas Monjalon April 10, 2017, 10:01 a.m. UTC | #2
2017-04-10 09:51, De Lara Guarch, Pablo:
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ilya Maximets
> > 
> > This fixes build in following configuration:
> > 
> > 	CONFIG_RTE_LIBRTE_CRYPTODEV=n
> > 	CONFIG_RTE_APP_CRYPTO_PERF=y
> > 
> > Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test
> > application")

Cc: stable@dpdk.org

> > Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
> 
> Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Applied, thanks
  

Patch

diff --git a/app/Makefile b/app/Makefile
index 4b3a448..c3aeebf 100644
--- a/app/Makefile
+++ b/app/Makefile
@@ -32,8 +32,11 @@ 
 include $(RTE_SDK)/mk/rte.vars.mk
 
 DIRS-$(CONFIG_RTE_TEST_PMD) += test-pmd
-DIRS-$(CONFIG_RTE_APP_CRYPTO_PERF) += test-crypto-perf
 DIRS-$(CONFIG_RTE_EXEC_ENV_LINUXAPP) += proc_info
 DIRS-$(CONFIG_RTE_LIBRTE_PDUMP) += pdump
 
+ifeq ($(CONFIG_RTE_LIBRTE_CRYPTODEV),y)
+DIRS-$(CONFIG_RTE_APP_CRYPTO_PERF) += test-crypto-perf
+endif
+
 include $(RTE_SDK)/mk/rte.subdir.mk