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.

Sign up for GitHub

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

Jump to bottom

gax-java/gax_java.bzl should use cfg = "exec" instead of cfg = "host" #2514

Closed
cushon opened this issue Feb 28, 2024 · 0 comments · Fixed by #2637
Closed

gax-java/gax_java.bzl should use cfg = "exec" instead of cfg = "host" #2514

cushon opened this issue Feb 28, 2024 · 0 comments · Fixed by #2637
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@cushon
Copy link
Contributor

cushon commented Feb 28, 2024

This .bzl file contains a reference to cfg = "host":

sdk-platform-java/gax-java/gax_java.bzl

Line 80 in 2de487b

cfg = "host",

The Bazel 'host' configuration is being renamed to 'exec', see: bazelbuild/bazel#17032
@cushon cushon mentioned this issue Feb 28, 2024
Closed
@mpeddada1 mpeddada1 added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. priority: p2 Moderately-important priority. Fix may not be included in next release. labels Feb 29, 2024
@cushon cushon mentioned this issue Apr 12, 2024
Merged
gcf-merge-on-green bot pushed a commit that referenced this issue Apr 22, 2024
@cushon
fix: replace cfg = "host" with cfg = "exec" (#2637)
6d673f3 
This is a no-op cleanup, 'host' is being renamed to 'exec', see: https://togithub.com/bazelbuild/bazel/issues/17032

Fixes #2514 ☕️
lqiu96 pushed a commit that referenced this issue May 22, 2024
@cushon @lqiu96
fix: replace cfg = "host" with cfg = "exec" (#2637)
e2565b2 
This is a no-op cleanup, 'host' is being renamed to 'exec', see: https://togithub.com/bazelbuild/bazel/issues/17032

Fixes #2514 ☕️
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
2 participants
@cushon @mpeddada1