From owner-svn-ports-head@freebsd.org Mon Aug 3 11:06:36 2015 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 591DC9B23AF; Mon, 3 Aug 2015 11:06:36 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id 474CC18D4; Mon, 3 Aug 2015 11:06:36 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 1033) id 45DB9167A; Mon, 3 Aug 2015 11:06:36 +0000 (UTC) Date: Mon, 3 Aug 2015 11:06:36 +0000 From: Alexey Dokuchaev To: marino@freebsd.org Cc: Gerald Pfeifer , Joe Marcus Clarke , ports-committers@freebsd.org, svn-ports-all@freebsd.org, "svn-ports-head@freebsd.org" Subject: Re: svn commit: r383894 - in head/ports-mgmt/portlint: . src Message-ID: <20150803110636.GA66558@FreeBSD.org> References: <201504130453.t3D4rQmX037343@svn.freebsd.org> <55BF0D7E.3070407@marino.st> <55BF17C9.10600@marino.st> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <55BF17C9.10600@marino.st> User-Agent: Mutt/1.5.23 (2014-03-12) X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 03 Aug 2015 11:06:36 -0000 On Mon, Aug 03, 2015 at 09:27:05AM +0200, John Marino wrote: > On 8/3/2015 9:19 AM, Gerald Pfeifer wrote: > > On Mon, 3 Aug 2015, John Marino wrote: > >>> is wrong about emulators/wine-devel/files/patch-partial-reservation > >>> and especially emulators/wine-devel/files/patch-dlls_kernel32_Makefile.in > >>> or lang/gcc6-devel/patch-unwind-ia64.h ? These are multi-file, "cumulative" patches; portlint(1) suggesting that they should be regenerated with "make makepatch" is a bug, but read on... > >> Please don't relax it. > > > > It is overy strict, so relaxing is the right thing to do. > > No, it's not. > We're trying to avoid patch churn. > > > Yes, except there are now lots of false positives. That is > > a problem. It renders portlint less useful (to the point that > > I am considering to not use it any longer since it's become an > > uphill battle submitting patches and bug reports regarding the > > increasing number of false positives coming from portlint). The first problem here is that you're both right. Second problem is that portlint(1) is often perceived as a call to action, (ir)regardless of the actual importance, merit, and fairness of found warnings/errors. Often I ask about necessity of some particular change, and people tell, "portlint complained, so I shut it up" even though it was not really warranted and made things if not worse definitely not better either. > > I am not planning to use `make makepatch`for new patches, either. > > My patches are perfectly fine and do not exhibit the problem you > > are concerned about, so why warn about them? In fact, `make > > makepatch` would remove key information in some cases. > > > > (Warning about patches that cary specific timezone information, > > not ones that lack "UTC" makes sense, of course.) Since it's easy to detect cumulative patch (either created by hand or with one of John's tools for example) and tell it apart from patch-per- file case, this check can be limited to the latter only? ./danfe