Skip to content

Remove Python2 from python_configure.bzl#28406

Merged
lidizheng merged 3 commits intogrpc:masterfrom
lidizheng:bazel-rm-py2
Dec 28, 2021
Merged

Remove Python2 from python_configure.bzl#28406
lidizheng merged 3 commits intogrpc:masterfrom
lidizheng:bazel-rm-py2

Conversation

@lidizheng
Copy link
Copy Markdown
Contributor

@lidizheng lidizheng commented Dec 22, 2021

As title.

Fixes #28026. Richard helped to offer some background about this issue. After discussion, we want to try remove the Python2-header-finding logic from our Bazel build.

This should unblock #28392.

@lidizheng lidizheng added lang/Python release notes: no Indicates if PR should not be in release notes labels Dec 22, 2021
@lidizheng lidizheng marked this pull request as ready for review December 23, 2021 18:20
@lidizheng
Copy link
Copy Markdown
Contributor Author

Known failure: Distribution Tests Python Linux: #28405

Copy link
Copy Markdown
Contributor

@gnossen gnossen left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for taking care of this!

@lidizheng lidizheng merged commit 9a593c7 into grpc:master Dec 28, 2021
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Dec 28, 2021
antoninbas added a commit to p4lang/p4runtime that referenced this pull request Jan 24, 2022
To include this fix: grpc/grpc#28406

We can upgrade to v1.44.0 when it is released.
antoninbas added a commit to p4lang/p4runtime that referenced this pull request Jan 24, 2022
* Update versions of dependencies for Bazel build

Our CI uses the latest available version of Bazel, which is now 5.0.
With this new Bazel release, the build started breaking, which required
upgrading dependencies to the following versions:
 * Protobuf: v3.18.1
 * gRPC: v1.43.0
 * googleapis: top-of-tree
 * rules_proto: top-of-tree
 * rules_go: v0.29.0

* Use gRPC v1.44.0-pre1

To include this fix: grpc/grpc#28406

We can upgrade to v1.44.0 when it is released.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/none imported Specifies if the PR has been imported to the internal repository lang/Python perf-change/none release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bazel BUILD fails during analysis phase when python2.7 binary is installed, but python2.7-dev is not.

2 participants