From owner-cvs-ports@FreeBSD.ORG Thu Feb 19 04:14:09 2004 Return-Path: Delivered-To: cvs-ports@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 1CC9916A4CE; Thu, 19 Feb 2004 04:14:09 -0800 (PST) Received: from www.russia.cz (support.portaone.com [195.70.151.35]) by mx1.FreeBSD.org (Postfix) with ESMTP id 6984B43D1F; Thu, 19 Feb 2004 04:14:08 -0800 (PST) (envelope-from sobomax@portaone.com) Received: from portaone.com (localhost [127.0.0.1]) (authenticated bits=0) by www.russia.cz (8.12.8p2/8.12.8) with ESMTP id i1JCE0Vt005075; Thu, 19 Feb 2004 13:14:03 +0100 (CET) (envelope-from sobomax@portaone.com) Message-ID: <4034A883.80907@portaone.com> Date: Thu, 19 Feb 2004 14:13:55 +0200 From: Maxim Sobolev User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4) Gecko/20030624 Netscape/7.1 (ax) X-Accept-Language: en-us, en, ru, uk MIME-Version: 1.0 References: <200402191122.i1JBMdHd026435@repoman.freebsd.org> <20040219112932.GA11187@xor.obsecurity.org> <4034A2C4.7030501@portaone.com> <20040219115641.GA11791@xor.obsecurity.org> <4034A78F.9080702@portaone.com> In-Reply-To: <4034A78F.9080702@portaone.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit cc: cvs-ports@FreeBSD.ORG cc: ports-committers@FreeBSD.ORG cc: cvs-all@FreeBSD.ORG cc: Kris Kennaway Subject: Re: cvs commit: ports/net/asterisk Makefile X-BeenThere: cvs-ports@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 19 Feb 2004 12:14:09 -0000 Maxim Sobolev wrote: > Kris Kennaway wrote: > >> On Thu, Feb 19, 2004 at 01:49:24PM +0200, Maxim Sobolev wrote: >> >>> Kris Kennaway wrote: >>> >>> >>>> On Thu, Feb 19, 2004 at 03:22:39AM -0800, Maxim Sobolev wrote: >>>> >>>> >>>>> sobomax 2004/02/19 03:22:39 PST >>>>> >>>>> FreeBSD ports repository >>>>> >>>>> Modified files: >>>>> net/asterisk Makefile Log: >>>>> Replace '${WRKDIRPREFIX}${PORTSDIR}/' with '${WRKDIR}/../../../' when >>>>> reffering other ports, since the former for some reason doesn't work >>>>> in bento environment (it works here just fine even with WRKDIRPREFIX >>>>> set). >>>>> >>>>> BTW, this is good evidence which shows that breakage on bento isn't >>>>> sufficient to mark port as BROKEN. >>>> >>>> >>>> >>>> It exposed your incorrect assumption and made you fix it, didn't it? >>>> ;-) >>>> >>>> You were bitten by the fact that bento (deliberately) builds ports in >>>> /a/ports, with /usr/ports a symlink to this directory. In other >>>> words, in legitimate environments, ${PORTSDIR} may not be equal to >>>> ${.CURDIR} even though both reference the same directory. >>> >>> >>> Or equially likely it exposes a bug in bento's setup, where >>> ${PORTSDIR} should be set to /a/ports not to /usr/ports. ;) >> >> >> >> No, as I said, this is a supported configuration. Moreover, I'm sure >> it's common for people to move their ports collection from /usr/ports >> to some other location and replace it with a symlink (in fact I've >> done that myself), and this works for all 10364 ports except yours, >> prior to this commit. > > > I still think that you aren't quite correct. You (and everyone who want > to move /usr/ports over) should have set PORTSDIR to its real location > (that is /a/ports in bento scripts), which would allow > '${WRKDIRPREFIX}${PORTSDIR}/' in ports Makefiles instead of much uglier > ${WRKDIR}/../../../' (or eaquially ugly construct involving .CURDIR). > The latter is worse because in this case it is impossible to put ports > directory (e.g. asterisk in this case) anywhere in the file system not > in the ${PORTSDIR} and have it building just fine, which is possible > with the former. IMO, this is much common and useful feature than > ability to move /usr/ports with the help of symlink. Also my version of behaviour is documented (ans has been for a long time) as the One True Way[tm], so that I'd suggest you to fix bento scripts. http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/porting-wrkdirprefix.html -Maxim