From owner-freebsd-ports@FreeBSD.ORG Sun Jan 9 17:59:29 2005 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 50E3D16A4CE; Sun, 9 Jan 2005 17:59:29 +0000 (GMT) Received: from mailout05.sul.t-online.com (mailout05.sul.t-online.com [194.25.134.82]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9570243D2D; Sun, 9 Jan 2005 17:59:28 +0000 (GMT) (envelope-from Alexander@Leidinger.net) Received: from fwd10.aul.t-online.de by mailout05.sul.t-online.com with smtp id 1CnhLb-0008U7-01; Sun, 09 Jan 2005 18:59:27 +0100 Received: from Andro-Beta.Leidinger.net (Ew9rM-ZrYeNtC4N-YvZalks9bEs+adg7ciGCkoXhihV1dehZ5dQdUe@[217.229.213.2]) by fmrl10.sul.t-online.com with esmtp id 1CnhLS-1wikD20; Sun, 9 Jan 2005 18:59:18 +0100 Received: from Magellan.Leidinger.net (Magellan.Leidinger.net [192.168.1.1]) j09HweW3084527; Sun, 9 Jan 2005 18:58:40 +0100 (CET) (envelope-from Alexander@Leidinger.net) Date: Sun, 9 Jan 2005 18:59:48 +0100 From: Alexander Leidinger To: "Simon L. Nielsen" Message-ID: <20050109185948.4470a02d@Magellan.Leidinger.net> In-Reply-To: <20050109170219.GF1187@zaphod.nitro.dk> References: <20050109143903.GC1187@zaphod.nitro.dk> <20050109174945.7f0353e4@Magellan.Leidinger.net> <20050109170219.GF1187@zaphod.nitro.dk> X-Mailer: Sylpheed-Claws 0.9.13 (GTK+ 1.2.10; i386-portbld-freebsd6.0) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-ID: Ew9rM-ZrYeNtC4N-YvZalks9bEs+adg7ciGCkoXhihV1dehZ5dQdUe@t-dialin.net X-TOI-MSGID: 2d79addf-ad38-404f-bbea-bc42b37dd8b4 cc: freebsd-ports@FreeBSD.org Subject: Re: Makeing fetchindex really mirror INDEX X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 09 Jan 2005 17:59:29 -0000 On Sun, 9 Jan 2005 18:02:20 +0100 "Simon L. Nielsen" wrote: > Mainly because we already have a ${INDEXFILE} target, and the two > would conflict, so I thought it would be simpler to keep the simple > target. > > make index in your patch only works because you don't define > ${INDEXFILE} as ${.CURDIR}/${INDEXFILE}, which it should be since Correct. > there might be a object directory. At least as I read it, I could be > wrong. At the time I wrote the first implementation of "fetchindex" it was supposed to be run at the time as "update". So the PORTSDIR has to be writable (except I've overlooked something). If this hasn't changed, there's no need for .CURDIR. > In general I agree my way is slight silly, because it's certainly not I don't think so, we just don't share the same implications. > the right way, I just don't see a really clean way to implement it the > right way (well, ${INDEXFILE}.bz2) could probably be used but it would > still only be half way there). Without changing the index target, I don't see a cleaner way. [...] > > BTW.: if I do it the right way (".PHONY: ${INDEXFILE}.bz2" instead of > > adding " .PHONY" to "${INDEXFILE}.bz2:"), it doesn't work here > > (6-current) as expected. > > I would guess the problem is also related to .CURDIR, but I'm not > sure. I don't see where .CURDIR is supposed to change the behavior here. We're talking about targets, and I've specified them the same at both places. Bye, Alexander. -- The computer revolution is over. The computers won. http://www.Leidinger.net Alexander @ Leidinger.net GPG fingerprint = C518 BC70 E67F 143F BE91 3365 79E2 9C60 B006 3FE7