From owner-cvs-all@FreeBSD.ORG Thu Feb 19 04:10:31 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 4F16316A5A3; Thu, 19 Feb 2004 04:10:31 -0800 (PST) Received: from www.russia.cz (web.portaone.com [195.70.151.35]) by mx1.FreeBSD.org (Postfix) with ESMTP id 2770B43D31; Thu, 19 Feb 2004 04:10:04 -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 i1JC9uVt004892; Thu, 19 Feb 2004 13:09:58 +0100 (CET) (envelope-from sobomax@portaone.com) Message-ID: <4034A78F.9080702@portaone.com> Date: Thu, 19 Feb 2004 14:09:51 +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 To: Kris Kennaway References: <200402191122.i1JBMdHd026435@repoman.freebsd.org> <20040219112932.GA11187@xor.obsecurity.org> <4034A2C4.7030501@portaone.com> <20040219115641.GA11791@xor.obsecurity.org> In-Reply-To: <20040219115641.GA11791@xor.obsecurity.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit cc: cvs-ports@FreeBSD.ORG cc: cvs-all@FreeBSD.ORG cc: ports-committers@FreeBSD.ORG Subject: Re: cvs commit: ports/net/asterisk Makefile X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 19 Feb 2004 12:10:31 -0000 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. -Maxim