Skip to content

Commit

Permalink
Disable broken layering_check for grpc
Browse files Browse the repository at this point in the history
This has been done upstream in the BCR for newer versions, but upgrading
right now requires more changes.
  • Loading branch information
keith committed May 21, 2024
1 parent 9de1fa1 commit 7e0129a
Show file tree
Hide file tree
Showing 3 changed files with 56 additions and 28 deletions.
8 changes: 8 additions & 0 deletions MODULE.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,14 @@ local_path_override(
path = "./third_party/googleapis",
)

single_version_override(
module_name = "grpc",
patch_strip = 1,
patches = [
"//third_party/grpc:00_disable_layering_check.patch",
],
)

# The following Bazel modules are not direct dependencies for building Bazel,
# but are required for visibility from DIST_ARCHIVE_REPOS in repositories.bzl
bazel_dep(name = "apple_support", version = "1.8.1")
Expand Down
63 changes: 35 additions & 28 deletions MODULE.bazel.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 13 additions & 0 deletions third_party/grpc/00_disable_layering_check.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
diff --git a/BUILD b/BUILD
index 83e6a3f740..b27df5b71e 100644
--- a/BUILD
+++ b/BUILD
@@ -29,7 +29,7 @@ licenses(["reciprocal"])
package(
default_visibility = ["//visibility:public"],
features = [
- "layering_check",
+ "-layering_check",
"-parse_headers",
],
)

0 comments on commit 7e0129a

Please sign in to comment.