devtools: remove NFP from test-build
Checks
Commit Message
NFP PMD does not have any external dependency.
It only requires Linux OS, so it is not needed
to be enabled in the test-build script.
Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
---
devtools/test-build.sh | 1 -
1 file changed, 1 deletion(-)
Comments
On Thu, Jul 19, 2018 at 4:03 AM, Pablo de Lara <
pablo.de.lara.guarch@intel.com> wrote:
> NFP PMD does not have any external dependency.
> It only requires Linux OS, so it is not needed
> to be enabled in the test-build script.
>
> Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> ---
> devtools/test-build.sh | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/devtools/test-build.sh b/devtools/test-build.sh
> index 3ea82bb46..f6db99fb0 100755
> --- a/devtools/test-build.sh
> +++ b/devtools/test-build.sh
> @@ -153,7 +153,6 @@ config () # <directory> <target> <options>
> sed -ri 's,(PMD_SZEDATA2=)n,\1y,' $1/.config
> test "$DPDK_DEP_ZLIB" != y || \
> sed -ri 's,(BNX2X_PMD=)n,\1y,' $1/.config
> - sed -ri 's,(NFP_PMD=)n,\1y,' $1/.config
> test "$DPDK_DEP_PCAP" != y || \
> sed -ri 's,(PCAP=)n,\1y,' $1/.config
> test -z "$ARMV8_CRYPTO_LIB_PATH" || \
> --
> 2.14.4
>
>
Acked: Alejandro Lucero <alejandro.lucero@netronome.com>
19/07/2018 13:11, Alejandro Lucero:
> On Thu, Jul 19, 2018 at 4:03 AM, Pablo de Lara <
> pablo.de.lara.guarch@intel.com> wrote:
>
> > NFP PMD does not have any external dependency.
> > It only requires Linux OS, so it is not needed
> > to be enabled in the test-build script.
> >
> > Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> >
> Acked: Alejandro Lucero <alejandro.lucero@netronome.com>
For reference, it could have been removed when enabling the PMD:
Fixes: 80987c40fd28 ("config: enable nfp driver on Linux")
Applied, thanks
@@ -153,7 +153,6 @@ config () # <directory> <target> <options>
sed -ri 's,(PMD_SZEDATA2=)n,\1y,' $1/.config
test "$DPDK_DEP_ZLIB" != y || \
sed -ri 's,(BNX2X_PMD=)n,\1y,' $1/.config
- sed -ri 's,(NFP_PMD=)n,\1y,' $1/.config
test "$DPDK_DEP_PCAP" != y || \
sed -ri 's,(PCAP=)n,\1y,' $1/.config
test -z "$ARMV8_CRYPTO_LIB_PATH" || \