[Buildroot] [PATCH 1/3] package/nodejs: use shared c-ares if available

[Buildroot] [PATCH 1/3] package/nodejs: use shared c-ares if available

[Martin Bark]

Signed-off-by: Martin Bark <martin@barkynet.com>
---
 package/nodejs/nodejs.mk | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/package/nodejs/nodejs.mk b/package/nodejs/nodejs.mk
index 43c5ff3..a469153 100644
--- a/package/nodejs/nodejs.mk
+++ b/package/nodejs/nodejs.mk
@@ -34,6 +34,11 @@ else
 NODEJS_CONF_OPTS += --with-intl=none
 endif
 
+ifeq ($(BR2_PACKAGE_C_ARES),y)
+NODEJS_DEPENDENCIES += c-ares
+NODEJS_CONF_OPTS += --shared-cares
+endif
+
 ifneq ($(BR2_PACKAGE_NODEJS_NPM),y)
 NODEJS_CONF_OPTS += --without-npm
 endif
-- 
2.7.4

[Buildroot] [PATCH 2/3] package/nodejs: use shared libhttpparser if available

[Martin Bark]

Signed-off-by: Martin Bark <martin@barkynet.com>
---
 package/nodejs/nodejs.mk | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/package/nodejs/nodejs.mk b/package/nodejs/nodejs.mk
index a469153..ba7ad3f 100644
--- a/package/nodejs/nodejs.mk
+++ b/package/nodejs/nodejs.mk
@@ -39,6 +39,11 @@ NODEJS_DEPENDENCIES += c-ares
 NODEJS_CONF_OPTS += --shared-cares
 endif
 
+ifeq ($(BR2_PACKAGE_LIBHTTPPARSER),y)
+NODEJS_DEPENDENCIES += libhttpparser
+NODEJS_CONF_OPTS += --shared-http-parser
+endif
+
 ifneq ($(BR2_PACKAGE_NODEJS_NPM),y)
 NODEJS_CONF_OPTS += --without-npm
 endif
-- 
2.7.4

[Buildroot] [PATCH 3/3] package/nodejs: use shared libuv if available

[Martin Bark]

Signed-off-by: Martin Bark <martin@barkynet.com>
---
 package/nodejs/nodejs.mk | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/package/nodejs/nodejs.mk b/package/nodejs/nodejs.mk
index ba7ad3f..d229d75 100644
--- a/package/nodejs/nodejs.mk
+++ b/package/nodejs/nodejs.mk
@@ -44,6 +44,11 @@ NODEJS_DEPENDENCIES += libhttpparser
 NODEJS_CONF_OPTS += --shared-http-parser
 endif
 
+ifeq ($(BR2_PACKAGE_LIBUV),y)
+NODEJS_DEPENDENCIES += libuv
+NODEJS_CONF_OPTS += --shared-libuv
+endif
+
 ifneq ($(BR2_PACKAGE_NODEJS_NPM),y)
 NODEJS_CONF_OPTS += --without-npm
 endif
-- 
2.7.4

[Buildroot] [PATCH 1/3] package/nodejs: use shared c-ares if available

[Thomas Petazzoni]

Hello,

On Sat, 15 Jul 2017 18:38:27 +0100, Martin Bark wrote:

> diff --git a/package/nodejs/nodejs.mk b/package/nodejs/nodejs.mk
> index 43c5ff3..a469153 100644
> --- a/package/nodejs/nodejs.mk
> +++ b/package/nodejs/nodejs.mk
> @@ -34,6 +34,11 @@ else
>  NODEJS_CONF_OPTS += --with-intl=none
>  endif
>  
> +ifeq ($(BR2_PACKAGE_C_ARES),y)
> +NODEJS_DEPENDENCIES += c-ares
> +NODEJS_CONF_OPTS += --shared-cares
> +endif

So what happens if BR2_PACKAGE_C_ARES is disabled? It is not
needed/used by nodejs, or some internal/built-in version is used?

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[Buildroot] [PATCH 1/3] package/nodejs: use shared c-ares if available

[Martin Bark]

Thomas,

On 15 July 2017 at 20:44, Thomas Petazzoni <
thomas.petazzoni@free-electrons.com> wrote:

> Hello,
>
> On Sat, 15 Jul 2017 18:38:27 +0100, Martin Bark wrote:
>
> > diff --git a/package/nodejs/nodejs.mk b/package/nodejs/nodejs.mk
> > index 43c5ff3..a469153 100644
> > --- a/package/nodejs/nodejs.mk
> > +++ b/package/nodejs/nodejs.mk
> > @@ -34,6 +34,11 @@ else
> >  NODEJS_CONF_OPTS += --with-intl=none
> >  endif
> >
> > +ifeq ($(BR2_PACKAGE_C_ARES),y)
> > +NODEJS_DEPENDENCIES += c-ares
> > +NODEJS_CONF_OPTS += --shared-cares
> > +endif
>
> So what happens if BR2_PACKAGE_C_ARES is disabled? It is not
> needed/used by nodejs, or some internal/built-in version is used?
>

nodejs always uses c-ares, libuv and libhttpparser, it just uses it's own
versions by default (see https://github.com/nodejs/node/tree/master/deps).
Not setting any options will continue to use the build in versions.

Thanks

Martin


> Best regards,
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20170716/3b774bb9/attachment.html>

[Buildroot] [PATCH 1/3] package/nodejs: use shared c-ares if available

[Thomas Petazzoni]

Hello,

On Sun, 16 Jul 2017 00:12:33 +0100, Martin Bark wrote:

> nodejs always uses c-ares, libuv and libhttpparser, it just uses it's own
> versions by default (see https://github.com/nodejs/node/tree/master/deps).
> Not setting any options will continue to use the build in versions.

Then in this case, we want nodejs to unconditionally select c-ares,
libuv and libhttpparser. Indeed, like most distributions, we prefer to
use the external libraries rather than the built-in ones.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[Buildroot] [PATCH 1/3] package/nodejs: use shared c-ares if available

[Martin Bark]

Thomas,

On 16 July 2017 at 08:16, Thomas Petazzoni <
thomas.petazzoni@free-electrons.com> wrote:

> Hello,
>
> On Sun, 16 Jul 2017 00:12:33 +0100, Martin Bark wrote:
>
> > nodejs always uses c-ares, libuv and libhttpparser, it just uses it's own
> > versions by default (see https://github.com/nodejs/node/tree/master/deps
> ).
> > Not setting any options will continue to use the build in versions.
>
> Then in this case, we want nodejs to unconditionally select c-ares,
> libuv and libhttpparser. Indeed, like most distributions, we prefer to
> use the external libraries rather than the built-in ones.
>

Sounds good, i prefer that approach.  I'll resubmit the patches to
unconditionally select the libraries.

Thanks

Martin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20170717/1379f39f/attachment.html>