unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: jlicht@fsfe.org
To: 74187@debbugs.gnu.org
Cc: "Jelle Licht" <jlicht@fsfe.org>,
	jlicht@fsfe.org, "Ricardo Wurmus" <rekado@elephly.net>,
	"Ludovic Courtès" <ludo@gnu.org>
Subject: [bug#74187] [PATCH v2 01/24] gnu: node: Replace customized nghttp2-for-node by nghttp2.
Date: Wed,  4 Dec 2024 08:49:23 +0100	[thread overview]
Message-ID: <3114fe5d3e5314a0ded1f259a21d1016f4234abc.1733298337.git.jlicht@fsfe.org> (raw)
In-Reply-To: <cover.1733298337.git.jlicht@fsfe.org>

From: Jelle Licht <jlicht@fsfe.org>

* gnu/packages/node.scm (node) [native-inputs]: Replace nghttp2-for-node by
nghttp2.
[inputs]: Ditto.
[arguments]<#:phases>: Adjust accordingly by skipping failing tests.

Change-Id: Ia3d63ea1c428c1353d6ec6fda394ccb016eb6603
---
 gnu/packages/node.scm | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/node.scm b/gnu/packages/node.scm
index 7c320a00d6..6414b94278 100644
--- a/gnu/packages/node.scm
+++ b/gnu/packages/node.scm
@@ -48,6 +48,7 @@ (define-module (gnu packages node)
   #:use-module (gnu packages icu4c)
   #:use-module (gnu packages libevent)
   #:use-module (gnu packages linux)
+  #:use-module (gnu packages networking)
   #:use-module (gnu packages node-xyz)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pkg-config)
@@ -155,6 +156,11 @@ (define-public node
                          ;; See also <https://github.com/nodejs/node/issues/25903>.
                          "test/sequential/test-performance.js"))
 
+             ;; These tests fail on recent versions of nghttp2
+             (for-each delete-file
+                       '("test/parallel/test-http2-methods.js"
+                         "test/parallel/test-http2-multiplex.js"))
+
              ;; This requires a DNS resolver.
              (delete-file "test/parallel/test-dns.js")
 
@@ -297,7 +303,7 @@ (define-public node
            http-parser
            icu4c
            libuv-for-node
-           `(,nghttp2-for-node "lib")
+           `(,nghttp2 "lib")
            openssl-1.1
            zlib
            ;; Regular build-time dependencies.
@@ -317,7 +323,7 @@ (define-public node
            http-parser
            icu4c
            libuv-for-node
-           `(,nghttp2-for-node "lib")
+           `(,nghttp2 "lib")
            openssl
            python-wrapper               ;for node-gyp (supports python3)
            zlib))
-- 
2.46.0





  reply	other threads:[~2024-12-04  7:52 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-03 12:01 [bug#74187] [PATCH 00/20] Bumping node to 20.18.0 and beyond jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 01/20] gnu: node: Replace customized nghttp2-for-node by nghttp2 jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 02/20] gnu: Remove nghttp2-for-node jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 03/20] gnu: node-lts: Replace customized c-ares-for-node by c-ares jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 04/20] gnu: Remove c-ares-for-node jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 05/20] gnu: Add libuv-for-node-lts jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 06/20] gnu: llhttp-bootstrap: Update to 8.1.2 jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 07/20] gnu: node-lts: Update to 20.18.0 [security fixes] jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 08/20] gnu: r-v8: Fix build with node-lts@20 jlicht
2024-11-03 18:29   ` Ricardo Wurmus
2024-11-03 12:03 ` [bug#74187] [PATCH 09/20] gnu: cwltool: Use node-lts instead of node jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 10/20] gnu: python-cwl-utils: " jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 11/20] gnu: ocaml-ezjsonm: " jlicht
2024-11-03 12:03 ` [bug#74187] [PATCH 12/20] gnu: js-of-ocaml: " jlicht
2024-11-03 12:04 ` [bug#74187] [PATCH 13/20] gnu: fmp: " jlicht
2024-11-03 12:04 ` [bug#74187] [PATCH 14/20] gnu: python-cloudscraper: " jlicht
2024-11-03 12:04 ` [bug#74187] [PATCH 15/20] gnu: qtwebengine-5: " jlicht
2024-11-03 12:04 ` [bug#74187] [PATCH 16/20] gnu: ruby-autoprefixer-rails: " jlicht
2024-11-03 12:04 ` [bug#74187] [PATCH 17/20] gnu: ruby-execjs: " jlicht
2024-11-03 12:04 ` [bug#74187] [PATCH 18/20] gnu: vlang: " jlicht
2024-11-03 12:04 ` [bug#74187] [PATCH 19/20] gnu: esbuild-node: " jlicht
2024-11-03 12:04 ` [bug#74187] [PATCH 20/20] gnu: node: Rename variable node-bootstrap and hide package jlicht
2024-11-18 10:12 ` [bug#74187] [PATCH 00/20] Bumping node to 20.18.0 and beyond Ludovic Courtès
2024-12-04  7:49 ` [bug#74187] [PATCH v2 00/24] " jlicht
2024-12-04  7:49   ` jlicht [this message]
2024-12-04  7:49   ` [bug#74187] [PATCH v2 02/24] gnu: Remove nghttp2-for-node jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 03/24] gnu: node-lts: Replace customized c-ares-for-node by c-ares jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 04/24] gnu: Remove c-ares-for-node jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 05/24] gnu: Add libuv-for-node-lts jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 06/24] gnu: llhttp-bootstrap: Update to 8.1.2 jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 07/24] gnu: node-lts: Update to 20.18.1 [security fixes] jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 08/24] gnu: node-uglify-js: Update to 3.19.3 jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 09/24] gnu: node-acorn: Update to 8.7.1 jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 10/24] gnu: node-nan: Update to 2.22.0 jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 11/24] gnu: node-addon-api: Update to 8.3.0 jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 12/24] gnu: r-v8: Fix build with node-lts@20 jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 13/24] gnu: cwltool: Use node-lts instead of node jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 14/24] gnu: python-cwl-utils: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 15/24] gnu: ocaml-ezjsonm: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 16/24] gnu: js-of-ocaml: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 17/24] gnu: fmp: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 18/24] gnu: python-cloudscraper: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 19/24] gnu: qtwebengine-5: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 20/24] gnu: ruby-autoprefixer-rails: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 21/24] gnu: ruby-execjs: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 22/24] gnu: vlang: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 23/24] gnu: esbuild-node: " jlicht
2024-12-04  7:49   ` [bug#74187] [PATCH v2 24/24] gnu: node: Rename variable node-bootstrap and hide package jlicht

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3114fe5d3e5314a0ded1f259a21d1016f4234abc.1733298337.git.jlicht@fsfe.org \
    --to=jlicht@fsfe.org \
    --cc=74187@debbugs.gnu.org \
    --cc=ludo@gnu.org \
    --cc=rekado@elephly.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).