NetBSD Problem Report #51172

From bjjl@db.qwabbl.com  Fri May 27 10:20:11 2016
Return-Path: <bjjl@db.qwabbl.com>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(Client CN "mail.netbsd.org", Issuer "Postmaster NetBSD.org" (verified OK))
	by mollari.NetBSD.org (Postfix) with ESMTPS id E05717A46A
	for <gnats-bugs@gnats.NetBSD.org>; Fri, 27 May 2016 10:20:10 +0000 (UTC)
Message-Id: <20160527102002.E72632D9A1@db.qwabbl.com>
Date: Fri, 27 May 2016 12:20:02 +0200 (CEST)
From: netbsd@benjaminlorenz.email
To: gnats-bugs@NetBSD.org
Subject: lang/nodejs compile error
X-Send-Pr-Version: 3.95

>Number:         51172
>Category:       pkg
>Synopsis:       lang/nodejs compile error with netbsd/current
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    fhajny
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri May 27 10:25:00 +0000 2016
>Closed-Date:    Sat Nov 05 19:22:09 +0000 2016
>Last-Modified:  Sat Nov 05 19:22:09 +0000 2016
>Originator:     Benjamin Lorenz
>Release:        NetBSD 7.99.29
>Organization:

>Environment:


System: NetBSD db.qwabbl.com 7.99.29 NetBSD 7.99.29 (GENERIC) #0: Thu May 26 09:51:40 CEST 2016 root@db.qwabbl.com:/usr/obj/sys/arch/amd64/compile/GENERIC amd64
Architecture: x86_64
Machine: amd64
>Description:
icu_small.json: icutrim.py config: Trim down ICU to just a certain locale set, needed for node.js use.
Shared object "libssl.so.1.0.0" not found
FAILED: /usr/pkg/src/lang/nodejs/work/node-v6.2.0/out/Release/icupkg -tl ../../deps/icu-small/source/data/in/icudt57l.dat /usr/pkg/src/lang/nodejs/work/node-v6.2.0/out/Release/obj/gen/icutmp/icudt57l.dat
gmake[1]: *** [tools/icu/icudata.target.mk:13: /usr/pkg/src/lang/nodejs/work/node-v6.2.0/out/Release/obj/gen/icutmp/icudt57l.dat] Error 1
gmake[1]: Leaving directory '/usr/pkg/src/lang/nodejs/work/node-v6.2.0/out'
gmake: *** [Makefile:66: node] Error 2
*** Error code 2



>How-To-Repeat:

>Fix:


>Release-Note:

>Audit-Trail:

Responsible-Changed-From-To: pkg-manager->fhajny
Responsible-Changed-By: leot@NetBSD.org
Responsible-Changed-When: Sun, 29 May 2016 07:23:23 +0000
Responsible-Changed-Why:
Filip, can you please give a look to it? (probably it's related
to the same openssl issue pointed out by Joerg)


From: "Tobias Nygren" <tnn@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/51172 CVS commit: pkgsrc/lang/nodejs
Date: Mon, 30 May 2016 19:52:50 +0000

 Module Name:	pkgsrc
 Committed By:	tnn
 Date:		Mon May 30 19:52:49 UTC 2016

 Modified Files:
 	pkgsrc/lang/nodejs: options.mk

 Log Message:
 Apply band-aid for PR pkg/51172 openssl rpath problem.

 I tried and failed to reverse engineer the build framework to add
 the rpath in the right place.
 Give up for now and force it with WRAP_EXTRA_ARGS.CXX.
 At least it makes the package build again.


 To generate a diff of this commit:
 cvs rdiff -u -r1.8 -r1.9 pkgsrc/lang/nodejs/options.mk

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: Filip Hajny <filip@joyent.com>
To: gnats-bugs@NetBSD.org
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=

From: matthew sporleder <msporleder@gmail.com>
To: "gnats-bugs@NetBSD.org" <gnats-bugs@netbsd.org>
Cc: fhajny@netbsd.org, "gnats-admin@netbsd.org" <gnats-admin@netbsd.org>, 
	"pkgsrc-bugs@netbsd.org" <pkgsrc-bugs@netbsd.org>, netbsd@benjaminlorenz.email
Subject: Re: PR/51172 CVS commit: pkgsrc/lang/nodejs
Date: Wed, 1 Jun 2016 21:26:27 -0400

 I also need the following for netbsd-6

 patch-deps_v8_src_assembler.cc
 $NetBSD$

 --- deps/v8/src/assembler.cc-orig       2016-05-31 00:33:35.000000000 +0000
 +++ deps/v8/src/assembler.cc
 @@ -1169,7 +1169,7 @@ ExternalReference ExternalReference::f32
  }

  static void f32_nearest_int_wrapper(float* param) {
 -  *param = nearbyintf(*param);
 +  *param = rintf(*param);
  }

  ExternalReference ExternalReference::f32_nearest_int_wrapper_function(

From: "Filip Hajny" <fhajny@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/51172 CVS commit: pkgsrc/lang
Date: Thu, 2 Jun 2016 09:57:32 +0000

 Module Name:	pkgsrc
 Committed By:	fhajny
 Date:		Thu Jun  2 09:57:32 UTC 2016

 Modified Files:
 	pkgsrc/lang/nodejs: Makefile Makefile.common distinfo options.mk
 	pkgsrc/lang/nodejs/patches: patch-deps_v8_src_types.h
 	pkgsrc/lang/nodejs4: Makefile

 Log Message:
 Remove the nodejs icu option and make nodejs use a system ICU
 package by default. Expand existing patch to fix NetBSD 6 build.
 Fixes PR pkg/51172.

 Bump PKGREVISION for lang/nodejs and lang/nodejs4.


 To generate a diff of this commit:
 cvs rdiff -u -r1.68 -r1.69 pkgsrc/lang/nodejs/Makefile
 cvs rdiff -u -r1.7 -r1.8 pkgsrc/lang/nodejs/Makefile.common
 cvs rdiff -u -r1.67 -r1.68 pkgsrc/lang/nodejs/distinfo
 cvs rdiff -u -r1.9 -r1.10 pkgsrc/lang/nodejs/options.mk
 cvs rdiff -u -r1.2 -r1.3 pkgsrc/lang/nodejs/patches/patch-deps_v8_src_types.h
 cvs rdiff -u -r1.15 -r1.16 pkgsrc/lang/nodejs4/Makefile

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->feedback
State-Changed-By: fhajny@NetBSD.org
State-Changed-When: Tue, 20 Sep 2016 09:29:30 +0000
State-Changed-Why:
Should have been fixed by bl3'ing icu. Can you confirm?


From: Benjamin Lorenz <inbox@benjaminlorenz.email>
To: gnats-bugs@gnats.netbsd.org
Cc: 
Subject: Re: pkg/51172
Date: Sat, 5 Nov 2016 09:28:56 +0100

 --=-klsjoN/2RtjTr205KOPE
 Content-Type: text/plain; charset="utf-8"

 All good. You can close this ticket.


 --=-klsjoN/2RtjTr205KOPE
 Content-Type: text/html; charset="utf-8"

 <html><head></head><body>All good. You can close this ticket.<div><br></div></body></html>
 --=-klsjoN/2RtjTr205KOPE--

State-Changed-From-To: feedback->closed
State-Changed-By: bsiegert@NetBSD.org
State-Changed-When: Sat, 05 Nov 2016 19:22:09 +0000
State-Changed-Why:
Confirmed fixed


>Unformatted:

NetBSD Home
NetBSD PR Database Search

(Contact us) $NetBSD: query-full-pr,v 1.39 2013/11/01 18:47:49 spz Exp $
$NetBSD: gnats_config.sh,v 1.8 2006/05/07 09:23:38 tsutsui Exp $
Copyright © 1994-2014 The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.