From owner-cvs-all@FreeBSD.ORG Tue Mar 1 05:37:53 2011 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id EEB19106567A for ; Tue, 1 Mar 2011 05:37:53 +0000 (UTC) (envelope-from dougb@dougbarton.us) Received: from mail2.fluidhosting.com (mx22.fluidhosting.com [204.14.89.5]) by mx1.freebsd.org (Postfix) with ESMTP id 94AA28FC0A for ; Tue, 1 Mar 2011 05:37:53 +0000 (UTC) Received: (qmail 21310 invoked by uid 399); 1 Mar 2011 05:37:50 -0000 Received: from router.ka9q.net (HELO ?192.168.2.9?) (dougb@dougbarton.us@75.60.237.91) by mail2.fluidhosting.com with ESMTPAM; 1 Mar 2011 05:37:50 -0000 X-Originating-IP: 75.60.237.91 X-Sender: dougb@dougbarton.us Message-ID: <4D6C862E.8060001@dougbarton.us> Date: Mon, 28 Feb 2011 21:37:50 -0800 From: Doug Barton Organization: http://SupersetSolutions.com/ User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.13) Gecko/20101207 Thunderbird/3.1.7 MIME-Version: 1.0 To: Sahil Tandon References: <201102271441.p1REfMrT016669@repoman.freebsd.org> <20110227193203.GA38353@magic.hamla.org> <4D6AB7BD.2060203@FreeBSD.org> <20110301013525.GA44112@magic.hamla.org> In-Reply-To: <20110301013525.GA44112@magic.hamla.org> X-Enigmail-Version: 1.1.1 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: cvs-ports@freebsd.org, araujo@FreeBSD.org, cvs-all@freebsd.org, ports-committers@freebsd.org Subject: Re: cvs commit: ports/net/trafshow Makefile distinfo X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 01 Mar 2011 05:37:54 -0000 On 2/28/2011 5:35 PM, Sahil Tandon wrote: > On Mon, 2011-02-28 at 01:18:32 -0300, Marcelo Araujo wrote: > >>> On 02/27/2011 11:32, Sahil Tandon wrote: >>> >>> If the IPv6 option is off by default (and thus, does not affect the >>> default package), why do you bump PORTREVISION? I just want to >>> understand for my own edification when dealing with similar >>> situations. >> >> Well, I've used the latest paragraph that discribe when we should bump >> PORTREVISION. >> >> """A rule of thumb is to ask yourself whether a change committed to a >> port is something which someone, somewhere, would benefit from having >> (either because of an enhancement, fix, or by virtue that the new >> package will actually work for them). If yes, the PORTREVISION should >> be bumped If you stop here, Marcelo's action seems perfectly reasonable, especially for a tool that's a) very fast to compile, b) not a critical dependency, c) doesn't update versions often, and d) growing a substantive new feature. And yes, I know I'm talking out of both sides of my head on this, my point being that reasonable minds can differ on what the right decision was here. :) Meanwhile, Marcelo I was being somewhat glib about defaulting all IPv6 options to "ON," but if you intend to do that please first test whether or not the tool is able to cope with having IPv6 support compiled in, but no INET6 in the kernel. If it still works ok then by all means you should default it to on. Doug (who will try not to be so glib next time) -- Nothin' ever doesn't change, but nothin' changes much. -- OK Go Breadth of IT experience, and depth of knowledge in the DNS. Yours for the right price. :) http://SupersetSolutions.com/