Re: PR/47420 CVS commit: pkgsrc/mail/milter-greylist

[Matthias Scheler <tron%NetBSD.org@localhost>]

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

From: Matthias Scheler <tron%NetBSD.org@localhost>
To: Richard PALO <richard.palo%baou.fr@localhost>
Cc: gnats-bugs%NetBSD.org@localhost
Subject: Re: PR/47420 CVS commit: pkgsrc/mail/milter-greylist
Date: Sun, 13 Jan 2013 10:11:04 +0000

 On Sun, Jan 13, 2013 at 06:10:12AM +0000, Richard PALO wrote:
 >  The advantage of the following, I believe even on netbsd, is that the 
 >  user can set BDB_DEFAULT to anything required (db1 .. db5) on any platform.
 >  
 >    CONFIGURE_ARGS+=       --enable-drac --with-drac-db=${DRACD_DB}
 >    +BDB_ACCEPTED=db1 db2 db3 db4 db5
 
 This line is superfluous.
 
 >    +.include "../../mk/bdb.buildlink3.mk"
 >    +.if ${BDB_TYPE} != "db1"
 >    +CPPFLAGS+=     -DUSE_DB185_EMULATION
 >    +.endif
 >    +LIBS+=${BDB_LIBS}
 
 But does that construct build under Solaris 11? The whole reason for the
 special treatment of SunOS is that you earlier reported that the build
 failed if "db1" was included in "BDB_ACCEPTED". If it works it is of
 course preferrable.
 
 On a sidenote: the user would have to set "BDB185_DEFAULT", not "BDB_DEFAULT".
 
        Kind regards
 
 -- 
 Matthias Scheler                                  http://zhadum.org.uk/