From owner-freebsd-ports@FreeBSD.ORG Wed Oct 10 11:24:07 2012 Return-Path: Delivered-To: freebsd-ports@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 462728B3 for ; Wed, 10 Oct 2012 11:24:07 +0000 (UTC) (envelope-from freebsd@grem.de) Received: from mail.grem.de (outcast.grem.de [213.239.217.27]) by mx1.freebsd.org (Postfix) with SMTP id 96D668FC14 for ; Wed, 10 Oct 2012 11:24:04 +0000 (UTC) Received: (qmail 38958 invoked by uid 89); 10 Oct 2012 11:24:04 -0000 Received: from unknown (HELO bsd64.grem.de) (mg@grem.de@80.190.102.230) by mail.grem.de with ESMTPA; 10 Oct 2012 11:24:04 -0000 Date: Wed, 10 Oct 2012 12:49:38 +0200 From: Michael Gmelin To: freebsd-ports@FreeBSD.org Subject: Re: HAVE_GNOME vs. bsd.ports.options.mk Message-ID: <20121010124938.3e77bb12@bsd64.grem.de> In-Reply-To: <20121010105757.GD26497@ithaqua.etoilebsd.net> References: <20121010121850.039fb6d2@bsd64.grem.de> <20121010102527.GB26497@ithaqua.etoilebsd.net> <20121010123322.0677a829@bsd64.grem.de> <20121010105757.GD26497@ithaqua.etoilebsd.net> X-Mailer: Claws Mail 3.8.1 (GTK+ 2.24.6; amd64-portbld-freebsd9.0) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Baptiste Daroussin X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 10 Oct 2012 11:24:07 -0000 On Wed, 10 Oct 2012 12:57:57 +0200 Baptiste Daroussin wrote: > On Wed, Oct 10, 2012 at 12:33:22PM +0200, Michael Gmelin wrote: > > On Wed, 10 Oct 2012 12:25:27 +0200 > > Baptiste Daroussin wrote: > > > > > On Wed, Oct 10, 2012 at 12:18:50PM +0200, Michael Gmelin wrote: > > > > Hi > > > > > > > > I noticed that HAVE_GNOME doesn't work properly with > > > > bsd.ports.options.mk yet, so > > > > > > > > .include > > > > .if ${HAVE_GNOME:Mgnomelibs}!="" > > > > # ... > > > > .endif > > > > .include > > > > > > > > won't work, while this > > > > > > > > .include > > > > .if ${HAVE_GNOME:Mgnomelibs}!="" > > > > # ... > > > > .endif > > > > .include > > > > > > > > does. > > > > > > > > AFAIK bsd.port.pre.mk/bsd.port.post.mk should be replaced by > > > > bsd.port.options.mk/bsd.port.mk in the long term, so having this > > > > work or documenting a workaround would help port maintainers > > > > who are in the process of updating the port structure. > > > > > > > > Michael > > > > > > > > -- > > > > Michael Gmelin > > > > _______________________________________________ > > > > freebsd-ports@freebsd.org mailing list > > > > http://lists.freebsd.org/mailman/listinfo/freebsd-ports > > > > To unsubscribe, send any mail to > > > > "freebsd-ports-unsubscribe@freebsd.org" > > > > > > Which ports are you speaking about? given that > > > HAVE_GNOME:#gnomelibs is refering to gnome 1.4 I think this part > > > can safely be dropped out. > > > > > > regards, > > > Bapt > > > > I'm talking about the feature in general, it also applies to > > HAVE_GNOME:Mesound, HAVE_GNOME:Mpygnomeextras etc. > > > > I stumbled over this while converting polish/gnugadu2 to OptionsNG > > (http://www.freebsd.org/cgi/query-pr.cgi?pr=172427). > > > > Michael > > > > -- > > Michael Gmelin > > imho HAVE_GNOME should die, as autodectection of what you have does > not fit at all with package building. Thing shouldn't been added > magically because they are on your system but only on explicit demand > of the maintainer/user. > > That said I'll have a look at the PR. > > regards, > Bapt I found this behaviour in many ports and I agree that by default people should be able to explicitly state what they want. On the other hand it can be extremely convenient to get all plugins your system supports by default (for those of us you build their ports themselves). I solved this in this PR by adding an "AUTODETECT" option, that allows the port to detect automatically only when explicitly asked by the builder. I had that turned on by default to make sure the port behaves exactly like it did before conversion to OptionsNG (it's not my lawn, you know). The committer changed that to be off by default, since this is a better solution for package building and I agree with him. Also note that there are a lot of ports that use either techniques for auto detection (e.g. checking for the existence of libraries to bring in functionality) and that those should be covered as well - simply not allowing auto detection will massively reduce functionality, so using an OPTION to allow it might be the way to go. I think AUTODETECT might be an interesting candidate for bsd.options.mk though. Michael -- Michael Gmelin