* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox