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
community/nodejs-current: upgrade to 11.1.0 #4116
Conversation
dont-run-gyp-files-for-bundled-deps.patch needs to be updated/checked it is causing the CI failure. |
Fixed patch:
|
Node.js 10.0.0 needs a LibreSSL patch:
|
@ncopa, please patch it for LibreSSL. |
10.1 is out https://github.com/nodejs/node/blob/master/doc/changelogs/CHANGELOG_V10.md#10.1.0 |
@ncopa reminder |
Hi gyus. Any way to how help you with that? I would like to install node 10. :-) |
@grimd34th The hold up is related to fact that a patch is needed for Node to compile with LibreSSL 2.7. |
community/nodejs-current/APKBUILD
Outdated
@@ -12,7 +12,7 @@ | |||
# | |||
pkgname=nodejs-current | |||
# The current stable version, i.e. non-LTS. | |||
pkgver=9.11.1 | |||
pkgver=10.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Current is 10.7.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
current is 10.10.0 now
pkgdesc="JavaScript runtime built on V8 engine - current stable version" | ||
url="https://nodejs.org/" | ||
arch="all" | ||
license="MIT" | ||
depends="ca-certificates" | ||
depends_dev="libuv" | ||
# gold is needed for mksnapshot | ||
makedepends="$depends_dev python2 openssl1.0-dev zlib-dev libuv-dev linux-headers | ||
makedepends="$depends_dev python2 openssl1.1-dev zlib-dev libuv-dev linux-headers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
openssl1.1-dev
is in testing repo and could not be used in community
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as in #5558 the correct package is openssl-dev
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
openssl1.1-dev
has been renamed to openssl-dev
and moved to main.
nodejs-current is 11.1.0 now and 10.13.0 is LTS |
Filed upgrade for LTS #5558 |
pkgdesc="JavaScript runtime built on V8 engine - current stable version" | ||
url="https://nodejs.org/" | ||
arch="all" | ||
license="MIT" | ||
depends="ca-certificates" | ||
depends_dev="libuv" | ||
# gold is needed for mksnapshot | ||
makedepends="$depends_dev python2 openssl1.0-dev zlib-dev libuv-dev linux-headers | ||
makedepends="$depends_dev python2 openssl1.1-dev zlib-dev libuv-dev linux-headers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as in #5558 the correct package is openssl-dev
pkgrel=4 | ||
|
||
pkgver=10.10.0 | ||
pkgrel=1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Node.js v11.1.0 is already out, can you update this PR? AFAIK pgrel should be set to 0
Modified 2016-11-26 by Jakub Jirutka <jakub@jirutka.cz> to update for | ||
Node.js 7.2.0 | ||
Modified 2016-11-26 by Jakub Jirutka <jakub@jirutka.cz> to update for Node.js 7.2.0 | ||
Modified 2018-04-27 to Node.js 10.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be changed to 11.1.0
@@ -1,27 +1,20 @@ | |||
# Contributor: Jakub Jirutka <jakub@jirutka.cz> | |||
# Maintainer: Jose-Luis Rivas <ghostbar@riseup.net> | |||
# | |||
# secfixes: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jirutka said that secfixes should remain for history purposes
this is fixed with d103c6e |
No description provided.