From owner-freebsd-ports@FreeBSD.ORG Sat Feb 23 23:52:12 2008 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C3F2616A403 for ; Sat, 23 Feb 2008 23:52:12 +0000 (UTC) (envelope-from leeym@leeym.com) Received: from an-out-0708.google.com (an-out-0708.google.com [209.85.132.251]) by mx1.freebsd.org (Postfix) with ESMTP id 9BFB413C467 for ; Sat, 23 Feb 2008 23:52:12 +0000 (UTC) (envelope-from leeym@leeym.com) Received: by an-out-0708.google.com with SMTP id c14so226938anc.13 for ; Sat, 23 Feb 2008 15:52:12 -0800 (PST) Received: by 10.100.9.20 with SMTP id 20mr2363287ani.75.1203809227933; Sat, 23 Feb 2008 15:27:07 -0800 (PST) Received: by 10.101.68.17 with HTTP; Sat, 23 Feb 2008 15:27:07 -0800 (PST) Message-ID: <759236930802231527iab7ced1ncaf3a117c9fe191c@mail.gmail.com> Date: Sat, 23 Feb 2008 15:27:07 -0800 From: "Yen-Ming Lee" To: "Wesley Shields" In-Reply-To: <20080223221346.GD73222@atarininja.org> MIME-Version: 1.0 References: <20080223155911.84fe423f.bsd-unix@embarqmail.com> <20080223212653.GC73222@atarininja.org> <20080223165016.8a36f06d.bsd-unix@embarqmail.com> <20080223221346.GD73222@atarininja.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline X-Content-Filtered-By: Mailman/MimeDel 2.1.5 Cc: Randy Pratt , lbr@FreeBSD.org, Felippe de Meirelles Motta , dougb@freebsd.org, freebsd-ports@freebsd.org, leeym@freebsd.org Subject: Re: Portmaster and added dependencies 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: Sat, 23 Feb 2008 23:52:12 -0000 [cc to the submitter of ports/120802 and lbr@] 2008/2/23, Wesley Shields : > > On Sat, Feb 23, 2008 at 04:50:16PM -0500, Randy Pratt wrote: > > On Sat, 23 Feb 2008 16:26:53 -0500 > > Wesley Shields wrote: > > > > > On Sat, Feb 23, 2008 at 03:59:11PM -0500, Randy Pratt wrote: > > > > I've been using portmaster for a couple of weeks and like what I've > > > > seen. However, I'm a bit confused on how dependencies changes are > to > > > > be handled. Here's a scenario: > > > > > > > > Events: > > > > > > > > 2008.02.17 01:50:08 UTC devel/p5-ExtUtils-CBuilder: update to 0.22 > > > > > > > > 2008.02.17 11:00:00 UTC update ports and devel/p5-ExtUtils-CBuilder > > > > was updated > > > > > > > > 2008.02.19 05:33:50 UTC devel/p5-ExtUtils-CBuilder: Add missing > deps > > > > ports/120802 (textproc/p5-Text-ParseWords > was > > > > added as a build/run dependency) > > > > > > Snipped the rest because I think this could have all been avoided by > > > bumping PORTREVISION when the dependency to p5-Text-ParseWords was > > > added. Ideally the dependency information should have been recorded > > > with the update to 0.22 (like is in ports/120802). I'm CC'ing leeym@ > > > who made the last commit. > > > > It would have been far easier that way of course but this isn't the > > first time a dependency change has been made to some port without > > bumping PORTREVISION and probably won't be the last. This situation > > only existed for a couple of days and affected only those who updated > > during the interim. > > Right, I was just pointing out that it is not necessarily a problem with > portmaster since PORTREVISION should have been bumped. :) Thank you > for brining this up, however, since it is a mistake in the port and may > be a nice addition to portmaster (if such a thing does not already > exist). > The fact behinds this commit is that Text::ParseWords and File::Spec are both in the core list of PERL, so ExtUtils::CBuilder will work no matter these "dependencies" installed or not. See http://www.freebsd.org/cgi/query-pr.cgi?pr=120802 for details. And, two problems in this case: 1. Do we need to add some modules into dependency if they are already in the core list of PERL itself. I myself prefer to remove them and keep the dependency tree as simple as possible, however submitter prefer to use the latest version, so I follow his way. Maybe you have different opinions? 2. What does it mean to bump the PORTREVISION? I heard two meanings, one is "you'd better reinstall it otherwise it won't work", and the other is "something updated and it will change the package". If it means the former one, it's not the case of ExtUtils::CBuilder. If it means the latter one, then it's my fault, I should bump PORTREVISION anyway. -- Yen-Ming Lee