Re: [dpdk-stable] [18.11 6/7] pmdinfogen: fix freebsd build

[Kevin Traynor <ktraynor@redhat.com>]

On 09/01/2020 15:53, Neil Horman wrote:
> On Thu, Jan 09, 2020 at 01:34:32PM +0000, Kevin Traynor wrote:
>> rte_compat.h is needed as part of the fixed commit
>> but it could not be found with meson on FreeBSD causing
>> a build failure. Add it's include directory.
>>
>> This is not an issue on DPDK master branch since
>> rte_compat.h was moved under lib/eal.
>>
>> Fixes: 3e41c8bc7584 ("eal: add ack interrupt API")
>>
>> Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
>> Cc: Neil Horman <nhorman@tuxdriver.com>
>> ---
>>  buildtools/pmdinfogen/meson.build | 5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/buildtools/pmdinfogen/meson.build b/buildtools/pmdinfogen/meson.build
>> index a219a8e96..ab2b20032 100644
>> --- a/buildtools/pmdinfogen/meson.build
>> +++ b/buildtools/pmdinfogen/meson.build
>> @@ -4,4 +4,9 @@
>>  pmdinfogen_inc = eal_inc
>>  pmdinfogen_inc += include_directories('../../lib/librte_pci')
>> +
>> +if host_machine.system() == 'freebsd'
>> +	pmdinfogen_inc += include_directories('../../lib/librte_compat')
>> +endif
>> +
> Actually, is there a corresponding Makefile change that needs to be made here
> too?
> 

It is building ok with make, so the header seems to getting pulled in
already along the way with the make build system.

Kevin.

> Neil
> 
>>  pmdinfogen = executable('pmdinfogen',
>>  	'pmdinfogen.c',
>> -- 
>> 2.21.1
>>
>>
>