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

change EAGAIN to EWOULDBLOCK to support Windows too #66

Closed
wants to merge 2 commits into from
Closed

change EAGAIN to EWOULDBLOCK to support Windows too #66

wants to merge 2 commits into from

Conversation

noxxi
Copy link

@noxxi noxxi commented Jan 9, 2015

This is the same problem described in libwww-perl/Net-HTTP#11 for Net::HTTP, but this time for libwww-perl. Both patches need to go together.

@karenetheridge
Copy link
Member

+1

@wchristian
Copy link

Any feedback on this? It's kind of important.

@Zarabozo
Copy link

Zarabozo commented Feb 9, 2015

Wow. This issue was driving me crazy! I had to patch both Net/HTTP/Methods.pm and LWP/Protocol/HTTP.pm. The only workaround without patching them is to use IO::Socket::SSL version 2.005, which is the last version still not supporting non-blocking sockets on Windows.

@karenetheridge
Copy link
Member

I will do a release of libwww-perl at the end of the day if I don't hear
anything from Mike Schilli.

On Mon, Feb 9, 2015 at 2:14 AM, Zarabozo notifications@github.com wrote:

Wow. This issue was driving me crazy! I had to patch both
Net/HTTP/Methods.pm and LWP/Protocol/HTTP.pm. The only workaround without
patching them is to use IO::Socket::SSL version 2.005, which is the last
version still not supporting non-blocking sockets on Windows.


Reply to this email directly or view it on GitHub
#66 (comment)
.

@karenetheridge
Copy link
Member

Merged; releasing imminently.

@Zarabozo
Copy link

@karenetheridge Are you able to do the same for Net::HTTP?

@karenetheridge
Copy link
Member

No, I never received comaint of that when the libwww-perl transfer
happened, sorry...

On Tue, Feb 10, 2015 at 2:02 AM, Zarabozo notifications@github.com wrote:

@karenetheridge https://github.com/karenetheridge Are you able to do
the same for Net::HTTP?


Reply to this email directly or view it on GitHub
#66 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants