Re: [OE-core] [PATCH] harfbuzz: remove meson flag work-around

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Fri, 2025-04-25 at 13:42 -0400, Trevor Woerner via lists.openembedded.org wrote:
> The issue is committed and resolved upstream, therefore remove the
> work-around of explicitly having to set the meson flag 'cpp_std=c++17' when
> using icu >= 75.
> 
> https://github.com/harfbuzz/harfbuzz/issues/4671
> https://github.com/harfbuzz/harfbuzz/pull/4734
> 
> Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> ---
>  meta/recipes-graphics/harfbuzz/harfbuzz_11.0.1.bb | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/meta/recipes-graphics/harfbuzz/harfbuzz_11.0.1.bb b/meta/recipes-graphics/harfbuzz/harfbuzz_11.0.1.bb
> index 8945467a001d..2febcbecda31 100644
> --- a/meta/recipes-graphics/harfbuzz/harfbuzz_11.0.1.bb
> +++ b/meta/recipes-graphics/harfbuzz/harfbuzz_11.0.1.bb
> @@ -32,9 +32,6 @@ PACKAGES =+ "${PN}-icu ${PN}-icu-dev ${PN}-subset"
>  
>  LEAD_SONAME = "libharfbuzz.so"
>  
> -# Remove when https://github.com/harfbuzz/harfbuzz/issues/4671 is resolved
> -EXTRA_OEMESON += "-Dcpp_std=c++17"
> -
>  do_install:append() {
>      # If no tools are installed due to PACKAGECONFIG then this directory might
>      # still be installed, so remove it to stop packaging warnings.
> 

Removing this caused failures on the autobuilder with gcc 15 e.g.:

https://autobuilder.yoctoproject.org/valkyrie/#/builders/2/builds/1474

but there were more in that build.

Cheers,

Richard