Skip to content

[build] Remove language tag from BUILD files#38501

Closed
ctiller wants to merge 3 commits intogrpc:masterfrom
ctiller:language
Closed

[build] Remove language tag from BUILD files#38501
ctiller wants to merge 3 commits intogrpc:masterfrom
ctiller:language

Conversation

@ctiller
Copy link
Member

@ctiller ctiller commented Jan 20, 2025

This is only ever set to C++, so removing it from bazel.

There are remnants left in cmakefile generation that I'm not planning on cleaning up (looked like some of the yaml/python pieces still use this)

@ctiller ctiller added the release notes: no Indicates if PR should not be in release notes label Jan 21, 2025
eugeneo pushed a commit to eugeneo/grpc that referenced this pull request Jan 23, 2025
This is only ever set to C++, so removing it from bazel.

There are remnants left in cmakefile generation that I'm not planning on cleaning up (looked like some of the yaml/python pieces still use this)

Closes grpc#38501

COPYBARA_INTEGRATE_REVIEW=grpc#38501 from ctiller:language c8aee06
PiperOrigin-RevId: 718198572
XuanWang-Amos pushed a commit to xrmx/grpc that referenced this pull request Jan 28, 2025
This is only ever set to C++, so removing it from bazel.

There are remnants left in cmakefile generation that I'm not planning on cleaning up (looked like some of the yaml/python pieces still use this)

Closes grpc#38501

COPYBARA_INTEGRATE_REVIEW=grpc#38501 from ctiller:language c8aee06
PiperOrigin-RevId: 718198572
paulosjca pushed a commit to paulosjca/grpc that referenced this pull request Feb 20, 2025
This is only ever set to C++, so removing it from bazel.

There are remnants left in cmakefile generation that I'm not planning on cleaning up (looked like some of the yaml/python pieces still use this)

Closes grpc#38501

COPYBARA_INTEGRATE_REVIEW=grpc#38501 from ctiller:language c8aee06
PiperOrigin-RevId: 718198572
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants