Skip Menu |

This queue is for tickets about the Pod-Perldoc CPAN distribution.

Report information
The Basics
Id: 76029
Status: resolved
Priority: 0/
Queue: Pod-Perldoc
People
Owner: Nobody in particular
Requestors: jdhedden [...] cpan.org
Cc: JAYALLEN [...] cpan.org
AdminCc:
Bug Information
Severity: Important
Broken in: 3.17
Fixed in: (no value)

History Show all quoted text
  Sun Mar 25 09:15:53 2012 jdhedden [...] cpan.org - Ticket created
Subject: [PATCH] Fix 'Is_dos' typo
Show quoted text
> perldoc -i -l File::Find
Can't locate object method "Is_dos" via package "Pod::Perldoc" at /usr/lib/perl5/5.12.4/Pod/Perldoc.pm line 1424. The 'Is_dos' at that line should be 'is_dos'. Patch attached.
Subject: patch.txt
--- Pod-Perldoc-3.17/lib/Pod/Perldoc.pm +++ Pod-Perldoc-fixed/lib/Pod/Perldoc.pm @@ -1423,7 +1423,7 @@ if(!$self->opt_i or $self->is_vms or $self->is_mswin32 - or $self->Is_dos or $self->is_os2 + or $self->is_dos or $self->is_os2 ) { # On a case-forgiving file system, or if case is important, # that is it, all we can do.
  Sun Mar 25 21:44:48 2012 mrallen1 [...] yahoo.com - Correspondence added
On Sun Mar 25 09:15:53 2012, JDHEDDEN wrote: Show quoted text
> > perldoc -i -l File::Find
> Can't locate object method "Is_dos" via package "Pod::Perldoc" at > /usr/lib/perl5/5.12.4/Pod/Perldoc.pm line 1424. > > The 'Is_dos' at that line should be 'is_dos'. Patch attached. >
Thanks for the report and the patch! Much appreciated.
  Sun Mar 25 21:44:49 2012 The RT System itself - Status changed from 'new' to 'open'
  Mon Mar 26 23:36:41 2012 mrallen1 [...] yahoo.com - Correspondence added
Thanks - applied in 80d1778
  Mon Mar 26 23:36:43 2012 mrallen1 [...] yahoo.com - Status changed from 'open' to 'patched'
  Mon May 21 10:14:36 2012 MARKF [...] cpan.org - Correspondence added
On Mon Mar 26 23:36:41 2012, mallen wrote: Show quoted text
> Thanks - applied in 80d1778
Any chance of a CPAN release?
  Mon May 21 10:14:38 2012 The RT System itself - Status changed from 'patched' to 'open'
  Mon May 21 12:05:22 2012 mrallen1 [...] yahoo.com - Status changed from 'open' to 'patched'
  Mon May 21 12:06:39 2012 mrallen1 [...] yahoo.com - Correspondence added
Show quoted text
> Any chance of a CPAN release?
Yeah, I was kind of waiting for 5.16.0 to drop, so I should have the tuits in a day or so. If you can't wait (or don't want to wait) you can grab a tarball of HEAD from github. Thanks.
  Mon May 21 12:06:41 2012 The RT System itself - Status changed from 'patched' to 'open'
  Fri Nov 09 03:07:41 2012 JAYALLEN [...] cpan.org - Correspondence added
I've re-fixed this bug four times now (on new installs and forced reinstalls) and finally went to GitHub to fork the repo and send you a pull request but then I see that you fixed it some time ago. I would really appreciate an update (with that as the only change if it makes it simpler) so I don't have to do this for a fifth time.
  Fri Nov 09 03:40:03 2012 JAYALLEN [...] cpan.org - Cc JAYALLEN added
  Sun Jan 27 23:11:55 2013 mrallen1 [...] yahoo.com - Correspondence added
On Fri Nov 09 03:07:41 2012, JAYALLEN wrote: Show quoted text
> I've re-fixed this bug four times now (on new installs and forced > reinstalls) and finally > went to GitHub to fork the repo and send you a pull request but then I > see that you > fixed it some time ago. I would really appreciate an update (with > that as the only > change if it makes it simpler) so I don't have to do this for a fifth > time.
I've released 3.18 to CPAN with this fix included.
  Sun Jan 27 23:11:56 2013 mrallen1 [...] yahoo.com - Status changed from 'open' to 'resolved'