Re: [OE-core] [PATCH] pulseaudio: add m4-native to DEPENDS

[Khem Raj <raj.khem@gmail.com>]

On 7/15/22 1:10 PM, Richard Purdie wrote:
> On Fri, 2022-07-15 at 13:05 -0400, Khem Raj wrote:
>>
>> On 7/15/22 7:05 AM, Ross Burton wrote:
>>> Normally m4-native ends up in the sysroot via the toolchain, but if a
>>> non-standard toolchain is used them m4-native may not be installed.
>>>
>>> However Pulseaudio explicitly checks for m4 in the meson.build, so add
>>> it to DEPENDS.
>>
>> I think default toolchain dependencies should be emulated by external
>> toolchains too. Or perhaps m4-native should be dropped from the internal
>> toolchain too.
> 
> See the section in OE-Core's layer.conf:
> 
> """
> # Avoid adding bison-native to the sysroot without a specific
> # dependency in the recipe. This means indirect dependencies
> # (e.g. X -> Y -> binutils-cross -> bison-native) no longer meet the
> # dependency incidentally. This improves determinism and avoids build
> # failures when people switch to external toolchains.
> # libarchive only needs e2fsprogs headers at buildtime
> SSTATE_EXCLUDEDEPS_SYSROOT += "\
>      .*->autoconf-native \
>      .*->automake-native \
>      .*->bison-native \
> """
> 
> If we add m4-native to that list, it would do that. Just need someone
> to work through all the failures it might generate.

interesting. m4 is mostly used by autotools based packages barring few 
exception like pulseaudio, Maybe moving m4 dependency to autotools class 
might be able to address most of fallouts.

> 
> Cheers,
> 
> Richard