Re: [PATCH 3/4] gnu: node: Do not use bundled dependencies.

[Jelle Licht <jlicht@fsfe.org>]

[-- Attachment #1: Type: text/plain, Size: 2271 bytes --]

2016-08-27 22:38 GMT+02:00 Alex Kost <alezost@gmail.com>:

> Jelle Licht (2016-08-27 14:23 +0300) wrote:
>
> > The Node build system was previously building its own copies of
> > C-ares and http-parser.
> >
> > * gnu/packages/node.scm (node)[inputs]: Add c-ares and http-parser.
> > [arguments]: Add configure flags for using system libraries.
> > ---
> >  gnu/packages/node.scm | 7 ++++++-
> >  1 file changed, 6 insertions(+), 1 deletion(-)
> >
> >
> > diff --git a/gnu/packages/node.scm b/gnu/packages/node.scm
> > index d1c5e1b..7c020e6 100644
> > --- a/gnu/packages/node.scm
> > +++ b/gnu/packages/node.scm
> > @@ -25,6 +25,7 @@
> >    #:use-module (guix derivations)
> >    #:use-module (guix download)
> >    #:use-module (guix build-system gnu)
> > +  #:use-module (gnu packages adns)
> >    #:use-module (gnu packages base)
> >    #:use-module (gnu packages compression)
> >    #:use-module (gnu packages gcc)
> > @@ -86,6 +87,8 @@ it does not buffer data, it can be interrupted at
> anytime.")
> >       '(#:configure-flags '("--shared-openssl"
> >                             "--shared-zlib"
> >                             "--shared-libuv"
> > +                           "--shared-cares"
> > +                           "--shared-http-parser"
> >                             "--without-snapshot")
> >         #:phases
> >         (modify-phases %standard-phases
> > @@ -158,7 +161,9 @@ it does not buffer data, it can be interrupted at
> anytime.")
> >      (inputs
> >       `(("libuv" ,libuv)
> >         ("openssl" ,tls:openssl)
> > -       ("zlib" ,zlib)))
> > +       ("zlib" ,compression:zlib)
>
> This change shouldn't belong this patch: here you use 'compression'
> prefix which is introduced by the next patch.  This would leave the git
> repo in a broken state on this commit.
>
> > +       ("http-parser" ,http-parser)
> > +       ("c-ares" ,c-ares)))
> >      (synopsis "Evented I/O for V8 JavaScript")
> >      (description "Node.js is a platform built on Chrome's JavaScript
> runtime
> >  for easily building fast, scalable network applications.  Node.js uses
> an
>
> --
> Alex
>

I probably put the wrong things together when rebasing. Should I provide
updated
patches in these threads, or should I just send in a new patch series?

- Jelle

[-- Attachment #2: Type: text/html, Size: 3372 bytes --]