>-----Original Message----- >From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] >Sent: Monday, January 9, 2017 4:25 PM >To: Mrozowicz, SlawomirX <slawomirx.mrozow...@intel.com>; Doherty, >Declan <declan.dohe...@intel.com> >Cc: dev@dpdk.org; Azarewicz, PiotrX T <piotrx.t.azarew...@intel.com>; >Kerlin, Marcin <marcin.ker...@intel.com>; Kobylinski, MichalX ><michalx.kobylin...@intel.com> >Subject: Re: [dpdk-dev] [PATCH v2 3/3] app/crypto-perf: introduce new >performance test application > >2017-01-05 17:50, Slawomir Mrozowicz: >> --- a/app/Makefile >> +++ b/app/Makefile >> @@ -38,5 +38,6 @@ DIRS-$(CONFIG_RTE_TEST_PMD) += test-pmd >> DIRS-$(CONFIG_RTE_LIBRTE_CMDLINE) += cmdline_test >> DIRS-$(CONFIG_RTE_EXEC_ENV_LINUXAPP) += proc_info >> DIRS-$(CONFIG_RTE_LIBRTE_PDUMP) += pdump >> +DIRS-$(CONFIG_RTE_APP_CRYPTO_PERF) += crypto-perf > >Could we rename the directory to test-crypto or test-crypto-perf for >consistency? > >> --- /dev/null >> +++ b/app/crypto-perf/Makefile >[...] >> +APP = dpdk-crypto-perf > >I think we should have "test" in the name of the app. >dpdk-test-crypto? >dpdk-test-crypto-perf?
I am going to change the directory to: /app/test-crypto-perf the app name to: dpdk-test-crypto-perf Sławomir
