27/03/2018 14:14, Kovacevic, Marko: > From: Marko Kovacevic <marko.kovace...@intel.com> > > Signed-off-by: Marko Kovacevic <marko.kovace...@intel.com>
No comment at all? This new version doesn't work with the env variable. It is probably worth few words. > --- a/devtools/test-build.sh > +++ b/devtools/test-build.sh > @@ -183,8 +183,6 @@ config () # <directory> <target> <options> > test -z "$ARMV8_CRYPTO_LIB_PATH" || \ > sed -ri 's,(PMD_ARMV8_CRYPTO=)n,\1y,' $1/.config > test -z "$AESNI_MULTI_BUFFER_LIB_PATH" || \ > - sed -ri 's,(PMD_AESNI_MB=)n,\1y,' $1/.config > - test -z "$AESNI_MULTI_BUFFER_LIB_PATH" || \ > sed -ri 's,(PMD_AESNI_GCM=)n,\1y,' $1/.config The purpose of test-build.sh is to test the build :) So you need to provide a way to enable this PMD if the dependency is available. Look at DPDK_DEP_SSL as an example.
