From owner-freebsd-ports@FreeBSD.ORG Mon Aug 20 15:52:35 2012 Return-Path: Delivered-To: ports@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id D71CA1065675; Mon, 20 Aug 2012 15:52:35 +0000 (UTC) (envelope-from bapt@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 88F3A8FC22; Mon, 20 Aug 2012 15:52:35 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.5/8.14.5) with ESMTP id q7KFqZI0055315; Mon, 20 Aug 2012 15:52:35 GMT (envelope-from bapt@FreeBSD.org) Received: (from bapt@localhost) by freefall.freebsd.org (8.14.5/8.14.5/Submit) id q7KFqZNi055314; Mon, 20 Aug 2012 15:52:35 GMT (envelope-from bapt@FreeBSD.org) X-Authentication-Warning: freefall.freebsd.org: bapt set sender to bapt@FreeBSD.org using -f Date: Mon, 20 Aug 2012 17:52:32 +0200 From: Baptiste Daroussin To: Oliver Fromme Message-ID: <20120820155232.GB23607@ithaqua.etoilebsd.net> References: <5012B7CF.9020002@FreeBSD.org> <201207271702.q6RH2G3O063639@lurza.secnetix.de> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="UHN/qo2QbUvPLonB" Content-Disposition: inline In-Reply-To: <201207271702.q6RH2G3O063639@lurza.secnetix.de> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: FreeBSD Ports , Scot Hetzel , Jase Thew Subject: Re: Question about new options framework (regression?) X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 20 Aug 2012 15:52:36 -0000 --UHN/qo2QbUvPLonB Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Jul 27, 2012 at 07:02:16PM +0200, Oliver Fromme wrote: >=20 > Jase Thew wrote: > > On 27/07/2012 10:41, Baptiste Daroussin wrote: > > >=20 > > > I think that is the more reasonnable, I'll add this when fully back.= I was > > > thinking of LATE_SET and LATE_UNSET but OVERRIDE_SET and OVERRIDE_UN= SET sounds > > > better to me. > >=20 > > What use-case are you thinking of that requires the ability for saved > > config to override manually specified config? If there isn't a > > compelling reason for this, then I'd personally much rather see the > > original behaviour restored rather than adding another two variables. >=20 > Baptiste is right ... The original behaviour is flawed, > because it depends on how the port's maintainer wrote the > Makefile. >=20 > For example, If you have WITH_FOO=3DYES in the options file, > and the port's Makefile checks whether WITH_FOO is set or > unset, then there is *no* way to override that, not even > with the old options framework. >=20 > In the case of zsh I was lucky, because the (old) Makefile > checked if WITH_ZSH_STATIC is set, while the options file > contained WITHOUT_ZSH_STATIC, so I could override that. > If it was checking whether WITHOUT_ZSH_STATIC was unset, > it wouldn't have worked. >=20 > Also, if I wanted to do it the other way round, i.e. set > WITH_ZSH_STATIC in the options file, there would be no > way to unset that on the command line. >=20 > So, Baptiste's approach to fix that alltogether is right, > in my opinion. >=20 > Best regards > Oliver >=20 Does this patch fits your needs? http://people.freebsd.org/~bapt/OVERRIDE_BLA.diff The initial configuration has been saved using make config. # make showconfig =3D=3D=3D> The following configuration options are available for zsh-5.0.0: DEBUG=3Doff: Install debug symbols DOCS=3Doff: Build and install the documentation GDBM=3Don: Enable GDBM support (GPL) MAILDIR=3Don: Enable support for Maildirs in MAIL(PATH) MEM=3Doff: Enable zsh-mem options MULTIBYTE=3Don: multibyte character support PCRE=3Doff: Use Perl Compatible Regular Expressions SECURE_FREE=3Don: Enable zsh-secure-free STATIC=3Doff: Build static executable/libraries =3D=3D=3D> Use 'make config' to modify these settings # OVERRIDE_SET=3DDOCS make showconfig =3D=3D=3D> The following configuration options are available for zsh-5.0.0: DEBUG=3Doff: Install debug symbols DOCS=3Don: Build and install the documentation GDBM=3Don: Enable GDBM support (GPL) MAILDIR=3Don: Enable support for Maildirs in MAIL(PATH) MEM=3Doff: Enable zsh-mem options MULTIBYTE=3Don: multibyte character support PCRE=3Doff: Use Perl Compatible Regular Expressions SECURE_FREE=3Don: Enable zsh-secure-free STATIC=3Doff: Build static executable/libraries =3D=3D=3D> Use 'make config' to modify these settings # OVERRIDE_SET=3DDOCS OVERRIDE_UNSET=3DGDBM make showconfig =3D=3D=3D> The following configuration options are available for zsh-5.0.0: DEBUG=3Doff: Install debug symbols DOCS=3Don: Build and install the documentation GDBM=3Doff: Enable GDBM support (GPL) MAILDIR=3Don: Enable support for Maildirs in MAIL(PATH) MEM=3Doff: Enable zsh-mem options MULTIBYTE=3Don: multibyte character support PCRE=3Doff: Use Perl Compatible Regular Expressions SECURE_FREE=3Don: Enable zsh-secure-free STATIC=3Doff: Build static executable/libraries =3D=3D=3D> Use 'make config' to modify these settings #=20 Sorry for having been long on this. regards, Bapt --UHN/qo2QbUvPLonB Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (FreeBSD) iEYEARECAAYFAlAyXUAACgkQ8kTtMUmk6EweXQCfVhpDrg5CNQNRH+bzuhzOF2xh rvoAnA9YKymgFdfMnLKcmwJ8muQWM5I+ =jRtH -----END PGP SIGNATURE----- --UHN/qo2QbUvPLonB--