>-----Original Message----- >From: Thomas Monjalon [mailto:[email protected]] >Sent: Monday, January 9, 2017 4:25 PM >To: Mrozowicz, SlawomirX <[email protected]>; Doherty, >Declan <[email protected]> >Cc: [email protected]; Azarewicz, PiotrX T <[email protected]>; >Kerlin, Marcin <[email protected]>; Kobylinski, MichalX ><[email protected]> >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
