all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#45862] [PATCH 0/1] Fix libnode and r-v8
@ 2021-01-14 14:30 zimoun
  2021-01-14 14:34 ` [bug#45862] [PATCH 1/1] gnu: libnode: Fix regression in unused phases zimoun
  2021-01-20 21:35 ` bug#45862: [PATCH 0/1] Fix libnode and r-v8 Ricardo Wurmus
  0 siblings, 2 replies; 3+ messages in thread
From: zimoun @ 2021-01-14 14:30 UTC (permalink / raw)
  To: 45862

Hi,

Commit 1d77141f6d introduces a regression about libnode.  See:

--8<---------------cut here---------------start------------->8---
$ for ci in $(git -C ~/src/guix/guix log --format="%H" 1d77141f6d^^..1d77141f6d) ;\
  do echo "# Commit: $ci" ;\
     guix time-machine --commit=$ci -- build libnode --no-grafts -v 0 ;\
  done

# Commit: 1d77141f6d5004cf10d1f9ead53855f25a973847
Mise à jour du canal « guix » depuis le dépôt Git « https://git.savannah.gnu.org/git/guix.git »...
guix build: error: build of `/gnu/store/yzpk9vr4ipa58vhfqcb66jpwjs808y3n-libnode-10.20.0.drv' failed

# Commit: ac80cbfab69b59d17fcf38ef9f4ce0acc0e5f9e6
Mise à jour du canal « guix » depuis le dépôt Git « https://git.savannah.gnu.org/git/guix.git »...
/gnu/store/7m6nkxcaky1423078mnxa9zkrxyz2c30-libnode-10.20.0
--8<---------------cut here---------------end--------------->8---

The patch fixes it.  The test is done against r-v8 which uses libnode.
Everything builds fine now.


All the best,
simon

zimoun (1):
  gnu: libnode: Fix regression in unused phases.

 gnu/packages/node.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)


base-commit: 88eb5370ec155902ed5fe7636e91e932079b338a
-- 
2.29.2





^ permalink raw reply	[flat|nested] 3+ messages in thread

* [bug#45862] [PATCH 1/1] gnu: libnode: Fix regression in unused phases.
  2021-01-14 14:30 [bug#45862] [PATCH 0/1] Fix libnode and r-v8 zimoun
@ 2021-01-14 14:34 ` zimoun
  2021-01-20 21:35 ` bug#45862: [PATCH 0/1] Fix libnode and r-v8 Ricardo Wurmus
  1 sibling, 0 replies; 3+ messages in thread
From: zimoun @ 2021-01-14 14:34 UTC (permalink / raw)
  To: 45862

Regression introduced in 1d77141f6d5004cf10d1f9ead53855f25a973847.
Reported by Pierre-Antoine Bouttier
<pierre-antoine.bouttier@univ-grenoble-alpes.fr>.

* gnu/packages/node.scm (libnode)[arguments]: Delete phase
"patch-node-shebang".
---
 gnu/packages/node.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/node.scm b/gnu/packages/node.scm
index 9e07cd906e..a495f6d763 100644
--- a/gnu/packages/node.scm
+++ b/gnu/packages/node.scm
@@ -8,6 +8,7 @@
 ;;; Copyright © 2018, 2019, 2020 Marius Bakke <mbakke@fastmail.com>
 ;;; Copyright © 2020 Pierre Langlois <pierre.langlois@gmx.com>
 ;;; Copyright © 2020 Ricardo Wurmus <rekado@elephly.net>
+;;; Copyright © 2021 Simon Tournier <zimon.toutoune@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -238,4 +239,5 @@ devices.")
         `(cons* "--shared" "--without-npm" ,flags))
        ((#:phases phases '%standard-phases)
         `(modify-phases ,phases
-           (delete 'patch-npm-shebang)))))))
+           (delete 'patch-npm-shebang)
+           (delete 'patch-node-shebang)))))))
-- 
2.29.2





^ permalink raw reply related	[flat|nested] 3+ messages in thread

* bug#45862: [PATCH 0/1] Fix libnode and r-v8
  2021-01-14 14:30 [bug#45862] [PATCH 0/1] Fix libnode and r-v8 zimoun
  2021-01-14 14:34 ` [bug#45862] [PATCH 1/1] gnu: libnode: Fix regression in unused phases zimoun
@ 2021-01-20 21:35 ` Ricardo Wurmus
  1 sibling, 0 replies; 3+ messages in thread
From: Ricardo Wurmus @ 2021-01-20 21:35 UTC (permalink / raw)
  To: zimoun; +Cc: 45862-done

Hi,

> Commit 1d77141f6d introduces a regression about libnode.  See: […]

Sorry about that!

> The patch fixes it.  The test is done against r-v8 which uses libnode.
> Everything builds fine now.

Indeed!  Applied and pushed with commit 41134f915f.

Thank you!

-- 
Ricardo




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-01-20 21:36 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-14 14:30 [bug#45862] [PATCH 0/1] Fix libnode and r-v8 zimoun
2021-01-14 14:34 ` [bug#45862] [PATCH 1/1] gnu: libnode: Fix regression in unused phases zimoun
2021-01-20 21:35 ` bug#45862: [PATCH 0/1] Fix libnode and r-v8 Ricardo Wurmus

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.