Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Which version of LWP is requiired? #7

Closed
tbaumann opened this issue Oct 10, 2013 · 1 comment
Closed

Which version of LWP is requiired? #7

tbaumann opened this issue Oct 10, 2013 · 1 comment
Assignees

Comments

@tbaumann
Copy link

Hi,

I use the LWP package that comes with (Centos|RHEL)5. But LWP-Protocol-Net-Curl uses some non-existing functions in LWP:UserAgent.

I removed those from the code and it seems to work now.
$ua->local_address and $ua->show_progres

You might want to test if they are defined before you use them. Looks like none of the two are actually really required of not available.

PS: Also making it a installable cpan module would be nice. ;)

creaktive added a commit that referenced this issue Oct 11, 2013
$ua->local_address available since LWP 5.834
$ua->show_progress available since LWP 5.815
@creaktive
Copy link
Owner

Thanks for reporting that!
BTW, we're already on the CPAN: https://metacpan.org/module/LWP::Protocol::Net::Curl
So you could just cpan install LWP::Protocol::Net::Curl :)
However, this GitHub repo is not the same format as the CPAN distro because I use Dist::Zilla for bookkeeping.

@ghost ghost assigned creaktive Jan 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants