Skip to content

Commit

Permalink
bazel: Layering check fixes
Browse files Browse the repository at this point in the history
These fixes are sufficient to get at least one module (pw_work_queue) to
build with layering check enabled.

Bug: 219091175
Change-Id: I3297f80186e717f128e7cfc7d97a3229f31f2dab
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/261092
Lint: Lint 🤖 <android-build-ayeaye@system.gserviceaccount.com>
Docs-Not-Needed: Ted Pudlik <tpudlik@google.com>
Commit-Queue: Ted Pudlik <tpudlik@google.com>
Reviewed-by: Armando Montanez <amontanez@google.com>
  • Loading branch information
tpudlik authored and CQ Bot Account committed Jan 17, 2025
1 parent a415024 commit 8bc7c12
Show file tree
Hide file tree
Showing 6 changed files with 16 additions and 5 deletions.
2 changes: 1 addition & 1 deletion pw_log_basic/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ cc_library(
":config_override",
":headers",
"//pw_log:pw_log.facade",
"//pw_string",
"//pw_string:builder",
"//pw_sys_io",
],
)
Expand Down
4 changes: 3 additions & 1 deletion pw_metric/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,11 @@ cc_library(
strip_include_prefix = "public",
deps = [
"//pw_assert",
"//pw_containers",
"//pw_containers:intrusive_list",
"//pw_log",
"//pw_preprocessor",
"//pw_span",
"//pw_tokenizer",
"//pw_tokenizer:base64",
],
)
Expand Down
2 changes: 2 additions & 0 deletions pw_string/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ cc_library(
":to_string",
":util",
"//pw_preprocessor",
"//pw_span",
"//pw_status",
],
)
Expand Down Expand Up @@ -94,6 +95,7 @@ cc_library(
":format",
":util",
"//pw_result",
"//pw_span",
"//pw_status",
"//third_party/fuchsia:stdcompat",
],
Expand Down
3 changes: 2 additions & 1 deletion pw_unit_test/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ cc_library(
"//pw_preprocessor",
"//pw_span",
"//pw_status",
"//pw_string",
"//pw_string:builder",
],
)

Expand Down Expand Up @@ -227,6 +227,7 @@ cc_library(
],
strip_include_prefix = "public",
deps = [
":event_handler",
":googletest_style_event_handler",
"//pw_preprocessor",
],
Expand Down
6 changes: 6 additions & 0 deletions pw_work_queue/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,11 @@ cc_library(
],
strip_include_prefix = "public",
deps = [
"//pw_assert:check",
"//pw_containers:inline_queue",
"//pw_function",
"//pw_metric:metric",
"//pw_span",
"//pw_status",
"//pw_sync:interrupt_spin_lock",
"//pw_sync:lock_annotations",
Expand All @@ -55,7 +57,11 @@ cc_library(
deps = [
":pw_work_queue",
":stl_test_thread",
":test_thread_header",
"//pw_function",
"//pw_log",
"//pw_sync:thread_notification",
"//pw_thread:thread",
"//pw_unit_test",
],
)
Expand Down
4 changes: 2 additions & 2 deletions third_party/fuchsia/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ cc_library(
cc_library(
name = "stdcompat",
srcs = [
"repo/sdk/lib/stdcompat/include/lib/stdcompat/bit.h",
"repo/sdk/lib/stdcompat/include/lib/stdcompat/internal/bit.h",
"repo/sdk/lib/stdcompat/include/lib/stdcompat/internal/constructors.h",
"repo/sdk/lib/stdcompat/include/lib/stdcompat/internal/exception.h",
"repo/sdk/lib/stdcompat/include/lib/stdcompat/internal/functional.h",
Expand All @@ -73,8 +73,8 @@ cc_library(
"repo/sdk/lib/stdcompat/include/lib/stdcompat/memory.h",
],
hdrs = [
"repo/sdk/lib/stdcompat/include/lib/stdcompat/bit.h",
"repo/sdk/lib/stdcompat/include/lib/stdcompat/functional.h",
"repo/sdk/lib/stdcompat/include/lib/stdcompat/internal/bit.h",
"repo/sdk/lib/stdcompat/include/lib/stdcompat/optional.h",
"repo/sdk/lib/stdcompat/include/lib/stdcompat/type_traits.h",
"repo/sdk/lib/stdcompat/include/lib/stdcompat/utility.h",
Expand Down

0 comments on commit 8bc7c12

Please sign in to comment.