diff mbox series

examples/ntb: fix build dependency

Message ID 20211116205135.15846-1-david.marchand@redhat.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series examples/ntb: fix build dependency |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS

Commit Message

David Marchand Nov. 16, 2021, 8:51 p.m. UTC 
  Caught while building with -Ddisable_drivers=*/*.
This example requires raw/ntb specific API.
Fix dependency to avoid a compilation error:

FAILED: examples/c590b3c@@dpdk-ntb@exe/ntb_ntb_fwd.c.o 
ccache gcc -Iexamples/c590b3c@@dpdk-ntb@exe -Iexamples -I../examples 
  -Iexamples/ntb -I../examples/ntb -I. -I../ -Iconfig -I../config
  -Ilib/eal/include -I../lib/eal/include -Ilib/eal/linux/include
  -I../lib/eal/linux/include -Ilib/eal/x86/include
  -I../lib/eal/x86/include -Ilib/eal/common -I../lib/eal/common
  -Ilib/eal -I../lib/eal -Ilib/kvargs -I../lib/kvargs
  -Ilib/telemetry/../metrics -I../lib/telemetry/../metrics
  -Ilib/telemetry -I../lib/telemetry -Ilib/mempool -I../lib/mempool
  -Ilib/ring -I../lib/ring -Ilib/net -I../lib/net -Ilib/mbuf
  -I../lib/mbuf -Ilib/ethdev -I../lib/ethdev -Ilib/meter
  -I../lib/meter -Ilib/cmdline -I../lib/cmdline -Ilib/rawdev
  -I../lib/rawdev -fdiagnostics-color=always -pipe -Wall
  -Winvalid-pch -Werror -O2 -g -include rte_config.h -Wextra
  -Wcast-qual -Wdeprecated -Wformat -Wformat-nonliteral
  -Wformat-security -Wmissing-declarations -Wmissing-prototypes
  -Wnested-externs -Wold-style-definition -Wpointer-arith
  -Wsign-compare -Wstrict-prototypes -Wundef -Wwrite-strings
  -Wno-missing-field-initializers -D_GNU_SOURCE -march=corei7
  -Wno-format-truncation -D_FILE_OFFSET_BITS=64
  -DALLOW_EXPERIMENTAL_API  -MD
  -MQ 'examples/c590b3c@@dpdk-ntb@exe/ntb_ntb_fwd.c.o'
  -MF 'examples/c590b3c@@dpdk-ntb@exe/ntb_ntb_fwd.c.o.d'
  -o 'examples/c590b3c@@dpdk-ntb@exe/ntb_ntb_fwd.c.o'
  -c ../examples/ntb/ntb_fwd.c
../examples/ntb/ntb_fwd.c:21:10: fatal error: rte_pmd_ntb.h:
  No such file or directory
 #include <rte_pmd_ntb.h>
          ^~~~~~~~~~~~~~~
compilation terminated.

Fixes: 5194299d6ef5 ("examples/ntb: support more functions")
Cc: stable@dpdk.org

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 examples/ntb/meson.build | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comments

Thomas Monjalon Nov. 17, 2021, 8:58 a.m. UTC | #1 
16/11/2021 21:51, David Marchand:
> Caught while building with -Ddisable_drivers=*/*.
> This example requires raw/ntb specific API.
> Fix dependency to avoid a compilation error:
> 
> FAILED: examples/c590b3c@@dpdk-ntb@exe/ntb_ntb_fwd.c.o 
> ccache gcc -Iexamples/c590b3c@@dpdk-ntb@exe -Iexamples -I../examples 
>   -Iexamples/ntb -I../examples/ntb -I. -I../ -Iconfig -I../config
>   -Ilib/eal/include -I../lib/eal/include -Ilib/eal/linux/include
>   -I../lib/eal/linux/include -Ilib/eal/x86/include
>   -I../lib/eal/x86/include -Ilib/eal/common -I../lib/eal/common
>   -Ilib/eal -I../lib/eal -Ilib/kvargs -I../lib/kvargs
>   -Ilib/telemetry/../metrics -I../lib/telemetry/../metrics
>   -Ilib/telemetry -I../lib/telemetry -Ilib/mempool -I../lib/mempool
>   -Ilib/ring -I../lib/ring -Ilib/net -I../lib/net -Ilib/mbuf
>   -I../lib/mbuf -Ilib/ethdev -I../lib/ethdev -Ilib/meter
>   -I../lib/meter -Ilib/cmdline -I../lib/cmdline -Ilib/rawdev
>   -I../lib/rawdev -fdiagnostics-color=always -pipe -Wall
>   -Winvalid-pch -Werror -O2 -g -include rte_config.h -Wextra
>   -Wcast-qual -Wdeprecated -Wformat -Wformat-nonliteral
>   -Wformat-security -Wmissing-declarations -Wmissing-prototypes
>   -Wnested-externs -Wold-style-definition -Wpointer-arith
>   -Wsign-compare -Wstrict-prototypes -Wundef -Wwrite-strings
>   -Wno-missing-field-initializers -D_GNU_SOURCE -march=corei7
>   -Wno-format-truncation -D_FILE_OFFSET_BITS=64
>   -DALLOW_EXPERIMENTAL_API  -MD
>   -MQ 'examples/c590b3c@@dpdk-ntb@exe/ntb_ntb_fwd.c.o'
>   -MF 'examples/c590b3c@@dpdk-ntb@exe/ntb_ntb_fwd.c.o.d'
>   -o 'examples/c590b3c@@dpdk-ntb@exe/ntb_ntb_fwd.c.o'
>   -c ../examples/ntb/ntb_fwd.c
> ../examples/ntb/ntb_fwd.c:21:10: fatal error: rte_pmd_ntb.h:
>   No such file or directory
>  #include <rte_pmd_ntb.h>
>           ^~~~~~~~~~~~~~~
> compilation terminated.
> 
> Fixes: 5194299d6ef5 ("examples/ntb: support more functions")
> Cc: stable@dpdk.org
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>

Applied, thanks.
diff mbox series

Patch

diff --git a/examples/ntb/meson.build b/examples/ntb/meson.build
index 37899dfeb6..18eaffdf21 100644
--- a/examples/ntb/meson.build
+++ b/examples/ntb/meson.build
@@ -12,11 +12,8 @@  if not is_linux
     subdir_done()
 endif
 
-deps += 'rawdev'
+deps += ['rawdev', 'raw_ntb']
 cflags += ['-D_FILE_OFFSET_BITS=64']
 sources = files(
         'ntb_fwd.c',
 )
-if dpdk_conf.has('RTE_RAW_NTB')
-    deps += 'raw_ntb'
-endif