[Buildroot] [PATCH 2/2] package/quickjs: depends on glibc
Peter Seiderer
ps.report at gmx.net
Sat Dec 5 23:14:50 UTC 2020
Hello Francois, Thomas,
On Sat, 5 Dec 2020 21:45:40 +0100, Thomas Petazzoni <thomas.petazzoni at bootlin.com> wrote:
> On Sat, 5 Dec 2020 07:44:42 +0100
> Francois Perrad <fperrad at gmail.com> wrote:
>
> > fix build http://autobuild.buildroot.org/results/d45/d456eefffd138a714e367f9520adb662f5e0f57e/
>
> Well, this configuration *is* using glibc, so clearly the patch you're
> proposing cannot fix this build failure.
>
> stdatomic.h is provided by gcc, not by the C library, so a C library
> option change most likely cannot be the proper solution.
>
> We already have several atomic-related options in toolchain/Config.in,
> but I'm not sure any of them is correct to express the need for
> <stdatomic.h>. In fact, I'm not quite sure in which circumstances is
> this header not provided by gcc.
According to [1] stdatomic.h is a ISO C11 feature, and despite the gcc info
supporting '-std=c11' since gcc-4.7 [2] the release notes from gcc-4.9 [3]
make mention of stdatomic.h:
ISO C11 atomics (the _Atomic type specifier and qualifier and the
<stdatomic.h> header) are now supported.
So a
depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C11/stdatomic.h
should do the job...
Regards,
Peter
[1] https://en.wikipedia.org/wiki/C11_(C_standard_revision)
[2] https://gcc.gnu.org/wiki/C11Status
[3] https://gcc.gnu.org/gcc-4.9/changes.html
>
> Thomas
More information about the buildroot
mailing list