* [Buildroot] [PATCH] Config.in: Introduce BR2_HOST_GCC_AT_LEAST_11
@ 2023-11-03 0:06 Adam Duskett
2023-11-03 0:06 ` [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0 Adam Duskett
2023-11-03 20:16 ` [Buildroot] [PATCH] Config.in: Introduce BR2_HOST_GCC_AT_LEAST_11 Thomas Petazzoni via buildroot
0 siblings, 2 replies; 7+ messages in thread
From: Adam Duskett @ 2023-11-03 0:06 UTC (permalink / raw)
To: buildroot; +Cc: Adam Duskett, Martin Bark, Thomas Petazzoni, Daniel Price
NodeJS 20 requires gcc >= 10.1. Unfortuantly, the only host gcc that has a
decimal place is 4.9, and every other HOST_GCC_AT_LEAST is an even release,
we round up to 11.
Signed-off-by: Adam Duskett <adam.duskett@amarulasolutions.com>
---
Config.in | 10 ++++++++++
Makefile | 2 +-
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/Config.in b/Config.in
index 556b6c2575..7dd49567f6 100644
--- a/Config.in
+++ b/Config.in
@@ -55,6 +55,16 @@ config BR2_HOST_GCC_AT_LEAST_9
default y if BR2_HOST_GCC_VERSION = "9"
select BR2_HOST_GCC_AT_LEAST_8
+config BR2_HOST_GCC_AT_LEAST_10
+ bool
+ default y if BR2_HOST_GCC_VERSION = "10"
+ select BR2_HOST_GCC_AT_LEAST_9
+
+config BR2_HOST_GCC_AT_LEAST_11
+ bool
+ default y if BR2_HOST_GCC_VERSION = "11"
+ select BR2_HOST_GCC_AT_LEAST_10
+
# When adding new entries above, be sure to update
# the HOSTCC_MAX_VERSION variable in the Makefile.
diff --git a/Makefile b/Makefile
index 3e85d5ef09..37deb35912 100644
--- a/Makefile
+++ b/Makefile
@@ -353,7 +353,7 @@ export HOSTARCH := $(shell LC_ALL=C $(HOSTCC_NOCCACHE) -v 2>&1 | \
# When adding a new host gcc version in Config.in,
# update the HOSTCC_MAX_VERSION variable:
-HOSTCC_MAX_VERSION := 9
+HOSTCC_MAX_VERSION := 11
HOSTCC_VERSION := $(shell V=$$($(HOSTCC_NOCCACHE) --version | \
sed -n -r 's/^.* ([0-9]*)\.([0-9]*)\.([0-9]*)[ ]*.*/\1 \2/p'); \
--
2.41.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0
2023-11-03 0:06 [Buildroot] [PATCH] Config.in: Introduce BR2_HOST_GCC_AT_LEAST_11 Adam Duskett
@ 2023-11-03 0:06 ` Adam Duskett
2023-11-03 20:16 ` Thomas Petazzoni via buildroot
2023-11-04 16:26 ` Arnout Vandecappelle via buildroot
2023-11-03 20:16 ` [Buildroot] [PATCH] Config.in: Introduce BR2_HOST_GCC_AT_LEAST_11 Thomas Petazzoni via buildroot
1 sibling, 2 replies; 7+ messages in thread
From: Adam Duskett @ 2023-11-03 0:06 UTC (permalink / raw)
To: buildroot; +Cc: Adam Duskett, Martin Bark, Thomas Petazzoni, Daniel Price
- Refactor 0001-add-qemu-wrapper-support.patch for 20.9.0
- NodeJS now requires GCC 10.1 header. However, as there is no
BR2_TOOLCHAIN_GCC_AT_LEAST_10_1, we round up to BR2_TOOLCHAIN_GCC_AT_LEAST_11.
- Drop the --without-dtrace and --without-etw config options as they no longer
exist.
- Update support/testing/tests/package/test_nodejs.py to use
BR2_TOOLCHAIN_EXTERNAL_BOOTLIN, otherwise the test fails as the
gcc version selected will be 7.
- Update the hash file for LICENSE.txt due to numerous changes, such
as the addition of new libraries and updated dates. No new license
types are added.
Tested with: ./support/testing/run-tests tests.package.test_nodejs
Signed-off-by: Adam Duskett <adam.duskett@amarulasolutions.com>
---
package/nodejs/Config.in | 8 +--
.../0001-add-qemu-wrapper-support.patch | 51 ++++++++++++-------
...ules-cjs-loader.js-adjust-default-pa.patch | 2 +-
package/nodejs/nodejs-src/nodejs-src.mk | 4 --
package/nodejs/nodejs.hash | 15 +++---
package/nodejs/nodejs.mk | 2 +-
support/testing/tests/package/test_nodejs.py | 3 ++
7 files changed, 50 insertions(+), 35 deletions(-)
diff --git a/package/nodejs/Config.in b/package/nodejs/Config.in
index c684f8eb86..72770095d0 100644
--- a/package/nodejs/Config.in
+++ b/package/nodejs/Config.in
@@ -7,19 +7,19 @@ config BR2_PACKAGE_NODEJS_ARCH_SUPPORTS
depends on BR2_TOOLCHAIN_HAS_SYNC_4
depends on BR2_PACKAGE_HOST_QEMU_USER_ARCH_SUPPORTS
-comment "nodejs needs a toolchain w/ C++, dynamic library, NPTL, gcc >= 7, wchar, host gcc >= 8"
+comment "nodejs needs a toolchain w/ C++, dynamic library, NPTL, gcc >= 11, wchar, host gcc >= 11"
depends on BR2_USE_MMU
depends on BR2_PACKAGE_NODEJS_ARCH_SUPPORTS
depends on !BR2_INSTALL_LIBSTDCPP || BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS_NPTL || \
- !BR2_HOST_GCC_AT_LEAST_8 || !BR2_TOOLCHAIN_GCC_AT_LEAST_7 || !BR2_USE_WCHAR
+ !BR2_HOST_GCC_AT_LEAST_11 || !BR2_TOOLCHAIN_GCC_AT_LEAST_11 || !BR2_USE_WCHAR
config BR2_PACKAGE_NODEJS
bool "nodejs"
depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL # libuv
depends on BR2_INSTALL_LIBSTDCPP
depends on BR2_PACKAGE_NODEJS_ARCH_SUPPORTS
- depends on BR2_HOST_GCC_AT_LEAST_8 # qemu
- depends on BR2_TOOLCHAIN_GCC_AT_LEAST_7
+ depends on BR2_HOST_GCC_AT_LEAST_11
+ depends on BR2_TOOLCHAIN_GCC_AT_LEAST_11
depends on BR2_USE_WCHAR
# uses fork()
depends on BR2_USE_MMU
diff --git a/package/nodejs/nodejs-src/0001-add-qemu-wrapper-support.patch b/package/nodejs/nodejs-src/0001-add-qemu-wrapper-support.patch
index 184f3d19b7..18f320ec5d 100644
--- a/package/nodejs/nodejs-src/0001-add-qemu-wrapper-support.patch
+++ b/package/nodejs/nodejs-src/0001-add-qemu-wrapper-support.patch
@@ -15,26 +15,18 @@ qemu-wrapper, so add @MAYBE_WRAPPER@ to the needed files and sed the path to
the qemu-wrapper on target builds, and remove @MAYBE_WRAPPER@ entirely on
host-builds.
+[Adam: Update for 20.9.0]
Signed-off-by: Adam Duskett <aduskett@gmail.com>
---
- node.gyp | 4 ++--
+ node.gyp | 6 ++++--
tools/v8_gypfiles/v8.gyp | 10 +++++-----
- 2 files changed, 7 insertions(+), 7 deletions(-)
+ 2 files changed, 9 insertions(+), 7 deletions(-)
diff --git a/node.gyp b/node.gyp
index 97cdbb531d..db5cf51ce6 100644
--- a/node.gyp
+++ b/node.gyp
-@@ -319,7 +319,7 @@
- 'action_name': 'run_mkcodecache',
- 'process_outputs_as_sources': 1,
- 'inputs': [
-- '<(mkcodecache_exec)',
-+ @MAYBE_WRAPPER@ '<(mkcodecache_exec)',
- ],
- 'outputs': [
- '<(SHARED_INTERMEDIATE_DIR)/node_code_cache.cc',
-@@ -346,7 +346,7 @@
+@@ -662,7 +662,7 @@
'action_name': 'node_mksnapshot',
'process_outputs_as_sources': 1,
'inputs': [
@@ -43,11 +35,36 @@ index 97cdbb531d..db5cf51ce6 100644
'<(node_snapshot_main)',
],
'outputs': [
+@@ -682,7 +682,7 @@
+ 'action_name': 'node_mksnapshot',
+ 'process_outputs_as_sources': 1,
+ 'inputs': [
+- '<(node_mksnapshot_exec)',
++ @MAYBE_WRAPPER@ '<(node_mksnapshot_exec)',
+ ],
+ 'outputs': [
+ '<(SHARED_INTERMEDIATE_DIR)/node_snapshot.cc',
+@@ -966,6 +966,7 @@
+ 'action_name': 'node_js2c',
+ 'process_outputs_as_sources': 1,
+ 'inputs': [
++ @MAYBE_WRAPPER@
+ '<(node_js2c_exec)',
+ '<@(library_files)',
+ '<@(deps_files)',
+@@ -975,6 +976,7 @@
+ '<(SHARED_INTERMEDIATE_DIR)/node_javascript.cc',
+ ],
+ 'action': [
++ @MAYBE_WRAPPER@
+ '<(node_js2c_exec)',
+ '<@(_outputs)',
+ 'lib',
diff --git a/tools/v8_gypfiles/v8.gyp b/tools/v8_gypfiles/v8.gyp
index 73099a03e4..88ca91efb4 100644
--- a/tools/v8_gypfiles/v8.gyp
+++ b/tools/v8_gypfiles/v8.gyp
-@@ -68,7 +68,7 @@
+@@ -81,7 +81,7 @@
{
'action_name': 'run_torque_action',
'inputs': [ # Order matters.
@@ -56,7 +73,7 @@ index 73099a03e4..88ca91efb4 100644
'<@(torque_files)',
],
'outputs': [
-@@ -99,7 +99,7 @@
+@@ -112,7 +112,7 @@
'<@(torque_outputs_inc)',
],
'action': [
@@ -65,7 +82,7 @@ index 73099a03e4..88ca91efb4 100644
'-o', '<(SHARED_INTERMEDIATE_DIR)/torque-generated',
'-v8-root', '<(V8_ROOT)',
'<@(torque_files_without_v8_root)',
-@@ -211,7 +211,7 @@
+@@ -224,7 +224,7 @@
{
'action_name': 'generate_bytecode_builtins_list_action',
'inputs': [
@@ -74,7 +91,7 @@ index 73099a03e4..88ca91efb4 100644
],
'outputs': [
'<(generate_bytecode_builtins_list_output)',
-@@ -395,7 +395,7 @@
+@@ -401,7 +401,7 @@
],
},
'inputs': [
@@ -83,7 +100,7 @@ index 73099a03e4..88ca91efb4 100644
],
'outputs': [
'<(INTERMEDIATE_DIR)/snapshot.cc',
-@@ -1505,7 +1505,7 @@
+@@ -1660,7 +1660,7 @@
{
'action_name': 'run_gen-regexp-special-case_action',
'inputs': [
diff --git a/package/nodejs/nodejs-src/0004-lib-internal-modules-cjs-loader.js-adjust-default-pa.patch b/package/nodejs/nodejs-src/0004-lib-internal-modules-cjs-loader.js-adjust-default-pa.patch
index 5b93998e5e..25c26b43bd 100644
--- a/package/nodejs/nodejs-src/0004-lib-internal-modules-cjs-loader.js-adjust-default-pa.patch
+++ b/package/nodejs/nodejs-src/0004-lib-internal-modules-cjs-loader.js-adjust-default-pa.patch
@@ -21,7 +21,7 @@ diff --git a/lib/internal/modules/cjs/loader.js b/lib/internal/modules/cjs/loade
index f1971c40a4..5fe3884156 100644
--- a/lib/internal/modules/cjs/loader.js
+++ b/lib/internal/modules/cjs/loader.js
-@@ -1261,7 +1261,8 @@ Module._initPaths = function() {
+@@ -1378,7 +1378,8 @@ Module._initPaths = function() {
path.resolve(process.execPath, '..') :
path.resolve(process.execPath, '..', '..');
diff --git a/package/nodejs/nodejs-src/nodejs-src.mk b/package/nodejs/nodejs-src/nodejs-src.mk
index 3452c93728..6fb550f158 100644
--- a/package/nodejs/nodejs-src/nodejs-src.mk
+++ b/package/nodejs/nodejs-src/nodejs-src.mk
@@ -40,8 +40,6 @@ NODEJS_SRC_CONF_OPTS = \
--shared-cares \
--shared-libuv \
--shared-nghttp2 \
- --without-dtrace \
- --without-etw \
--cross-compiling \
--dest-os=linux \
--ninja
@@ -97,8 +95,6 @@ define HOST_NODEJS_SRC_CONFIGURE_CMDS
PYTHON=$(HOST_DIR)/bin/python3 \
$(HOST_DIR)/bin/python3 configure.py \
--prefix=$(HOST_DIR) \
- --without-dtrace \
- --without-etw \
--shared-openssl \
--shared-openssl-includes=$(HOST_DIR)/include \
--shared-openssl-libpath=$(HOST_DIR)/lib \
diff --git a/package/nodejs/nodejs.hash b/package/nodejs/nodejs.hash
index eaf41f6437..da027cc17a 100644
--- a/package/nodejs/nodejs.hash
+++ b/package/nodejs/nodejs.hash
@@ -1,9 +1,8 @@
-# From https://nodejs.org/dist/v16.20.0/SHASUMS256.txt.asc
-sha256 aca4794ca60f2e17689a700eeaa95d7adcd5fd01cdd2e4f9d596ac41b600b796 node-v16.20.0-linux-arm64.tar.xz
-sha256 5b79addab37bbdc31b442f8e648efad0b5d3aa5f3e0d682f23f4db483408eb2a node-v16.20.0-linux-armv7l.tar.xz
-sha256 b439a8f874e8c2fad287ba75a3229407551f5ff9eb41e45e43e9bf67f184f618 node-v16.20.0-linux-ppc64le.tar.xz
-sha256 dff21020b555cc165a1ac36da7d4f6c810b35409c94e00afc51d5d370aae47ae node-v16.20.0-linux-x64.tar.xz
-sha256 e0990f992234e40a51fe11f92c3816c93a77e1b081145d3dd762cd1026345349 node-v16.20.0.tar.xz
-
+# From https://nodejs.org/dist/v20.9.0/SHASUMS256.txt.asc
+sha256 ced3ecece4b7c3a664bca3d9e34a0e3b9a31078525283a6fdb7ea2de8ca5683b node-v20.9.0-linux-arm64.tar.xz
+sha256 64099d7e62bdbb8efd52c216a6759b3cdd77c38aa993096fdee9f875d95b2d8e node-v20.9.0-linux-armv7l.tar.xz
+sha256 3c6cea5d614cfbb95d92de43fbc2f8ecd66e431502fe5efc4f3c02637897bd45 node-v20.9.0-linux-ppc64le.tar.xz
+sha256 9033989810bf86220ae46b1381bdcdc6c83a0294869ba2ad39e1061f1e69217a node-v20.9.0-linux-x64.tar.xz
+sha256 a23d96810abf0455426b349d47ce5310f33095b7bc0571b9cc510f481c3a4519 node-v20.9.0.tar.xz
# Locally calculated
-sha256 ba325815d3df8819bebaf37cad67d6e1f82271e1e4a1189b53abd28e261977d6 LICENSE
+sha256 a6871483905199033882d66453649675f0e88ffaa07f9f8acb257ce7504bc758 LICENSE
diff --git a/package/nodejs/nodejs.mk b/package/nodejs/nodejs.mk
index 2a8c570932..7c8345cbcf 100644
--- a/package/nodejs/nodejs.mk
+++ b/package/nodejs/nodejs.mk
@@ -5,7 +5,7 @@
################################################################################
# _VERSION, _SOURCE and _SITE must be kept empty to avoid downloading anything
-NODEJS_COMMON_VERSION = 16.20.0
+NODEJS_COMMON_VERSION = 20.9.0
NODEJS_COMMON_SOURCE = node-v$(NODEJS_COMMON_VERSION).tar.xz
NODEJS_COMMON_SITE = http://nodejs.org/dist/v$(NODEJS_COMMON_VERSION)
diff --git a/support/testing/tests/package/test_nodejs.py b/support/testing/tests/package/test_nodejs.py
index b63d44c97e..2884529c6e 100644
--- a/support/testing/tests/package/test_nodejs.py
+++ b/support/testing/tests/package/test_nodejs.py
@@ -10,6 +10,7 @@ class TestNodeJSBasic(infra.basetest.BRTest):
BR2_cortex_a9=y
BR2_ARM_ENABLE_VFP=y
BR2_TOOLCHAIN_EXTERNAL=y
+ BR2_TOOLCHAIN_EXTERNAL_BOOTLIN=y
BR2_PACKAGE_NODEJS=y
BR2_TARGET_ROOTFS_CPIO=y
# BR2_TARGET_ROOTFS_TAR is not set
@@ -34,6 +35,7 @@ class TestNodeJSModuleHostBin(infra.basetest.BRTest):
BR2_cortex_a9=y
BR2_ARM_ENABLE_VFP=y
BR2_TOOLCHAIN_EXTERNAL=y
+ BR2_TOOLCHAIN_EXTERNAL_BOOTLIN=y
BR2_PACKAGE_NODEJS=y
BR2_PACKAGE_NODEJS_MODULES_ADDITIONAL="lodash"
BR2_PACKAGE_HOST_NODEJS_BIN=y
@@ -60,6 +62,7 @@ class TestNodeJSModuleHostSrc(infra.basetest.BRTest):
BR2_cortex_a9=y
BR2_ARM_ENABLE_VFP=y
BR2_TOOLCHAIN_EXTERNAL=y
+ BR2_TOOLCHAIN_EXTERNAL_BOOTLIN=y
BR2_PACKAGE_NODEJS=y
BR2_PACKAGE_NODEJS_MODULES_ADDITIONAL="lodash"
BR2_PACKAGE_HOST_NODEJS_SRC=y
--
2.41.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH] Config.in: Introduce BR2_HOST_GCC_AT_LEAST_11
2023-11-03 0:06 [Buildroot] [PATCH] Config.in: Introduce BR2_HOST_GCC_AT_LEAST_11 Adam Duskett
2023-11-03 0:06 ` [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0 Adam Duskett
@ 2023-11-03 20:16 ` Thomas Petazzoni via buildroot
1 sibling, 0 replies; 7+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-11-03 20:16 UTC (permalink / raw)
To: Adam Duskett; +Cc: Daniel Price, Martin Bark, buildroot
On Thu, 2 Nov 2023 18:06:49 -0600
Adam Duskett <adam.duskett@amarulasolutions.com> wrote:
> NodeJS 20 requires gcc >= 10.1. Unfortuantly, the only host gcc that has a
> decimal place is 4.9, and every other HOST_GCC_AT_LEAST is an even release,
> we round up to 11.
>
> Signed-off-by: Adam Duskett <adam.duskett@amarulasolutions.com>
> ---
> Config.in | 10 ++++++++++
> Makefile | 2 +-
> 2 files changed, 11 insertions(+), 1 deletion(-)
Applied to master with an improved commit title and commit log. Thanks!
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0
2023-11-03 0:06 ` [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0 Adam Duskett
@ 2023-11-03 20:16 ` Thomas Petazzoni via buildroot
2023-11-11 16:31 ` Romain Naour
2023-11-04 16:26 ` Arnout Vandecappelle via buildroot
1 sibling, 1 reply; 7+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-11-03 20:16 UTC (permalink / raw)
To: Adam Duskett; +Cc: Daniel Price, Martin Bark, buildroot
On Thu, 2 Nov 2023 18:06:50 -0600
Adam Duskett <adam.duskett@amarulasolutions.com> wrote:
> - Refactor 0001-add-qemu-wrapper-support.patch for 20.9.0
>
> - NodeJS now requires GCC 10.1 header. However, as there is no
> BR2_TOOLCHAIN_GCC_AT_LEAST_10_1, we round up to BR2_TOOLCHAIN_GCC_AT_LEAST_11.
>
> - Drop the --without-dtrace and --without-etw config options as they no longer
> exist.
>
> - Update support/testing/tests/package/test_nodejs.py to use
> BR2_TOOLCHAIN_EXTERNAL_BOOTLIN, otherwise the test fails as the
> gcc version selected will be 7.
>
> - Update the hash file for LICENSE.txt due to numerous changes, such
> as the addition of new libraries and updated dates. No new license
> types are added.
>
> Tested with: ./support/testing/run-tests tests.package.test_nodejs
>
> Signed-off-by: Adam Duskett <adam.duskett@amarulasolutions.com>
> ---
> package/nodejs/Config.in | 8 +--
> .../0001-add-qemu-wrapper-support.patch | 51 ++++++++++++-------
> ...ules-cjs-loader.js-adjust-default-pa.patch | 2 +-
> package/nodejs/nodejs-src/nodejs-src.mk | 4 --
> package/nodejs/nodejs.hash | 15 +++---
> package/nodejs/nodejs.mk | 2 +-
> support/testing/tests/package/test_nodejs.py | 3 ++
> 7 files changed, 50 insertions(+), 35 deletions(-)
Applied to master, thanks.
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0
2023-11-03 0:06 ` [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0 Adam Duskett
2023-11-03 20:16 ` Thomas Petazzoni via buildroot
@ 2023-11-04 16:26 ` Arnout Vandecappelle via buildroot
2023-11-07 17:02 ` Yann E. MORIN
1 sibling, 1 reply; 7+ messages in thread
From: Arnout Vandecappelle via buildroot @ 2023-11-04 16:26 UTC (permalink / raw)
To: Adam Duskett, buildroot; +Cc: Martin Bark, Thomas Petazzoni, Daniel Price
On 03/11/2023 01:06, Adam Duskett wrote:
> - Refactor 0001-add-qemu-wrapper-support.patch for 20.9.0
>
> - NodeJS now requires GCC 10.1 header. However, as there is no
> BR2_TOOLCHAIN_GCC_AT_LEAST_10_1, we round up to BR2_TOOLCHAIN_GCC_AT_LEAST_11.
GCC 10.1 is the first GCC release [1] (they start counting at 1, weird
people). So I think it really should be _AT_LEAST_10.
Regards,
Arnout
[1] https://gcc.gnu.org/gcc-10/
>
> - Drop the --without-dtrace and --without-etw config options as they no longer
> exist.
>
> - Update support/testing/tests/package/test_nodejs.py to use
> BR2_TOOLCHAIN_EXTERNAL_BOOTLIN, otherwise the test fails as the
> gcc version selected will be 7.
>
> - Update the hash file for LICENSE.txt due to numerous changes, such
> as the addition of new libraries and updated dates. No new license
> types are added.
>
> Tested with: ./support/testing/run-tests tests.package.test_nodejs
>
> Signed-off-by: Adam Duskett <adam.duskett@amarulasolutions.com>
[snip]
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0
2023-11-04 16:26 ` Arnout Vandecappelle via buildroot
@ 2023-11-07 17:02 ` Yann E. MORIN
0 siblings, 0 replies; 7+ messages in thread
From: Yann E. MORIN @ 2023-11-07 17:02 UTC (permalink / raw)
To: Arnout Vandecappelle
Cc: Adam Duskett, Daniel Price, Martin Bark, Thomas Petazzoni,
buildroot
Adam, All,
On 2023-11-04 17:26 +0100, Arnout Vandecappelle via buildroot spake thusly:
> On 03/11/2023 01:06, Adam Duskett wrote:
> >- NodeJS now requires GCC 10.1 header. However, as there is no
> > BR2_TOOLCHAIN_GCC_AT_LEAST_10_1, we round up to BR2_TOOLCHAIN_GCC_AT_LEAST_11.
> GCC 10.1 is the first GCC release [1] (they start counting at 1, weird
> people). So I think it really should be _AT_LEAST_10.
Adam, will you try and downgrade to _AT_LEAST_10, as Arnout suggested?
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0
2023-11-03 20:16 ` Thomas Petazzoni via buildroot
@ 2023-11-11 16:31 ` Romain Naour
0 siblings, 0 replies; 7+ messages in thread
From: Romain Naour @ 2023-11-11 16:31 UTC (permalink / raw)
To: Thomas Petazzoni, Adam Duskett; +Cc: buildroot, Daniel Price, Martin Bark
Hello,
Le 03/11/2023 à 21:16, Thomas Petazzoni via buildroot a écrit :
> On Thu, 2 Nov 2023 18:06:50 -0600
> Adam Duskett <adam.duskett@amarulasolutions.com> wrote:
>
>> - Refactor 0001-add-qemu-wrapper-support.patch for 20.9.0
>>
>> - NodeJS now requires GCC 10.1 header. However, as there is no
>> BR2_TOOLCHAIN_GCC_AT_LEAST_10_1, we round up to BR2_TOOLCHAIN_GCC_AT_LEAST_11.
>>
>> - Drop the --without-dtrace and --without-etw config options as they no longer
>> exist.
>>
>> - Update support/testing/tests/package/test_nodejs.py to use
>> BR2_TOOLCHAIN_EXTERNAL_BOOTLIN, otherwise the test fails as the
>> gcc version selected will be 7.
>>
>> - Update the hash file for LICENSE.txt due to numerous changes, such
>> as the addition of new libraries and updated dates. No new license
>> types are added.
>>
>> Tested with: ./support/testing/run-tests tests.package.test_nodejs
>>
>> Signed-off-by: Adam Duskett <adam.duskett@amarulasolutions.com>
>> ---
>> package/nodejs/Config.in | 8 +--
>> .../0001-add-qemu-wrapper-support.patch | 51 ++++++++++++-------
>> ...ules-cjs-loader.js-adjust-default-pa.patch | 2 +-
>> package/nodejs/nodejs-src/nodejs-src.mk | 4 --
>> package/nodejs/nodejs.hash | 15 +++---
>> package/nodejs/nodejs.mk | 2 +-
>> support/testing/tests/package/test_nodejs.py | 3 ++
>> 7 files changed, 50 insertions(+), 35 deletions(-)
>
> Applied to master, thanks.
The TestNodeJSBasic fail in gitlab since host-gcc is 10.2.1
$ gcc --version
gcc (Debian 10.2.1-6) 10.2.1 20210110
https://gitlab.com/buildroot.org/buildroot/-/jobs/5464930066
It seems we should really try to downgrade to _AT_LEAST_10 as suggested by
Arnout and Yann.
Also the package/nodejs/Config.in.host doesn't seems updated by this version
bump (I can still select BR2_PACKAGE_HOST_NODEJS).
Best regards,
Romain
>
> Thomas
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-11-11 16:31 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-03 0:06 [Buildroot] [PATCH] Config.in: Introduce BR2_HOST_GCC_AT_LEAST_11 Adam Duskett
2023-11-03 0:06 ` [Buildroot] [PATCH] package/nodejs: bump version to 20.9.0 Adam Duskett
2023-11-03 20:16 ` Thomas Petazzoni via buildroot
2023-11-11 16:31 ` Romain Naour
2023-11-04 16:26 ` Arnout Vandecappelle via buildroot
2023-11-07 17:02 ` Yann E. MORIN
2023-11-03 20:16 ` [Buildroot] [PATCH] Config.in: Introduce BR2_HOST_GCC_AT_LEAST_11 Thomas Petazzoni via buildroot
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.