Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 07 Jan 2008 19:21:53 -0800
From:      Xin LI <delphij@delphij.net>
To:        Scot Hetzel <swhetzel@gmail.com>
Cc:        ports@freebsd.org, sergei@freebsd.org, Xin LI <delphij@freebsd.org>
Subject:   Re: sysutils/cfengine: invalid use of USE_BDB
Message-ID:  <4782EC51.6020108@delphij.net>
In-Reply-To: <790a9fff0801051224p4fb8481pbcc24051b2438dcf@mail.gmail.com>
References:  <790a9fff0801051224p4fb8481pbcc24051b2438dcf@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
This is a multi-part message in MIME format.
--------------040209010407020403070705
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Scot Hetzel wrote:
> I have noticed that sysutils/cfengine is using USE_BDB incorrectly.
> 
> USE_BDB?= 46
> 
> The USE_BDB variable is not a user setable variable. If someone were
> to read this Makefile, they would get the wrong impression that they
> should set USE_BDB in there /etc/make.conf, ports.conf, or
> portupgrade.conf file.   When they should instead be setting the
> WITH_BDB_VER variable.
> 
> The USE_BDB should be set to the minimum version of Berkeley DB that
> the port will build with.
> 
> USE_BDB= 44+
> 
> And left up to the person installing the port to specify a higher version.
> 
> If you want to take the choice away from the installer and have the
> port always use BDB 4.6, then the port should use:
> 
> USE_BDB= 46
> 
> Which will prevent the use of BDB 4.4 and 4.5 from ever being used by this port.

I think the proper use of *_BDB variables is that:

WITH_BDB?=	46

(An advisory usage), and

USE_BDB=	yes

as in the attachment.

Cheers,
- --
Xin LI <delphij@delphij.net>	http://www.delphij.net/
FreeBSD - The Power to Serve!
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.4 (FreeBSD)

iD8DBQFHguxRi+vbBBjt66ARAuHIAKC5rRxKOvsAveDPEoAXbQvKxa4giACfQEpO
EJk7hRJusCc6sWaTpLfiCoM=
=yUim
-----END PGP SIGNATURE-----

--------------040209010407020403070705
Content-Type: text/plain;
 name="cfengine-bdb-usage.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="cfengine-bdb-usage.diff"

Index: Makefile
===================================================================
RCS file: /home/ncvs/ports/sysutils/cfengine/Makefile,v
retrieving revision 1.40
diff -u -p -r1.40 Makefile
--- Makefile	4 Jan 2008 23:54:17 -0000	1.40
+++ Makefile	8 Jan 2008 03:16:59 -0000
@@ -16,7 +16,8 @@ COMMENT=	A systems administration tool f
 
 USE_RC_SUBR=	cfexecd cfservd
 
-USE_BDB?=	46
+WITH_BDB_VER?=	46
+USE_BDB=	yes
 USE_OPENSSL=	yes
 GNU_CONFIGURE=	yes
 CONFIGURE_TARGET=	--build=${MACHINE_ARCH}-portbld-freebsd${OSREL}

--------------040209010407020403070705--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4782EC51.6020108>