From owner-svn-ports-all@FreeBSD.ORG Tue Sep 4 16:31:11 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 DD327106566C; Tue, 4 Sep 2012 16:31:10 +0000 (UTC) (envelope-from flo@smeets.im) Received: from mail.solomo.de (mail.solomo.de [85.214.62.193]) by mx1.freebsd.org (Postfix) with ESMTP id 4323D8FC15; Tue, 4 Sep 2012 16:31:09 +0000 (UTC) Received: from mail.solomo.de (localhost [127.0.0.1]) by mail.solomo.de (Postfix) with ESMTP id 8735AC381E; Tue, 4 Sep 2012 18:31:08 +0200 (CEST) X-Virus-Scanned: amavisd-new at solomo.de Received: from mail.solomo.de ([127.0.0.1]) by mail.solomo.de (mail.solomo.de [127.0.0.1]) (amavisd-new, port 10024) with LMTP id iqWez4yyPKIt; Tue, 4 Sep 2012 18:31:08 +0200 (CEST) Received: from nibbler-osx-wlan.fritz.box (unknown [85.22.118.132]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by mail.solomo.de (Postfix) with ESMTPSA id 9828CC381B; Tue, 4 Sep 2012 18:31:07 +0200 (CEST) Message-ID: <50462CCA.5060306@smeets.im> Date: Tue, 04 Sep 2012 18:31:06 +0200 From: Florian Smeets User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:16.0) Gecko/20120828 Thunderbird/16.0 MIME-Version: 1.0 To: Alex Kozlov 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> In-Reply-To: <20120904161424.GA28248@ravenloft.kiev.ua> X-Enigmail-Version: 1.5a1pre Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enig3B3D97AE413492126B37AAC3" Cc: svn-ports-head@freebsd.org, Doug Barton , svn-ports-all@freebsd.org, ports-committers@freebsd.org, Shaun Amott , Chris Rees 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:31:11 -0000 This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig3B3D97AE413492126B37AAC3 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable 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 fil= e >>> (have EoL on every line including last). >>> >>>> If you have the list of files with this problem and aren't intereste= d >>>> 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 mo= nths, >>> 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 p= atch. >=20 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. Florian --------------enig3B3D97AE413492126B37AAC3 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- iEYEARECAAYFAlBGLMoACgkQapo8P8lCvwkWxACgn+kNMe8UznmfaDHCqhu1UJns shAAoKMqAz3BAq743yferzPi78nXCtpR =6Bo/ -----END PGP SIGNATURE----- --------------enig3B3D97AE413492126B37AAC3--