[prev in list] [next in list] [prev in thread] [next in thread]
List: coreutils
Subject: Re: realpath edge cases
From: Pádraig_Brady <P () draigBrady ! com>
Date: 2017-06-29 6:48:27
Message-ID: 21a83313-a986-a02f-bd08-8a2db1dd31c9 () draigBrady ! com
[Download RAW message or body]
On 28/06/17 17:49, Assaf Gordon wrote:
> Hello Pádraig,
>
>> On Jun 28, 2017, at 12:29, Pádraig Brady <P@draigBrady.com> 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
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic