Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 102344
Status: resolved
Priority: 0/
Queue: Pod-Simple
People
Owner: Nobody in particular
Requestors: RUSSELLJ [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 3.30
Fixed in: 3.30_1

History Show all quoted text
  Wed Feb 25 07:27:59 2015 RUSSELLJ [...] cpan.org - Ticket created
Subject: Change in Pod::Simple::Search->find() behaviour - no longer includes current directory in search paths
Prior to v3.30, Pod::Simple::Search->find() would recurse from the current directory (well '.') searching for pod. However 3.30 is no longer including the current directory as a search path even when it is in @INC. Its causing failures in modules that use Pod::Simple, such as Dancer2. It's not clear from the changelog entries if this was an accidental change in behaviour or not.
  Thu Feb 26 12:24:24 2015 dwheeler [...] cpan.org - Status changed from 'new' to 'patched'
  Thu Feb 26 12:24:43 2015 dwheeler [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #102344] Change in Pod::Simple::Search->find() behaviour - no longer includes current directory in search paths
Date: Thu, 26 Feb 2015 09:23:55 -0800
To: bug-Pod-Simple [...] rt.cpan.org
From: "David E. Wheeler" <dwheeler [...] cpan.org>
On Feb 25, 2015, at 4:28 AM, Russell Jenkins via RT <bug-Pod-Simple@rt.cpan.org> wrote: Show quoted text
> Prior to v3.30, Pod::Simple::Search->find() would recurse from the current directory (well '.') searching for pod. > > However 3.30 is no longer including the current directory as a search path even when it is in @INC. Its causing failures in modules that use Pod::Simple, such as Dancer2. > > It's not clear from the changelog entries if this was an accidental change in behaviour or not.
Sort of. Used to be that survey() excluded ‘.’ but find() did not. I unified the two in 3.30 to use the algorithm previously used just for survey(). But looking at the code and docs, I can see no reason why ‘.’ should be excluded by survey(), either, unless it’s to avoid duplicate directories while testing or something. So in a1be54b, I changed the code to instead filter out duplicate directories, rather than ‘.’. https://github.com/theory/pod-simple/commit/a1be54bbcf0fd201bbdcb72dab66d33f2293a9c4 Let me know if that fixes the issue for you. Best, David
  Tue Aug 18 13:47:56 2015 dwheeler [...] cpan.org - Status changed from 'patched' to 'resolved'
  Tue Aug 18 13:47:57 2015 dwheeler [...] cpan.org - Fixed in 3.30_1 added