Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nodejs: update to 14.7.0 #23433

Closed
wants to merge 1 commit into from
Closed

Conversation

Noah-Huppert
Copy link
Contributor

@Noah-Huppert Noah-Huppert commented Jul 7, 2020

Works on my machine (x86_64):

% ./xbps-src pkg nodejs
% sudo xbps-install --repository "$PWD/hostdir/binpkgs/node14/" -Syu nodejs
% node -v
v14.5.0

I removed the patches b/c I assume in this version they aren't needed? Someone familiar with the package should confirm this.

@ericonr
Copy link
Member

ericonr commented Jul 7, 2020

@q66 is it still broken for ppc?

@Noah-Huppert
Copy link
Contributor Author

Power PC?

@Noah-Huppert
Copy link
Contributor Author

CI build timed out. It took a long time to build on my laptop as well.

@fosslinux
Copy link
Contributor

Add [ci skip] to the commit message if you push again.

@Noah-Huppert
Copy link
Contributor Author

Pushed with CI skip.

@bn4t
Copy link
Contributor

bn4t commented Aug 9, 2020

Would be great to have this merged. Currently can't use tailwindcss because of a bug in the version that's currently available in the repo.

@Noah-Huppert
Copy link
Contributor Author

Bumping to 14.7.0 rn.

@Noah-Huppert Noah-Huppert changed the title nodejs: update to 14.5.0 nodejs: update to 14.7.0 Aug 10, 2020
@Noah-Huppert
Copy link
Contributor Author

Now PR is for nodejs 14.7.0. Build took ~1:30 hours so skipping the CI.

@q5sys
Copy link
Contributor

q5sys commented Sep 9, 2020

This would be a really helpful update as some versions in the Node 13.x range have buggy ES module support or need --experimental-modules at the command line.

@Noah-Huppert
Copy link
Contributor Author

Not really sure why the PR hasn't been merged...

@q5sys
Copy link
Contributor

q5sys commented Sep 9, 2020 via email

@Noah-Huppert
Copy link
Contributor Author

bumping this PR to 4.10.0 now.

@Johnnynator Johnnynator self-assigned this Sep 9, 2020
@Johnnynator
Copy link
Member

I will look into this PR this weekend.

@Noah-Huppert
Copy link
Contributor Author

Noah-Huppert commented Sep 9, 2020

bumping this PR to 4.10.0 now.

Having issues, I wonder if 4.10.0 requires new dependencies.

  g++ -o /builddir/node-v14.10.0/out/Release/obj.target/libnode/src/node.o ../src/node.cc '-DV8_DEPRECATION_WARNINGS' '-DV8_IMMINENT_DEPRECATION_WARNINGS' '-D__STDC_FORMAT_MACROS' '-DOPENSSL_NO_PINSHARED' '-DOPENSSL_THREADS' '-DNODE_ARCH="x64"' '-DNODE_PLATFORM="linux"' '-DNODE_WANT_INTERNALS=1' '-DV8_DEPRECATION_WARNINGS=1' '-DNODE_OPENSSL_SYSTEM_CERT_PATH=""' '-DHAVE_INSPECTOR=1' '-DNODE_ENABLE_LARGE_CODE_PAGES=1' '-D__POSIX__' '-DNODE_USE_V8_PLATFORM=1' '-DNODE_HAVE_I18N_SUPPORT=1' '-DHAVE_OPENSSL=1' -I../src -I/builddir/node-v14.10.0/out/Release/obj/gen -I/builddir/node-v14.10.0/out/Release/obj/gen/include -I/builddir/node-v14.10.0/out/Release/obj/gen/src -I../deps/histogram/src -I../deps/uvwasi/include -I../deps/v8/include -I../deps/brotli/c/include -I../deps/openssl/openssl/include  -Wall -Wextra -Wno-unused-parameter -pthread -Wall -Wextra -Wno-unused-parameter -m64 -O3 -fno-omit-frame-pointer -fno-rtti -fno-exceptions -std=gnu++1y -MMD -MF /builddir/node-v14.10.0/out/Release/.deps//builddir/node-v14.10.0/out/Release/obj.target/libnode/src/node.o.d.raw     -fstack-clash-protection -D_FORTIFY_SOURCE=2 -mtune=generic -O2 -pipe  -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64  -c
../src/node.cc: In function 'int node::Start(int, char**)':
../src/node.cc:1092:42: error: 'UV_METRICS_IDLE_TIME' was not declared in this scope
 1092 |     uv_loop_configure(uv_default_loop(), UV_METRICS_IDLE_TIME);
      |                                          ^~~~~~~~~~~~~~~~~~~~
make[1]: *** [libnode.target.mk:313: /builddir/node-v14.10.0/out/Release/obj.target/libnode/src/node.o] Error 1
rm 80455764c1d44a2c5307476ee8e59d862d62cc4f.intermediate
make: *** [Makefile:104: node] Error 2
=> ERROR: nodejs-14.10.0_1: do_build: 'make LD="$CXX" LDFLAGS+=-ldl ${makejobs} V=1' exited with 2
=> ERROR:   in do_build() at srcpkgs/nodejs/template:101

@Johnnynator
Copy link
Member

bumping this PR to 4.10.0 now.

Having issues, I wonder if 4.10.0 requires new dependencies.

  g++ -o /builddir/node-v14.10.0/out/Release/obj.target/libnode/src/node.o ../src/node.cc '-DV8_DEPRECATION_WARNINGS' '-DV8_IMMINENT_DEPRECATION_WARNINGS' '-D__STDC_FORMAT_MACROS' '-DOPENSSL_NO_PINSHARED' '-DOPENSSL_THREADS' '-DNODE_ARCH="x64"' '-DNODE_PLATFORM="linux"' '-DNODE_WANT_INTERNALS=1' '-DV8_DEPRECATION_WARNINGS=1' '-DNODE_OPENSSL_SYSTEM_CERT_PATH=""' '-DHAVE_INSPECTOR=1' '-DNODE_ENABLE_LARGE_CODE_PAGES=1' '-D__POSIX__' '-DNODE_USE_V8_PLATFORM=1' '-DNODE_HAVE_I18N_SUPPORT=1' '-DHAVE_OPENSSL=1' -I../src -I/builddir/node-v14.10.0/out/Release/obj/gen -I/builddir/node-v14.10.0/out/Release/obj/gen/include -I/builddir/node-v14.10.0/out/Release/obj/gen/src -I../deps/histogram/src -I../deps/uvwasi/include -I../deps/v8/include -I../deps/brotli/c/include -I../deps/openssl/openssl/include  -Wall -Wextra -Wno-unused-parameter -pthread -Wall -Wextra -Wno-unused-parameter -m64 -O3 -fno-omit-frame-pointer -fno-rtti -fno-exceptions -std=gnu++1y -MMD -MF /builddir/node-v14.10.0/out/Release/.deps//builddir/node-v14.10.0/out/Release/obj.target/libnode/src/node.o.d.raw     -fstack-clash-protection -D_FORTIFY_SOURCE=2 -mtune=generic -O2 -pipe  -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64  -c
../src/node.cc: In function 'int node::Start(int, char**)':
../src/node.cc:1092:42: error: 'UV_METRICS_IDLE_TIME' was not declared in this scope
 1092 |     uv_loop_configure(uv_default_loop(), UV_METRICS_IDLE_TIME);
      |                                          ^~~~~~~~~~~~~~~~~~~~
make[1]: *** [libnode.target.mk:313: /builddir/node-v14.10.0/out/Release/obj.target/libnode/src/node.o] Error 1
rm 80455764c1d44a2c5307476ee8e59d862d62cc4f.intermediate
make: *** [Makefile:104: node] Error 2
=> ERROR: nodejs-14.10.0_1: do_build: 'make LD="$CXX" LDFLAGS+=-ldl ${makejobs} V=1' exited with 2
=> ERROR:   in do_build() at srcpkgs/nodejs/template:101

https://nodejs.org/en/blog/release/v14.9.0/

^ they upgraded to libuv 1.39. So the libuv package in Void also needs updating.

@Johnnynator
Copy link
Member

Nodejs 14 (12.18.3 also) do have a broken --shared-libuv config flag, they do link against out/Release/obj.target/deps/uv/libuv.a first, and to -luv only later. Which also should be fixed.

@leahneukirchen
Copy link
Member

f88309c

@Noah-Huppert
Copy link
Contributor Author

@leahneukirchen THANK YOU! Appreciate you very much 😁

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants