Re: [PATCH v6 7/8] app: reduce app dependencies

[David Marchand <david.marchand@redhat.com>]

Hello,

Cc: Akhil.

On Tue, Feb 18, 2025 at 12:16 PM Anatoly Burakov
<anatoly.burakov@intel.com> wrote:
> diff --git a/app/test-crypto-perf/meson.build b/app/test-crypto-perf/meson.build
> index fb48d9ec29..f056431bf4 100644
> --- a/app/test-crypto-perf/meson.build
> +++ b/app/test-crypto-perf/meson.build
> @@ -19,7 +19,7 @@ sources = files(
>          'cperf_test_verify.c',
>          'main.c',
>  )
> -deps += ['cryptodev', 'net', 'security']
> +deps += ['cryptodev']
>  optional_deps += 'crypto_scheduler'
>
>  cflags += no_wvla_cflag

- This app does seem to rely on lib/net.

[989/1266] Compiling C object
app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o
FAILED: app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o
ccache aarch64-linux-gnu-gcc -Iapp/dpdk-test-crypto-perf.p -Iapp
-I../app -Ilib/cryptodev -I../lib/cryptodev -Ilib/mbuf -I../lib/mbuf
-Ilib/mempool -I../lib/mempool -Ilib/ring -I../lib/ring -I. -I..
-Iconfig -I../config -Ilib/eal/include -I../lib/eal/include
-Ilib/eal/linux/include -I../lib/eal/linux/include
-Ilib/eal/arm/include -I../lib/eal/arm/include -I../kernel/linux
-Ilib/eal/common -I../lib/eal/common -Ilib/eal -I../lib/eal
-Ilib/kvargs -I../lib/kvargs -Ilib/log -I../lib/log -Ilib/metrics
-I../lib/metrics -Ilib/telemetry -I../lib/telemetry -Ilib/rcu
-I../lib/rcu -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64
-Wall -Winvalid-pch -Wextra -Werror -std=c11 -O3 -include rte_config.h
-Wvla -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-packed-not-aligned
-Wno-missing-field-initializers -D_GNU_SOURCE -mcpu=cortex-a72
-DALLOW_EXPERIMENTAL_API -Wno-vla -MD -MQ
app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o -MF
app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o.d -o
app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o -c
../app/test-crypto-perf/cperf_ops.c
../app/test-crypto-perf/cperf_ops.c:6:10: fatal error: rte_ether.h: No
such file or directory
    6 | #include <rte_ether.h>
      |          ^~~~~~~~~~~~~
compilation terminated.
ninja: build stopped: subcommand failed.


- Btw, rte_security.h is included in app/test-crypto-perf/cperf_ops.c.
However, given there is a #ifdef, I suspect the intention was to have
lib/security as an optional dep.


-- 
David Marchand