Skip Menu |

This queue is for tickets about the Alien-wxWidgets CPAN distribution.

Report information
The Basics
Id: 20350
Status: resolved
Priority: 0/
Queue: Alien-wxWidgets

People
Owner: Nobody in particular
Requestors: dha [...] panix.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.16
Fixed in: (no value)



Subject: inc/Archive/Extract.pm causes t/zz_pod.t to fail
It's missing a "=back". Patch below ------ *** inc/Archive/Extract.pm Wed Jul 5 15:59:24 2006 --- inc/Archive/Extract-rev.pm Fri Jul 7 14:52:10 2006 *************** *** 924,929 **** --- 924,931 ---- Maybe this module should use something like C<File::Type> to determine the type, rather than blindly trust the suffix. + =back + =head1 AUTHORS This module by
On Fri Jul 07 15:04:45 2006, guest wrote: Show quoted text
> It's missing a "=back". Patch below
I applied your patch, then I realized that it made no sense to test POD for modules I did not make and was not going to install anyway, so I just restricted the POD tests to my own code. Thanks! Mattia
Subject: Re: [rt.cpan.org #20350] inc/Archive/Extract.pm causes t/zz_pod.t to fail
Date: Fri, 7 Jul 2006 16:17:52 -0400
To: via RT <bug-Alien-wxWidgets [...] rt.cpan.org>
From: "David H. Adler" <dha [...] panix.com>
On Fri, Jul 07, 2006 at 04:16:02PM -0400, via RT wrote: Show quoted text
> > <URL: http://rt.cpan.org/Ticket/Display.html?id=20350 > > > On Fri Jul 07 15:04:45 2006, guest wrote:
> > It's missing a "=back". Patch below
> > I applied your patch, then I realized that > it made no sense to test POD for modules I did not make > and was not going to install anyway, so I just restricted > the POD tests to my own code.
I can see that. I'm really a big fan of making those kinds of tests opt in for release. they can really screw things up if you're installing via CPAN.pm. In this case, I figured it was a small enough thing that I'd just patch it, though. *shrug* dha -- David H. Adler - <dha@panix.com> - http://www.panix.com/~dha/ well, um, there will be guitars - marm0t