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

Flip --incompatible_enforce_config_setting_visibility in .bazelrc to prevent regression #58832

Merged
merged 3 commits into from
Dec 13, 2022

Conversation

meteorcloudy
Copy link
Member

@google-ml-butler google-ml-butler bot added the size:XS CL Change Size: Extra Small label Dec 9, 2022
@meteorcloudy
Copy link
Member Author

/cc @learning-to-play

@meteorcloudy
Copy link
Member Author

Looks like tf_runtime needs to be fixed first.

@meteorcloudy
Copy link
Member Author

# ERROR: /tf/tensorflow/tensorflow/compiler/mlir/tfrt/jit/default/BUILD:16:16: in cc_library rule //tensorflow/compiler/mlir/tfrt/jit/default:tf_jitrt_query_of_death: alias '@tf_runtime//:linux_x86_64' referring to target '@tf_runtime//:linux_k8' is not visible from target '//tensorflow/compiler/mlir/tfrt/jit/default:tf_jitrt_query_of_death'. Check the visibility declaration of the former target if you think the dependency is legitimate
# ERROR: /tf/tensorflow/tensorflow/compiler/mlir/tfrt/jit/default/BUILD:16:16: Analysis of target '//tensorflow/compiler/mlir/tfrt/jit/default:tf_jitrt_query_of_death' failed

@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Dec 10, 2022
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Dec 10, 2022
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Dec 10, 2022
@meteorcloudy meteorcloudy changed the title Flip two incompatible flags in .bazelrc to prevent regression Flip --incompatible_enforce_config_setting_visibility in .bazelrc to prevent regression Dec 12, 2022
meteorcloudy added a commit to meteorcloudy/runtime that referenced this pull request Dec 12, 2022
copybara-service bot pushed a commit to tensorflow/runtime that referenced this pull request Dec 12, 2022
copybara-service bot pushed a commit to google/tsl that referenced this pull request Dec 13, 2022
…bazelrc to prevent regression

Imported from GitHub PR tensorflow/tensorflow#58832

Related: bazelbuild/bazel#12932 and bazelbuild/bazel#12933
Copybara import of the project:

--
b4edfd47bd9a0c0363a9feb1eb1b48d067e4644e by Yun Peng <pcloudy@google.com>:

Flip two incompatible flags in .bazelrc to prevent regression

--
c33c0dd6e5744393fe2b1909efc91dc19bd6dda1 by Yun Peng <pcloudy@google.com>:

Add visibility to more targets

--
34d1bfe1564f0905993c26a891f2385b83480e3d by Yun Peng <pcloudy@google.com>:

--incompatible_config_setting_private_default_visibility cannot be flipped yet

Merging this change closes #58832

PiperOrigin-RevId: 494895621
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Dec 13, 2022
…bazelrc to prevent regression

Imported from GitHub PR tensorflow/tensorflow#58832

Related: bazelbuild/bazel#12932 and bazelbuild/bazel#12933
Copybara import of the project:

--
b4edfd47bd9a0c0363a9feb1eb1b48d067e4644e by Yun Peng <pcloudy@google.com>:

Flip two incompatible flags in .bazelrc to prevent regression

--
c33c0dd6e5744393fe2b1909efc91dc19bd6dda1 by Yun Peng <pcloudy@google.com>:

Add visibility to more targets

--
34d1bfe1564f0905993c26a891f2385b83480e3d by Yun Peng <pcloudy@google.com>:

--incompatible_config_setting_private_default_visibility cannot be flipped yet

Merging this change closes #58832

PiperOrigin-RevId: 494895621
@copybara-service copybara-service bot merged commit bb4fdd2 into tensorflow:master Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review Pull request awaiting review ready to pull PR ready for merge process size:XS CL Change Size: Extra Small
Projects
PR Queue
  
Approved by Reviewer
Development

Successfully merging this pull request may close these issues.

None yet

4 participants