Re: [Buildroot] [PATCH] package/graphene: needs a toolchain with threads

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello François,

On Sun, 11 Aug 2024 10:07:55 +0200
Francois Perrad <francois.perrad@gadz.org> wrote:

> /home/autobuild/autobuild/instance-9/output-1/host/aarch64-buildroot-linux-uclibc/sysroot/usr/include/features.h:220:5: warning: #warning requested reentrant code, but thread support was disabled [-Wcpp]
> 
> fix http://autobuild.buildroot.org/results/9b3/9b357b1ef2eccd3110d0ecaaff1ae4883dff9280
> 
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
>  package/graphene/Config.in | 4 ++++
>  1 file changed, 4 insertions(+)

Thanks a lot, but I believe this is papering over the real issue.
Indeed the code has some nice:

#ifdef HAVE_PTHREAD
...
#endif

around all thread related code. However, the:

threadlib = dependency("threads")

in meson.build adds -pthread to the build command line, causing this
issue. I am not an expert in Meson so I'm not 100% sure how to fix
this, and to understand 100% what is the semantic of
dependency("threads"), but this is what needs to be fixed (or
alternatively reported upstream). Maybe meson.build needs to be changed this way:

test if pthread.h is here, and if it is:
 set HAVE_PHTHREAD
 threadlib = dependency("threads")

something along those lines. Could you have a look?

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot