mbox series

[bug#48396,0/4] Add nodejs 16.1.0

Message ID 20210513130211.4057-1-mjbecze@riseup.net
Headers show
Series Add nodejs 16.1.0 | expand

Message

Martin Becze May 13, 2021, 1:02 p.m. UTC 
This patch set add nodejs 16.1.0 as well as updates some dependencies of node lts which where need for 16.1.0 to build.


Martin Becze (4):
  gnu: Add llhttp-6.0.
  gnu: node-lts: Updated c-ares and icu4c.
  gnu: node-lts: Add phase 'delete-cluster-tests.
  gnu: Added node-16.1.

 gnu/packages/node.scm | 57 ++++++++++++++++++++++++++++++++++++-------
 1 file changed, 48 insertions(+), 9 deletions(-)

Comments

M Feb. 25, 2022, 10:54 a.m. UTC | #1 
> +(define llhttp-6.0
> +  (package
> +    (inherit llhttp-bootstrap)
> +    (version "6.0.0")

Instead of defining a package variant of llhttp-bootstrap, why not just
update 'llhtp-bootstrap', to keep things simple by just having the
latest version of llhttp in Guix?

> +(define-public node-16.1
> +  (package
> +    (inherit node-lts)
> +    (version "16.1.0")

Likewise.

(Also, how come that the LTS version in Guix (14.18.3) is newer than
the regular version '10.24.1'?)

Greetings,
Maxime.