Front page | perl.perl5.porters |
Postings from November 2016
Re: v5.25.5-184-ga5540cf breaks texinfo
Thread Previous
From:
Karl Williamson
Date:
November 2, 2016 19:50
Subject:
Re: v5.25.5-184-ga5540cf breaks texinfo
Message ID:
0308f25a-4c5d-cf03-c7d2-6dcb9a61302f@khwilliamson.com
On 11/02/2016 07:52 AM, Mathieu Arnold wrote:
> Is there anything I can help with so that this regression gets fixed ?
You were right to bring this up again. You should file a ticket against
perl so that it doesn't get lost. Besides that, when reporting, it's
better to include the full commit message of the breaking commit. It
turns out that this commit is mine, and if I had seen my name in your
first email, I would have investigated at the time. But it didn't jump
out at me from the email that this is something I might know something
about.
>
> Le 25/10/2016 à 14:40, James E Keenan a écrit :
>> On 10/25/2016 08:15 AM, Mathieu Arnold wrote:
>>> Hello,
>>>
>>> Like every week, I updated the perl5-devel FreeBSD port (this week, from
>>> v5.25.5-115-g4eadd82 to v5.25.6-72-g92c843f).
>>> Perl built just fine, and make test runs just fine fine too.
>>>
>>> Today, I discovered that texinfo could not be built with the new Perl
>>> version, after a quick git bisect (git bisect is your god), I ended up
>>> with v5.25.5-183-g8204e83 being the last good, and v5.25.5-184-ga5540cf
>>> being the first bad.
>>>
>>> The error I get in texinfo is:
>>>
>>> Modification of a read-only value attempted at
>>> ../tp/Texinfo/Convert/Line.pm line 309.
>>>
>>> That line being:
>>>
>>> } elsif ($text =~ s/^(([^\s\p{InFullwidth}]|[\x{202f}\x{00a0}])+)//) {
>>>
>>>
>>> When that fails, $text contains " Texinfo documentation system".
>>>
>>> Adding use diagnostics to that file, I get https://pastebin.com/T9VfP2cE
>>>
>>> It turns out that the problem is \p{InFullwidth} (comes from
>>> Unicode::EastAsianWidth) and its usage, in that case (I don't know
>>> texinfo at all) got broken.
>>>
>>
>> For reference, Texinfo::Convert::Line is part of the GNU texinfo library:
>>
>> http://ftpmirror.gnu.org/texinfo/texinfo-6.1.tar.gz
>>
>>
>>
>
>
Thread Previous