From owner-svn-ports-all@FreeBSD.ORG Sat Nov 22 21:49:26 2014 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 53CD0AEC; Sat, 22 Nov 2014 21:49:26 +0000 (UTC) Received: from apnoea.adamw.org (apnoea.adamw.org [204.109.59.150]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id CDAA88F4; Sat, 22 Nov 2014 21:49:25 +0000 (UTC) Received: from [192.168.192.25] (75-161-3-142.albq.qwest.net [75.161.3.142]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by apnoea.adamw.org (Postfix) with ESMTPSA id AD8CE11FF58; Sat, 22 Nov 2014 14:49:24 -0700 (MST) Content-Type: text/plain; charset=iso-8859-1 Mime-Version: 1.0 (Mac OS X Mail 8.1 \(1993\)) Subject: Re: svn commit: r373100 - in head: . Mk databases/glom databases/libzdb databases/opendbx databases/pglesslog databases/qt4-pgsql-plugin databases/qt5-sqldrivers-pgsql databases/rubygem-do_postgres dat... From: Adam Weinberger In-Reply-To: <547103CD.2020309@physics.org> Date: Sat, 22 Nov 2014 14:49:23 -0700 Content-Transfer-Encoding: quoted-printable Message-Id: <913FC640-B7AB-4020-A0A0-3B2DA4225DD9@adamw.org> References: <201411222040.sAMKe9tk086292@svn.freebsd.org> <48EBC1CA-CB07-47E0-88A3-94B1ED7B31A0@adamw.org> <547103CD.2020309@physics.org> To: Chris Rees X-Mailer: Apple Mail (2.1993) Cc: Chris Rees , svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 22 Nov 2014 21:49:26 -0000 > On 22 Nov, 2014, at 14:44, Chris Rees wrote: >=20 > On 22/11/2014 21:31, Adam Weinberger wrote: >> On 22 Nov, 2014, at 13:40, Chris Rees wrote: >>> Author: crees >>> Date: Sat Nov 22 20:40:08 2014 >>> New Revision: 373100 >>> URL: https://svnweb.freebsd.org/changeset/ports/373100 >>> QAT: https://qat.redports.org/buildarchive/r373100/ >>>=20 >>> Log: >>> Finally retire USE_PGSQL >>>=20 >>> + USE_PGSQL=3Dserver becomes USES=3Dpgsql and WANT_PGSQL=3Dserver >> USES=3Dperl5 has USE_PERL5, USES=3Dpython has USE_PYTHON, USES=3Dopenal= has USE_OPENAL, USES=3Dfam has USE_FAM. >>=20 >> Why is pgsql WANT_ instead of USE_? >=20 > Hm. A couple of reasons really; USE_PGSQL was used before by = bsd.database.mk, and with the two coexisting perhaps sharing the = variable was not desired; I remember back in December last year when = writing this that people weren't keen on separate variables at all, = preferring to use arguments to USES. During that discussion, no-one = mentioned that WANT_PGSQL wasn't desirable (perhaps it wasn't even = discussed). >=20 > I could rename it to USE_PGSQL now the code is removed from = bsd.database.mk I suppose... but is it necessary? I don't personally = think it's a great idea to reuse variables that used to mean something = else... Definitely makes sense not to use USE_PGSQL while the transition is = underway. But now that the transition is complete, USE_PGSQL is a NOOP = unless USES=3Dpgsql is there, no? So if somebody tried to do = USE_PGSQL=3Dyes it wouldn't work, same as it won't work now. My suggestion would always be to aim for consistency across the = different USES modules, but I can't even begin to guess how much work it = took to get the USES=3Dpgsql transition done! # Adam --=20 Adam Weinberger adamw@adamw.org http://www.adamw.org