[EXTERNAL] Re: [PATCH v14 4/4] eal: add PMU support to tracing library

[Tomasz Duszynski]

>-----Original Message-----
>From: David Marchand <david.marchand at redhat.com>
>Sent: Friday, October 11, 2024 3:29 PM
>To: Tomasz Duszynski <tduszynski at marvell.com>
>Cc: Jerin Jacob <jerinj at marvell.com>; Sunil Kumar Kori <skori at marvell.com>; Tyler Retzlaff
><roretzla at linux.microsoft.com>; Ruifeng.Wang at arm.com; bruce.richardson at intel.com; dev at dpdk.org;
>konstantin.v.ananyev at yandex.ru; mattias.ronnblom at ericsson.com; mb at smartsharesystems.com;
>thomas at monjalon.net; zhoumin at loongson.cn; stephen at networkplumber.org
>Subject: [EXTERNAL] Re: [PATCH v14 4/4] eal: add PMU support to tracing library
>
>On Fri, Oct 11, 2024 at 11: 50 AM Tomasz Duszynski <tduszynski@ marvell. com> wrote: > diff --git
>a/lib/eal/meson. build b/lib/eal/meson. build > index e1d6c4cf17. . 2b1a1eb283 100644 > ---
>a/lib/eal/meson. build > +++ b/lib/eal/meson. build 
>On Fri, Oct 11, 2024 at 11:50 AM Tomasz Duszynski <tduszynski at marvell.com> wrote:
>> diff --git a/lib/eal/meson.build b/lib/eal/meson.build index
>> e1d6c4cf17..2b1a1eb283 100644
>> --- a/lib/eal/meson.build
>> +++ b/lib/eal/meson.build
>> @@ -14,7 +14,7 @@ subdir(exec_env)
>>
>>  subdir(arch_subdir)
>>
>> -deps += ['log', 'kvargs']
>> +deps += ['log', 'kvargs', 'pmu']
>
>Quick comment as I saw a failure in CI.
>
>An optional library should not be a required dependency for EAL.
>You need something like (untested):
>
>if dpdk_conf.has('RTE_LIB_PMU')
>    deps += ['pmu']
>endif
>

Okay, I'll double check. Thanks. 

>>  if not is_windows
>>      deps += ['telemetry']
>>  endif
>
>
>--
>David Marchand