tags 786642 fixed-upstream thanks I just commited Robert's patch to the upstream git so it will be part of the next run.
Thanks, Mt.
On Sun, May 24, 2015 at 12:48:59PM +0200, Robert Luberda wrote:
> tags 786642 patch
> thanks
>
> Hi,
>
> David Prévot pisze:
> >
> > It might be related to the change introduced to fix #744366, thus
> > X-D-CCing Robert who proposed the initial patch (and may have a clear
> > view about what is going on).
>
> Yes, it's my fault, thanks for pointing this out.
>
>
> > manpages-fr-extra package, and built with po4a 0.45), currently starts
> > with:
> > .\" Setup ////////////////////////////////////////////////////////////////
> > [skipped]
> > When building the same page with the latest po4a (0.46), the translated
> > top.1 man page starts with:
> > \" Setup ////////////////////////////////////////////////////////////////
>
> The bug lies in the following fragment of Man.pm code:
> if ($line =~ m/^[.']*$/) {
> if ($c !~ m/^\s+$/) {
> # This commentted line may be comment for the next
> paragraph
> push @next_comments, [substr($line,0,1),$c];
>
> While adding the above substr() call in my previous patch, I failed to
> notice that $line might be actually empty. The attached patch fixes this
> oversight.
>
> Regards,
> robert
>
>
>
>
>
>
>
> Index: lib/Locale/Po4a/Man.pm
> ===================================================================
> --- lib/Locale/Po4a/Man.pm (wersja 2777)
> +++ lib/Locale/Po4a/Man.pm (kopia robocza)
> @@ -686,8 +686,8 @@
>
> if ($line =~ m/^[.']*$/) {
> if ($c !~ m/^\s+$/) {
> - # This commentted line may be comment for the next paragraph
> - push @next_comments, [substr($line,0,1),$c];
> + # This commented line may be comment for the next paragraph
> + push @next_comments, [$line ? substr($line,0,1) : '.', $c];
> }
> if ($line =~ m/^[.']+$/) {
> # those lines are ignored
--
Every day of my life I am forced to add another name to the list of people
who piss me off! --- Calvin
signature.asc
Description: Digital signature
