fix: Make gapic-generator-python compatible with protobuf 3.14.0 (native namespace package) by vam-google · Pull Request #753 · googleapis/gapic-generator-python · GitHub
Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

fix: Make gapic-generator-python compatible with protobuf 3.14.0 (native namespace package)#753

Merged
vam-google merged 2 commits into
googleapis:masterfrom
vam-google:master
Jan 28, 2021
Merged

fix: Make gapic-generator-python compatible with protobuf 3.14.0 (native namespace package)#753
vam-google merged 2 commits into
googleapis:masterfrom
vam-google:master

Conversation

@vam-google

Copy link
Copy Markdown
Contributor

More details about Python namespace packaging here: https://packaging.python.org/guides/packaging-namespace-packages/#native-namespace-packages

The protobuf changes, which made this fix necessary are here:
protocolbuffers/protobuf#7902
protocolbuffers/protobuf#7908

The tracking bug for this issue (probably not the only one) googleapis/gapic-generator#3334

This is only part of the fix, for the proper fix other google-namespaced python packages must be migrated to pkgutil-style-namespace-packages and make sure they do not have namespace_packages in their setup.py file (an artifact from the legacy pkg_resources-stylepackages)

…kaged as native namespace package)

More details about Python namespace packaging here: https://packaging.python.org/guides/packaging-namespace-packages/#native-namespace-packages

The protobuf changes, which made this fix necessary are here:
protocolbuffers/protobuf#7902
protocolbuffers/protobuf#7908

The tracking bug for this issue (probably not the only one) googleapis/gapic-generator#3334

This is only part of the fix, for the proper fix other google-namespaced python packages must be changed as well.
@noahdietz

Copy link
Copy Markdown
Contributor

@busunkim96 busunkim96 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Vadym!

@codecov

codecov Bot commented Jan 28, 2021

Copy link
Copy Markdown

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants