Re: LWPng patch to work with URI

• Messages sorted by: [ date ][ thread ][ subject ][ author ]
• Next message: Rick Tuinenburg: "Streaming Animated Gif"
• Previous message: Roderick Schertler: "Re: $^X rather than $Config{perlpath}"
• In reply to: Blair Zajac: "LWPng patch to work with URI"

Blair Zajac <bzajac@geostaff.com> writes:

> When I tried to use my WebFS::FileCopy routines they were failing.
> To fix the problem, I've patched LWPng to use the URI module instead
> of URL.  While this patch may not have picked up all the changes
> needed for LWPng, I was successfully able to run make test on LWPng
> and WebFS::FileCopy fine without any problems.

I am uploading LWPng-alpha-0.24 now with your patch. Thanks!

> One question: in URI::_generic.pm the following line appears:
> 
> *epath = \&path;
> 
> and in URI::URL.pm the following routine appears:
> 
> sub epath
> {
>     my $path = shift->SUPER::path(@_);
>     $path =~ s/;.*//;
>     $path;
> }
> 
> While they are almost identical, they are different.  Shouldn't just
> one be kept and the other discarded?

I guess I should kill the epath alias in URI::_generic.pm.  Done.

Regards,
Gisle

• Next message: Rick Tuinenburg: "Streaming Animated Gif"
• Previous message: Roderick Schertler: "Re: $^X rather than $Config{perlpath}"
• In reply to: Blair Zajac: "LWPng patch to work with URI"