From owner-freebsd-ports@FreeBSD.ORG Thu Jun 21 09:26:20 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 668431065796; Thu, 21 Jun 2012 09:26:20 +0000 (UTC) (envelope-from martymac@FreeBSD.org) Received: from lmtp.galacsys.net (webmail.galacsys.net [IPv6:2001:1b78:0:1:d918:51d7:0:1]) by mx1.freebsd.org (Postfix) with ESMTP id 00BE98FC16; Thu, 21 Jun 2012 09:26:20 +0000 (UTC) Received: from martymac.org (webmail.galacsys.net [217.24.81.215]) by lmtp.galacsys.net (Postfix) with ESMTP id 2A79D1FA5D49; Thu, 21 Jun 2012 11:26:19 +0200 (CEST) From: "Ganael LAPLANCHE" To: Matthew Seaman , Baptiste Daroussin X-Openwebmail-Date: Thu, 21 Jun 2012 12:26:19 +0200 Message-Id: <20120621090226.M48537@FreeBSD.org> In-Reply-To: <4FE1C1EA.8070901@FreeBSD.org> References: <4FE12F2D.9080302@FreeBSD.org> <20120620061136.GA79164@ithaqua.etoilebsd.net> <4FE1829D.6030109@FreeBSD.org> <20120620091321.GA83730@ithaqua.etoilebsd.net> <20120620094431.M12775@martymac.org> <20120620103412.GB83730@ithaqua.etoilebsd.net> <4FE1C1EA.8070901@FreeBSD.org> X-Mailer: Open WebMail 2.01 20030425 X-OriginatingIP: 157.99.64.43 (ganael.laplanche@martymac.org) MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Date: Thu, 21 Jun 2012 09:26:20 +0000 (UTC) Cc: ports@FreeBSD.org, Michael Scheidell Subject: Re: optionsng and tinderbox? 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: Thu, 21 Jun 2012 09:26:20 -0000 On Wed, 20 Jun 2012 13:28:26 +0100, Matthew Seaman wrote Hi Matthew, Baptiste, Thank you very much for your answers. With your patch, Matthew, setting : openldap24-client_SET+=FETCH in make.conf now works for net/openldap24-client : $ cd /usr/ports/net/openldap24-client $ make -V PORT_OPTIONS DOCS EXAMPLES FETCH NLS Great :) > [...] > Shouldn't make.conf / commandline settings override OPTIONSFILE rather > than the other way round? Seems there's not much point in being able to > set options from make.conf unless that is so, as OPTIONSFILE would be > created more often than not whenever make(1) was invoked in the port's > directory. I think that command-line options should always override file ones, but the main problem here is that we cannot distinguish what comes from the command line from what comes from make.conf. What would sound logical to me would be the following order of precedence : make.conf -> overridden by option files -> overridden by command line But maybe the idea of setting xxx_SET from the command line was just silly ? > [...] > Before my patches the port had: > > % make -V UNIQUENAME -V LATEST_LINK -V OPTIONSFILE > openldap24 > openldap24-client > /var/db/ports/openldap24/options > > % cd ../openldap24-server > % make -V UNIQUENAME -V LATEST_LINK -V OPTIONSFILE > openldap24 > openldap24-server > /var/db/ports/openldap24/options This is no more the case, see : http://www.freebsd.org/cgi/cvsweb.cgi/ports/net/openldap24-server/Makefile.diff?r1=1.200;r2=1.201 $ cd /usr/ports/net/openldap24-client $ make -V UNIQUENAME -V LATEST_LINK -V OPTIONSFILE openldap24-client openldap24-client /var/db/ports/openldap24-client/options $ cd /usr/ports/net/openldap24-server $ make -V UNIQUENAME -V LATEST_LINK -V OPTIONSFILE openldap24-server openldap24-server /var/db/ports/openldap24-server/options > However on inspection, openldap24-server and openldap24-client take > different sets of options: -server takes a super-set of -client. So > having a shared OPTIONSFILE doesn't make sense to me [...] You're right ! This is now fixed in the current ports tree, but your patch reversed that change :p Could you remove the OPTIONS_DIR patch for net/openldap24-server ? Thank you very much, Best regards, -- Ganael LAPLANCHE http://www.martymac.org | http://contribs.martymac.org FreeBSD: martymac , http://www.FreeBSD.org