Re: pkg/34613: devel/libgnome database/db4 Solaris

[john heasley <heas%shrubbery.net@localhost>]

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

From: john heasley <heas%shrubbery.net@localhost>
To: Juan RP <juan%xtrarom.org@localhost>
Cc: pkgsrc-bugs%NetBSD.org@localhost, Gilles Dauphin <dauphin%enst.fr@localhost>
Subject: Re: pkg/34613: devel/libgnome database/db4 Solaris
Date: Mon, 25 Sep 2006 22:19:09 -0700

 Mon, Sep 25, 2006 at 07:57:07PM +0200, Juan RP:
 > On Mon, 25 Sep 2006 17:55:00 +0000 (UTC)
 > "Gilles Dauphin" <dauphin%enst.fr@localhost> wrote:
 > 
 > > I think the bug is in database/db4 
 > > 
 > > Maybe this patch in db4:
 > > 
 > > [root@redhot 1068] pwd
 > > /home/cvsmywork/pkgsrc/databases/db4
 > > [root@redhot 1069] diff -bu buildlink3.mk.orig buildlink3.mk
 > > --- buildlink3.mk.orig  Mon Jul 10 16:00:22 2006
 > > +++ buildlink3.mk       Mon Sep 25 19:48:52 2006
 > > @@ -16,6 +16,11 @@
 > >  BUILDLINK_PKGSRCDIR.db4?=      ../../databases/db4
 > >  BUILDLINK_INCDIRS.db4?=                include/db4
 > >  BUILDLINK_LDADD.db4=           -ldb4
 > > +.if ${OPSYS} == "SunOS"
 > > +BUILDLINK_LDADD.db4=           -R${LOCALBASE}/lib
 > > +BUILDLINK_LDADD.db4+=          -L${LOCALBASE}/lib
 > > +BUILDLINK_LDADD.db4+=          -ldb4
 > > +.endif
 > >  BUILDLINK_TRANSFORM+=          l:db-4:db4
 > >  
 > >  .  include "../../mk/bsd.prefs.mk"
 > 
 > I don't think this fix is the correct way, because buildlink shouldn't
 > need to look for libraries in ${LOCALBASE}/lib, there's BUILDLINK_DIR
 > for that.
 
 This made it work for me (Solaris 9); with no change to databases/db4:
 
 cvs diff: Diffing devel/libgnome
 Index: devel/libgnome/Makefile
 ===================================================================
 RCS file: /cvsroot/pkgsrc/devel/libgnome/Makefile,v
 retrieving revision 1.60
 diff -r1.60 Makefile
 67a68
 > .include "../../databases/db4/buildlink3.mk"