From owner-svn-ports-all@FreeBSD.ORG Tue Sep 4 16:48:01 2012 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C7E68106564A; Tue, 4 Sep 2012 16:48:01 +0000 (UTC) (envelope-from shaun@FreeBSD.org) Received: from alpha.inerd.com (alpha.inerd.com [204.109.56.118]) by mx1.freebsd.org (Postfix) with ESMTP id 97B538FC12; Tue, 4 Sep 2012 16:48:01 +0000 (UTC) Received: from charon.picobyte.net (localhost [127.0.0.1]) by alpha.inerd.com (Postfix) with ESMTP id D03D7147CD0; Tue, 4 Sep 2012 16:47:54 +0000 (UTC) Date: Tue, 4 Sep 2012 17:47:53 +0100 From: Shaun Amott To: Florian Smeets Message-ID: <20120904164752.GA79147@charon.picobyte.net> References: <201209032233.q83MXSa2053725@svn.freebsd.org> <50453146.2080707@smeets.im> <504537D2.5050608@FreeBSD.org> <50454279.4020309@smeets.im> <50457770.4060402@FreeBSD.org> <20120904041502.GA70861@ravenloft.kiev.ua> <20120904161424.GA28248@ravenloft.kiev.ua> <50462CCA.5060306@smeets.im> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <50462CCA.5060306@smeets.im> User-Agent: Mutt/1.5.20 (FreeBSD i386) Cc: svn-ports-head@freebsd.org, Doug Barton , svn-ports-all@freebsd.org, ports-committers@freebsd.org, Chris Rees , Alex Kozlov Subject: Re: svn commit: r303626 - head/www/xpi-firebug X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Sep 2012 16:48:01 -0000 On Tue, Sep 04, 2012 at 06:31:06PM +0200, Florian Smeets wrote: > On 04.09.12 18:14, Alex Kozlov wrote: > > On Tue, Sep 04, 2012 at 12:32:44PM +0100, Chris Rees wrote: > >> On 4 Sep 2012 05:15, "Alex Kozlov" wrote: > >>> On Mon, Sep 03, 2012 at 08:37:20PM -0700, Doug Barton wrote: > >>>> On 09/03/2012 16:51, Florian Smeets wrote: > >>>>> So I reread the complete thread (r301039 and followups) again, the > >>>>> only problem there seems to be is pchecker complaining, > >>>> It's a problem for anything that uses 'while read ...' to process > >>>> those files, including portmaster. > >>> I think it breaks all parsers that assume that distinfo is a text file > >>> (have EoL on every line including last). > >>> > >>>> If you have the list of files with this problem and aren't interested > >>>> in fixing them, pass the list along and I'll do it. > >>> I plan to fix all occurrences of this in ports tree once every few months, > >>> but if you wish to fix it now, here is a list: > >>> audio/pianobar/distinfo > >>> devel/pecl-libevent/distinfo > >>> www/xpi-firebug/distinfo > >>> www/xpi-showip/distinfo > >>> www/trac-accountmanager/distinfo > >>> mail/meta1/distinfo > >>> net/istgt/distinfo > >>> sysutils/fusefs-rar2fs/distinfo > >> Is this a diff/patch bug? > >> Perhaps it could be fixed as well... or could its behaviour be thought of > >> as correct? > > I tried many times but could not find the culprit. But I also suspect patch. > > > > No i don't think so, it is already in the diff you download from GNATS. > I suspect it is a bug in getpatch, or one of the functions it calls, in > query-pr.cgi. If you look at the patch in the PR with edit-pr (vi or > vim) and use :set list you will see that the newline is there. This is > why I emailed Shaun, so let's wait what he has to say. > > BTW this does not only affect distinfo, it's always the last file of the > patch, so there are a lot of patch-* files which have a missing newline > at the end. It's entirely possible the PR script is being overzealous in trimming the whitespace from around patch blocks. I will look into rectifying this. Shaun -- Shaun Amott // PGP: 0x6B387A9A "A foolish consistency is the hobgoblin of little minds." - Ralph Waldo Emerson