Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 4 Sep 2012 17:47:53 +0100
From:      Shaun Amott <shaun@FreeBSD.org>
To:        Florian Smeets <flo@smeets.im>
Cc:        svn-ports-head@freebsd.org, Doug Barton <dougb@freebsd.org>, svn-ports-all@freebsd.org, ports-committers@freebsd.org, Chris Rees <utisoft@gmail.com>, Alex Kozlov <ak@freebsd.org>
Subject:   Re: svn commit: r303626 - head/www/xpi-firebug
Message-ID:  <20120904164752.GA79147@charon.picobyte.net>
In-Reply-To: <50462CCA.5060306@smeets.im>
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> <CADLo838vy-%2BjvWD6ONN0KXOMFDBV=ZB-34k9ZJx7hiJs2kM83w@mail.gmail.com> <20120904161424.GA28248@ravenloft.kiev.ua> <50462CCA.5060306@smeets.im>

next in thread | previous in thread | raw e-mail | index | archive | help
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" <ak@freebsd.org> 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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120904164752.GA79147>