* [Buildroot] [PATCH 1/2] package/quickjs: bump to version 2020-11-08
@ 2020-12-05 6:44 Francois Perrad
2020-12-05 6:44 ` [Buildroot] [PATCH 2/2] package/quickjs: depends on glibc Francois Perrad
2020-12-05 20:49 ` [Buildroot] [PATCH 1/2] package/quickjs: bump to version 2020-11-08 Thomas Petazzoni
0 siblings, 2 replies; 5+ messages in thread
From: Francois Perrad @ 2020-12-05 6:44 UTC (permalink / raw
To: buildroot
Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
package/quickjs/quickjs.hash | 2 +-
package/quickjs/quickjs.mk | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/package/quickjs/quickjs.hash b/package/quickjs/quickjs.hash
index 076e4bddb..5dbd4575f 100644
--- a/package/quickjs/quickjs.hash
+++ b/package/quickjs/quickjs.hash
@@ -1,2 +1,2 @@
# Locally calculated
-sha256 0021a3e8cdc6b61e225411d05e2841d2437e1ccf4b4cabb9a5f7685ebfb57717 quickjs-2020-09-06.tar.xz
+sha256 2e9d63dab390a95ed365238f21d8e9069187f7ed195782027f0ab311bb64187b quickjs-2020-11-08.tar.xz
diff --git a/package/quickjs/quickjs.mk b/package/quickjs/quickjs.mk
index 557ffb196..1a2e63278 100644
--- a/package/quickjs/quickjs.mk
+++ b/package/quickjs/quickjs.mk
@@ -4,7 +4,7 @@
#
################################################################################
-QUICKJS_VERSION = 2020-09-06
+QUICKJS_VERSION = 2020-11-08
QUICKJS_SOURCE = quickjs-$(QUICKJS_VERSION).tar.xz
QUICKJS_SITE = https://bellard.org/quickjs
QUICKJS_LICENSE = MIT
--
2.27.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 2/2] package/quickjs: depends on glibc
2020-12-05 6:44 [Buildroot] [PATCH 1/2] package/quickjs: bump to version 2020-11-08 Francois Perrad
@ 2020-12-05 6:44 ` Francois Perrad
2020-12-05 20:45 ` Thomas Petazzoni
2020-12-05 20:49 ` [Buildroot] [PATCH 1/2] package/quickjs: bump to version 2020-11-08 Thomas Petazzoni
1 sibling, 1 reply; 5+ messages in thread
From: Francois Perrad @ 2020-12-05 6:44 UTC (permalink / raw
To: buildroot
fix build http://autobuild.buildroot.org/results/d45/d456eefffd138a714e367f9520adb662f5e0f57e/
missing include <stdatomic.h>
Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
package/quickjs/Config.in | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/package/quickjs/Config.in b/package/quickjs/Config.in
index ee53744bd..91b8bf8dd 100644
--- a/package/quickjs/Config.in
+++ b/package/quickjs/Config.in
@@ -2,7 +2,7 @@ config BR2_PACKAGE_QUICKJS
bool "quickjs"
depends on !BR2_STATIC_LIBS
# No way to check for fenv support.
- depends on !BR2_TOOLCHAIN_USES_UCLIBC
+ depends on BR2_TOOLCHAIN_USES_GLIBC
depends on BR2_USE_MMU # fork()
help
QuickJS is a small and embeddable Javascript engine.
@@ -11,6 +11,6 @@ config BR2_PACKAGE_QUICKJS
https://bellard.org/quickjs/
-comment "quickjs needs a glibc or musl toolchain w/ dynamic library"
+comment "quickjs needs a glibc toolchain w/ dynamic library"
depends on BR2_USE_MMU
- depends on BR2_STATIC_LIBS || BR2_TOOLCHAIN_USES_UCLIBC
+ depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_USES_GLIBC
--
2.27.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 2/2] package/quickjs: depends on glibc
2020-12-05 6:44 ` [Buildroot] [PATCH 2/2] package/quickjs: depends on glibc Francois Perrad
@ 2020-12-05 20:45 ` Thomas Petazzoni
2020-12-05 23:14 ` Peter Seiderer
0 siblings, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2020-12-05 20:45 UTC (permalink / raw
To: buildroot
On Sat, 5 Dec 2020 07:44:42 +0100
Francois Perrad <fperrad@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.
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/2] package/quickjs: bump to version 2020-11-08
2020-12-05 6:44 [Buildroot] [PATCH 1/2] package/quickjs: bump to version 2020-11-08 Francois Perrad
2020-12-05 6:44 ` [Buildroot] [PATCH 2/2] package/quickjs: depends on glibc Francois Perrad
@ 2020-12-05 20:49 ` Thomas Petazzoni
1 sibling, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2020-12-05 20:49 UTC (permalink / raw
To: buildroot
On Sat, 5 Dec 2020 07:44:41 +0100
Francois Perrad <fperrad@gmail.com> wrote:
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
> package/quickjs/quickjs.hash | 2 +-
> package/quickjs/quickjs.mk | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
Applied to master, thanks.
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 2/2] package/quickjs: depends on glibc
2020-12-05 20:45 ` Thomas Petazzoni
@ 2020-12-05 23:14 ` Peter Seiderer
0 siblings, 0 replies; 5+ messages in thread
From: Peter Seiderer @ 2020-12-05 23:14 UTC (permalink / raw
To: buildroot
Hello Francois, Thomas,
On Sat, 5 Dec 2020 21:45:40 +0100, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:
> On Sat, 5 Dec 2020 07:44:42 +0100
> Francois Perrad <fperrad@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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-12-05 23:14 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-12-05 6:44 [Buildroot] [PATCH 1/2] package/quickjs: bump to version 2020-11-08 Francois Perrad
2020-12-05 6:44 ` [Buildroot] [PATCH 2/2] package/quickjs: depends on glibc Francois Perrad
2020-12-05 20:45 ` Thomas Petazzoni
2020-12-05 23:14 ` Peter Seiderer
2020-12-05 20:49 ` [Buildroot] [PATCH 1/2] package/quickjs: bump to version 2020-11-08 Thomas Petazzoni
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.