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

Python 3 support #38

Closed
danklein10 opened this issue Jun 5, 2015 · 9 comments
Closed

Python 3 support #38

danklein10 opened this issue Jun 5, 2015 · 9 comments

Comments

@danklein10
Copy link

Is there a plan to add support for python 3?

It looks like there only should be slight modifications for that (such as handling import of xmlrpclib).

@advornic
Copy link

advornic commented Jun 9, 2015

+1

@chiesax
Copy link

chiesax commented Jul 21, 2015

then why not just do it? I think in the spirit of python3 should be renamed to jsonrpc. This will also avoid confusions.

@chiesax
Copy link

chiesax commented Jul 21, 2015

after some tentatives with python3, I don't think its just a small matter. Requires some effort, as the api of urllib has changed.

@chiesax
Copy link

chiesax commented Jul 22, 2015

somebody already did it! See tcalmant/jsonrpclib... its called jsonrpclib-pelix. Have not tested yet.

@danklein10
Copy link
Author

I have tested jsonrpclib-pelix with both Python 2 and 3 from the client side and Python 2 on the server side.
Overall, I found it stable and on top of Python 3 itself it offers some more features which might be useful, for example raising an app error instead of protocol error in case of an error raised by the server by the user.

@iten123
Copy link

iten123 commented Nov 17, 2015

+1, my project requires jsonrpclib's Python 3 support. Is there any plan doing this or encourage us to use jsonrpclib-pelix?

@chiesax
Copy link

chiesax commented Nov 18, 2015

We should probably ask to @joshmarshall...

@collinanderson
Copy link

See here for part of the status: #39 (comment)

@joshmarshall
Copy link
Owner

Should be addressed in 0.2.1

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

No branches or pull requests

6 participants