From owner-freebsd-ports@FreeBSD.ORG Sun Feb 24 00:30:25 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 9C56E16A403; Sun, 24 Feb 2008 00:30:25 +0000 (UTC) (envelope-from wxs@atarininja.org) Received: from syn.atarininja.org (syn.csh.rit.edu [129.21.60.158]) by mx1.freebsd.org (Postfix) with ESMTP id 711A013C461; Sun, 24 Feb 2008 00:30:25 +0000 (UTC) (envelope-from wxs@atarininja.org) Received: by syn.atarininja.org (Postfix, from userid 1001) id 2B5C55C5B; Sat, 23 Feb 2008 19:33:30 -0500 (EST) Date: Sat, 23 Feb 2008 19:33:30 -0500 From: Wesley Shields To: Yen-Ming Lee Message-ID: <20080224003330.GE73222@atarininja.org> References: <20080223155911.84fe423f.bsd-unix@embarqmail.com> <20080223212653.GC73222@atarininja.org> <20080223165016.8a36f06d.bsd-unix@embarqmail.com> <20080223221346.GD73222@atarininja.org> <759236930802231527iab7ced1ncaf3a117c9fe191c@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <759236930802231527iab7ced1ncaf3a117c9fe191c@mail.gmail.com> User-Agent: Mutt/1.5.17 (2007-11-01) 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: Sun, 24 Feb 2008 00:30:25 -0000 On Sat, Feb 23, 2008 at 03:27:07PM -0800, Yen-Ming Lee wrote: > [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? I understand both viewpoints and don't feel strongly either way. > 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. Personally, the saying I use is "if it affects the package in a significant way, bump PORTREVISION." In this case I would have bumped PORTREVISION, but since it's covered in perl itself, I can understand why you didn't. -- WXS