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

MSG_WAITALL problem on Windows 7 #103

Closed
pigay opened this issue Dec 10, 2017 · 5 comments
Closed

MSG_WAITALL problem on Windows 7 #103

pigay opened this issue Dec 10, 2017 · 5 comments

Comments

@pigay
Copy link
Contributor

pigay commented Dec 10, 2017

Hi,

I'm trying to use irodsclient on Windows 7. The Python I installed is 3.6.3.

Instead of ValueError like in #53, I get OSError in sock.recv()

Adding OSError in the list of caught exceptions is sufficient to have it work, but is it the good think to do?

Cheers,

Pierre

@adetorcy
Copy link
Contributor

Can you print the full traceback?

@pigay
Copy link
Contributor Author

pigay commented Dec 11, 2017

Here it is. sorry for the screenshot, I failed to use my VM copy/paste...

msg_waitall_tb

@adetorcy
Copy link
Contributor

That seems reasonable. I will look into this and #104 once version 0.7.0 is out. There may be other parts of the code that expect a *nix environment.
Windows support is not a priority at this point but I will try to make the code portable as much as possible.

@pigay
Copy link
Contributor Author

pigay commented Dec 14, 2017

I understand that Windows is not in your top priorities. Your help is much appreciated.

If you agree, I would like to propose PRs to fix the issues I encounter. You could accept/reject/discuss them at your convenience, after 0.7.0 is released for example.

@adetorcy
Copy link
Contributor

Yes, absolutely. Thank you.

pigay added a commit to pigay/python-irodsclient that referenced this issue Dec 15, 2017
adetorcy pushed a commit that referenced this issue Jan 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants