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

Support for python3.10 #15

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

skazantsev
Copy link

@skazantsev skazantsev commented Oct 20, 2023

Hi Cyril,

First off, thank you for creating the package, after many years it's still being used in the Cloud9 IDE as a python debugger.

As mentioned in #13, the package fails to install on Python3.10 due to removal of PyThreadState.use_tracing (python/cpython#87926).

This pull request adds support for Python3.10 by conditionally setting PyThreadState.use_tracing or PyThreadState.cframe.use_tracing depending on the Python version.

I tested it on Python3.6 and 3.10.

I also bumped the version to 1.4.2, I don't know if that's acceptable or you'd want to set it to 1.5.0 or something.

Any chance this PR can be merged and released to PyPI?

Regards,
Sergey

@cmorisse
Copy link
Owner

cmorisse commented Oct 23, 2023

Hello Sergey,
Thanks for the patch and your kind word.
I will merge it next week as 1.5 and release it to pypi.
Do you allow me to add you to the authors list ?
Thanks again
Kind regards

@skazantsev
Copy link
Author

Awesome, thank you!

Do you allow me to add you to thé authors list ?

Sure, go ahead!

@cmorisse cmorisse changed the base branch from master to py31x-support October 31, 2023 09:35
@cmorisse cmorisse merged commit 3b73f71 into cmorisse:py31x-support Oct 31, 2023
@cmorisse
Copy link
Owner

Thank you very much !!!!!

@skazantsev skazantsev deleted the python3.10-support branch October 31, 2023 17:38
@skazantsev skazantsev restored the python3.10-support branch October 31, 2023 17:38
@skazantsev
Copy link
Author

Thank you for releasing it!

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.

2 participants