Re: PR/51172 CVS commit: pkgsrc/lang/nodejs

[Filip Hajny <filip%joyent.com@localhost>]

The following reply was made to PR pkg/51172; it has been noted by GNATS.

From: Filip Hajny <filip%joyent.com@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: PR/51172 CVS commit: pkgsrc/lang/nodejs
Date: Tue, 31 May 2016 16:49:50 +0200

 Right now the package lets node.js build (and use) its own embedded =
 ('small-icu') icu binary, which is what is failing to link in OpenSSL =
 properly (though it works fine on SmartOS, not sure why). This is what =
 Tobias worked around yesterday. Enabling the =E2=80=98icu' option makes =
 node.js use our (full) icu, and that works fine.
 
 My suggestion is to simply remove the icu option and make node.js use =
 our (system) icu always. The embedded icu approach is fuzzy, and =
 building with icu completely disabled (--without-intl) makes the build =
 fail on both NetBSD and SmartOS with a strange set of errors that =
 suggest such configuration is not tested or supported at all.
 
 Unless there are objections, I=E2=80=99ll commit this later tonight.
 
 -F=