2016-08-27 22:38 GMT+02:00 Alex Kost <alez...@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
