Skip Menu |

This queue is for tickets about the PodParser CPAN distribution.

Report information
The Basics
Id: 580
Status: resolved
Priority: 0/
Queue: PodParser

People
Owner: Nobody in particular
Requestors: chris [...] clotho.com
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 1.18
Fixed in: (no value)



Subject: Pod::Parser test fails on Mac OSX
When running make test, test #4 of t/pod/find.t fails on Mac OS X (aka darwin). The reason is that pod_where is looking for perfunc.pod in $privlib, but on darwin it is installed at $privlib/pods/perlfunc.pod. The patch below fixes the test failure by adding a subdirectory to the search path. I'm not sure why the previous search path adjustment was commented out in the first place... This feature was apparently added in 1.17 and removed in 1.18. Chris Dolan diff -u lib/Pod/Find.pm.orig lib/Pod/Find.pm --- lib/Pod/Find.pm.orig Fri Sep 1 10:11:39 2000 +++ lib/Pod/Find.pm Thu May 9 16:29:57 2002 @@ -349,10 +349,12 @@ # Add location of pod documentation for perl man pages (eg perlfunc) # This is a pod directory in the private install tree - #my $perlpoddir = File::Spec->catdir($Config::Config{'installprivlib'}, - # 'pod'); - #push (@search_dirs, $perlpoddir) - # if -d $perlpoddir; + foreach my $subdir ('pod', 'pods') { + my $perlpoddir = File::Spec->catdir($Config::Config{'installprivlib'}, + $subdir); + push (@search_dirs, $perlpoddir) + if -d $perlpoddir; + } # Add location of binaries such as pod2text push (@search_dirs, $Config::Config{'scriptdir'})
From: marekr [...] cpan.org
A similar patch has been applied some time ago - please check whether a recent Pod::Parser still has these problems. -Marek [guest - Thu May 9 17:40:31 2002]: Show quoted text
> When running make test, test #4 of t/pod/find.t fails on Mac OS X (aka > darwin). The reason is that pod_where is looking for perfunc.pod > in $privlib, but on darwin it is installed at > $privlib/pods/perlfunc.pod. The patch below fixes the test failure > by adding a subdirectory to the search path. I'm not sure why the > previous search path adjustment was commented out in the first > place... This feature was apparently added in 1.17 and removed in > 1.18. > > Chris Dolan > > > diff -u lib/Pod/Find.pm.orig lib/Pod/Find.pm > --- lib/Pod/Find.pm.orig Fri Sep 1 10:11:39 2000 > +++ lib/Pod/Find.pm Thu May 9 16:29:57 2002 > @@ -349,10 +349,12 @@ > > # Add location of pod documentation for perl man pages (eg > perlfunc) > # This is a pod directory in the private install tree > - #my $perlpoddir = File::Spec-
> >catdir($Config::Config{'installprivlib'},
> - # 'pod'); > - #push (@search_dirs, $perlpoddir) > - # if -d $perlpoddir; > + foreach my $subdir ('pod', 'pods') { > + my $perlpoddir = File::Spec-
> >catdir($Config::Config{'installprivlib'},
> + $subdir); > + push (@search_dirs, $perlpoddir) > + if -d $perlpoddir; > + } > > # Add location of binaries such as pod2text > push (@search_dirs, $Config::Config{'scriptdir'})
See the ticket history - this ought to be already fixed. Feel free to reopen this ticket if you still see problems with Pod-Parser-1.30 -Marek