Mon Aug 18 03:31:47 2003spoon [...] cpan.org - Ticket created
Date:
Mon, 18 Aug 2003 17:30:51 +1000
To:
bug-class-factory-util [...] rt.cpan.org
Subject:
Find subclasses in @INC
From:
Iain Truskett <spoon [...] cpan.org>
Heyo,
I was contemplating whether to use Class::Factory::Util for DTF::Builder
but found you don't look very far for subclasses.
The attached patch makes searching look in @INC as well as where the
module came from.
cheers,
--
Iain.
Message body is not shown because sender requested not to inline it.
Sat Aug 23 21:52:49 2003DROLSKY [...] cpan.org - Told changed from Not set to Sat Aug 23 21:52:49 2003
Sat Aug 23 22:05:05 2003autarch [...] urth.org - Correspondence added
Date:
Sat, 23 Aug 2003 21:04:25 -0500 (CDT)
From:
Dave Rolsky <autarch [...] urth.org>
To:
Iain Truskett via RT <bug-Class-Factory-Util [...] rt.cpan.org>
Subject:
Re: [cpan #3278] Find subclasses in @INC
RT-Send-Cc:
On Mon, 18 Aug 2003, Iain Truskett via RT wrote:
Show quoted text
> I was contemplating whether to use Class::Factory::Util for DTF::Builder
> but found you don't look very far for subclasses.
>
> The attached patch makes searching look in @INC as well as where the
> module came from.
I applied it. I'll try uploading it to PAUSE but it may still be broken.
-dave
/*=======================
House Absolute Consulting
www.houseabsolute.com
=======================*/
Sun Aug 24 08:22:31 2003PODMASTER [...] cpan.org - Correspondence added
Let's not forget about PERL5LIB and PERLLIB,
something along these lines
(ps , on win32 perllib is treated like path):
require File::Spec;
for my $i( $ENV{PERL5LIB}, $ENV{PERLLIB} ){
next unless defined $i and length $i;
if( $^O =~ /win32/i ){
require Config;
my $sep = quotemeta $Config::Config{path_sep};
for my $p( grep length, split /$sep/, $i){
push @theINC, File::Spec->canonpath($p);
}
} else {
push @theINC, File::Spec->canonpath($i);
}
}
}
Sun Aug 24 09:23:11 2003spoon [...] cpan.org - Correspondence added
RT-Send-CC:
podmaster [...] cpan.org, spoon [...] cpan.org
[PODMASTER - Sun Aug 24 08:22:31 2003]:
Show quoted text
> Let's not forget about PERL5LIB and PERLLIB,
Why? The appropriate one will already be in @INC and correctly
parsed. There's no need to scan it twice.
Also, if PERL5LIB is set, PERLLIB should be ignored.
I can see a case for using File::Spec to fiddle with the
paths though.
cheers,
--
Iain.
Sun Aug 24 11:04:25 2003DROLSKY [...] cpan.org - Taken
Mon Aug 25 10:19:06 2003Guest - Correspondence added
Show quoted text
> > Let's not forget about PERL5LIB and PERLLIB,
>
> Why? The appropriate one will already be in @INC and correctly
> parsed. There's no need to scan it twice.
>
> Also, if PERL5LIB is set, PERLLIB should be ignored.
whoops, I guess it's been a while since I read `perldoc perlrun'
Show quoted text
> I can see a case for using File::Spec to fiddle with the
> paths though.
portability++
Mon Aug 25 20:08:00 2003DROLSKY [...] cpan.org - Fixed in 1.6 added
Mon Aug 25 20:08:19 2003DROLSKY [...] cpan.org - Status changed from 'new' to 'resolved'
RT for rt.cpan.org