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

3.20.1 python 3.11 compatibility #17

Merged
merged 3 commits into from
Dec 6, 2022
Merged

3.20.1 python 3.11 compatibility #17

merged 3 commits into from
Dec 6, 2022

Conversation

cbouss
Copy link

@cbouss cbouss commented Dec 5, 2022

The PyFrameObject structure members have been removed from the public C API in python 3.11.

This adds a patch from upstream to make protobuf 3.20.1 compatible with python 3.11.

Patch adapted from: protocolbuffers/protobuf#10403

Tested that it builds on python 3.11.

Jira: https://anaconda.atlassian.net/browse/PKG-862

Copy link

@ELundby45 ELundby45 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

recipe/meta.yaml Outdated Show resolved Hide resolved
@anaconda-pkg-build
Copy link

Linter check found the following problems: The following problems have been found:

WARNING: /tmp/abs_16yb324y5b/clone/recipe/meta.yaml:57: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_16yb324y5b/clone/recipe/meta.yaml:59: missing_license_url: The recipe is missing a license_url
Warnings were found

@cbouss cbouss merged commit 059437e into master Dec 6, 2022
@cbouss cbouss deleted the 3.20.1_py311 branch December 6, 2022 14:35
@anaconda-pkg-build
Copy link

Linter check found the following problems: The following problems have been found:

WARNING: /tmp/abs_5fm6z17046/clone/recipe/meta.yaml:57: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_5fm6z17046/clone/recipe/meta.yaml:59: missing_license_url: The recipe is missing a license_url
Warnings were found

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.

4 participants