[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: realpath edge cases
From: |
Pádraig Brady |
Subject: |
Re: realpath edge cases |
Date: |
Mon, 28 Aug 2017 22:33:10 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 28/06/17 23:48, Pádraig Brady wrote:
> On 28/06/17 17:49, Assaf Gordon wrote:
>> Hello Pádraig,
>>
>>> On Jun 28, 2017, at 12:29, Pádraig Brady <address@hidden> wrote:
>>>
>>> On 27/06/17 18:36, Assaf Gordon wrote:
>>>> On Wed, Jun 28, 2017 at 12:55:29AM +0000, Assaf Gordon wrote:
>>>>> Attached are three patches for realpath's documentation:
>>> I'm getting this message repeated 5 times when building coreutils.info:
>>>
>>> Negative repeat count does nothing at
>>> /usr/share/texinfo/Texinfo/Convert/Line.pm line 124.
>>
>> I'm not able to reproduce this warning (tried with texinfo versions from
>> 4.8a to 6.4) -
>> I wonder what triggered it.
>
> Eek, sorry I should have said I used texinfo-6.1-4.fc25.x86_64
>
>> Versions older than 6.1 warned about a missing "@menu" item, I'll add that
>> before pushing
>> (despite 6.0 being too old for coreutils, and texinfo>6.1 allow omitting the
>> @menu).
>
> The warning is caused by the @exdents (which occur nowhere else in the file).
> The code producing the warning is:
>
> if ($line->{'indent_length'}) {
> $result .= ' ' x ($line->{'indent_length'} - $line->{'counter'});
> print STDERR "INDENT.L($line->{'counter'})\n" if ($line->{'DEBUG'});
> }
>
> So 'counter' must be more that 'indent_length' which must be non zero.
> It's only a warning so it doesn't matter really.
> I'll leave it up to you if you want to leave the @exdents in place
>
> cheers,
> Pádraig
>
>
I'm applying these without the @exdent
thanks,
Pádraig
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: realpath edge cases,
Pádraig Brady <=