svn commit: r303626 - head/www/xpi-firebug

[Florian Smeets]

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 at 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.

Florian

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 163 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/svn-ports-head/attachments/20120904/6e893cc8/attachment.sig>