Bug#724498: perlsyn(1) contains POD errors

[Niko Tyni]

clone 724498 -1
retitle -1 perl: installman should die on pod errors like pod2man does
reassign -1 perl 5.20.0-4
severity -1 normal
submitter -1 !
tags -1 = upstream
thanks

On Tue, Sep 24, 2013 at 09:43:46PM +0300, Niko Tyni wrote:
> On Tue, Sep 24, 2013 at 03:25:56PM +0300, Damyan Ivanov wrote:
> > Package: perl-doc
> > Version: 5.18.1-4
> > Severity: minor
> > File: /usr/share/man/man1/perlsyn.1.gz
> > Tags: upstream
>  
> > The perlsyn(1) manual page contains a POD ERRORS section:
> 
> Thanks for noticing.
> 
> > I wonder how was this built
> 
> It's done by the 'installman' script, and that uses Pod::Man directly
> instead of calling the pod2man script (which nowadays dies on errors.)
> 
> > since all other POD-shipping packages fail to 
> > build with perl 5.18 if their POD has errors :)
> 
> Yeah, feels a bit like we're applying double standards :) I doubt this
> is an intentional choice upstream but I don't really know. Possibly we
> should ask.

On Tue, Sep 24, 2013 at 01:58:24PM -0700, Russ Allbery wrote:

> Or, alternately, add errors => 'die' to the constructor arguments for
> Pod::Man, which is all that pod2man does.

The perlsyn.pod syntax errors have been fixed now, but before closing
this bug I'm cloning a separate one about making 'installman' die on
errors like pod2man does (one way or another; errors => 'die' as per is
indeed probably the best choice.)
-- 
Niko Tyni   ntyni at debian.org