workspace: Adding bazel build formatter/linter
Adds in buildifier bazel targets that autoformat/lint the entire
WORKSPACE. This can be invoked by running;
'bazel run //:buildifier'
or
'bazel run //:buildifier_test'
Change-Id: Iaaf7f6241e29894eca94aa6b5d0ca3b33255b7f7
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/36160
Reviewed-by: Keir Mierle <keir@google.com>
Commit-Queue: Keir Mierle <keir@google.com>
diff --git a/BUILD b/BUILD
index 31c2571..35c1531 100644
--- a/BUILD
+++ b/BUILD
@@ -12,5 +12,50 @@
# License for the specific language governing permissions and limitations under
# the License.
+load(
+ "@com_github_bazelbuild_buildtools//buildifier:def.bzl",
+ "buildifier",
+ "buildifier_test",
+)
+
licenses(["notice"]) # Apache License 2.0
-exports_files(["tsconfig.json"], visibility = ["//:__subpackages__"])
+
+exports_files(
+ ["tsconfig.json"],
+ visibility = ["//:__subpackages__"],
+)
+
+# Fix all Bazel relevant files.
+buildifier(
+ name = "buildifier",
+ # Ignore gn and CIPD outputs in formatting.
+ # NOTE: These globs are not Bazel native and are passed directly
+ # through to the buildifier binary.
+ # TODO: Remove these globs when
+ # https://github.com/bazelbuild/buildtools/issues/623 is addressed.
+ exclude_patterns = [
+ "./.environment/**/*",
+ "./.presubmit/**/*",
+ "./.out/**/*",
+ ],
+)
+
+# Test to ensure all Bazel build files are properly formatted.
+buildifier_test(
+ name = "buildifier_test",
+ srcs = glob(
+ [
+ "**/*.bazel",
+ "**/*.bzl",
+ "**/*.oss",
+ "**/*.sky",
+ "**/BUILD",
+ ],
+ # Node modules do not play nice with buildifier. Exclude these
+ # generated Bazel files from format testing.
+ exclude = ["**/node_modules/**/*"],
+ ) + ["WORKSPACE"],
+ diff_command = "diff -u",
+ mode = "diff",
+ verbose = True,
+)
diff --git a/WORKSPACE b/WORKSPACE
index 573d3f2..677be52 100644
--- a/WORKSPACE
+++ b/WORKSPACE
@@ -17,10 +17,45 @@
managed_directories = {"@npm": ["node_modules"]},
)
-# Set up build_bazel_rules_nodejs.
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
load("@bazel_tools//tools/build_defs/repo:git.bzl", "git_repository")
+# Set up Starlark library
+# Required by: io_bazel_rules_go, com_google_protobuf.
+# Used in modules: None.
+# This must be instantiated before com_google_protobuf as protobuf_deps() pulls
+# in an older version of bazel_skylib. However io_bazel_rules_go requires a
+# newer version.
+http_archive(
+ name = "bazel_skylib",
+ sha256 = "1c531376ac7e5a180e0237938a2536de0c54d93f5c278634818e0efc952dd56c",
+ urls = [
+ "https://github.com/bazelbuild/bazel-skylib/releases/download/1.0.3/bazel-skylib-1.0.3.tar.gz",
+ "https://mirror.bazel.build/github.com/bazelbuild/bazel-skylib/releases/download/1.0.3/bazel-skylib-1.0.3.tar.gz",
+ ],
+)
+
+load("@bazel_skylib//:workspace.bzl", "bazel_skylib_workspace")
+
+bazel_skylib_workspace()
+
+# Set up protobuf rules
+# Required by: pigweed, com_github_bazelbuild_buildtools.
+# Used in modules: //pw_protobuf.
+http_archive(
+ name = "com_google_protobuf",
+ sha256 = "71030a04aedf9f612d2991c1c552317038c3c5a2b578ac4745267a45e7037c29",
+ strip_prefix = "protobuf-3.12.3",
+ url = "https://github.com/protocolbuffers/protobuf/archive/v3.12.3.tar.gz",
+)
+
+load("@com_google_protobuf//:protobuf_deps.bzl", "protobuf_deps")
+
+protobuf_deps()
+
+# Set up build_bazel_rules_nodejs.
+# Required by: pigweed.
+# Used in modules: //pw_web_ui.
http_archive(
name = "build_bazel_rules_nodejs",
sha256 = "4952ef879704ab4ad6729a29007e7094aef213ea79e9f2e94cbe1c9a753e63ef",
@@ -65,6 +100,8 @@
)
# Setup embedded C/C++ toolchains.
+# Required by: pigweed.
+# Used in modules: //pw_polyfill, //pw_build (all pw_cc* targets).
git_repository(
name = "bazel_embedded",
commit = "89c05fa415218abd2e24fa7016cb7903317d606b",
@@ -111,7 +148,7 @@
register_gcc_arm_none_toolchain()
-# Fetch LLVM/Clang compiler and register for toolchain resolution.
+# Fetch LLVM/Clang host compiler and register for toolchain resolution.
load(
"@bazel_embedded//toolchains/compilers/llvm:llvm_repository.bzl",
"llvm_repository",
@@ -128,4 +165,53 @@
register_clang_toolchain()
+# Registers platforms for use with toolchain resolution
register_execution_platforms("//pw_build/platforms:all")
+
+# Setup Golang toolchain rules
+# Required by: bazel_gazelle, com_github_bazelbuild_buildtools.
+# Used in modules: None.
+http_archive(
+ name = "io_bazel_rules_go",
+ sha256 = "d1ffd055969c8f8d431e2d439813e42326961d0942bdf734d2c95dc30c369566",
+ urls = [
+ "https://mirror.bazel.build/github.com/bazelbuild/rules_go/releases/download/v0.24.5/rules_go-v0.24.5.tar.gz",
+ "https://github.com/bazelbuild/rules_go/releases/download/v0.24.5/rules_go-v0.24.5.tar.gz",
+ ],
+)
+
+load(
+ "@io_bazel_rules_go//go:deps.bzl",
+ "go_register_toolchains",
+ "go_rules_dependencies",
+)
+
+go_rules_dependencies()
+
+go_register_toolchains()
+
+# Setup bazel package manager for golang
+# Required by: com_github_bazelbuild_buildtools.
+# Used in modules: None
+http_archive(
+ name = "bazel_gazelle",
+ sha256 = "b85f48fa105c4403326e9525ad2b2cc437babaa6e15a3fc0b1dbab0ab064bc7c",
+ urls = [
+ "https://mirror.bazel.build/github.com/bazelbuild/bazel-gazelle/releases/download/v0.22.2/bazel-gazelle-v0.22.2.tar.gz",
+ "https://github.com/bazelbuild/bazel-gazelle/releases/download/v0.22.2/bazel-gazelle-v0.22.2.tar.gz",
+ ],
+)
+
+load("@bazel_gazelle//:deps.bzl", "gazelle_dependencies")
+
+gazelle_dependencies()
+
+# Setup bazel buildtools (bazel linter and formatter)
+# Required by: pigweed
+# Used in modules: //:all (bazel specific tools).
+http_archive(
+ name = "com_github_bazelbuild_buildtools",
+ sha256 = "c28eef4d30ba1a195c6837acf6c75a4034981f5b4002dda3c5aa6e48ce023cf1",
+ strip_prefix = "buildtools-4.0.1",
+ url = "https://github.com/bazelbuild/buildtools/archive/4.0.1.tar.gz",
+)
diff --git a/pw_assert_log/BUILD b/pw_assert_log/BUILD
index 25768d1..fa2eb93 100644
--- a/pw_assert_log/BUILD
+++ b/pw_assert_log/BUILD
@@ -23,6 +23,9 @@
pw_cc_library(
name = "headers",
+ srcs = [
+ "assert_log.cc",
+ ],
hdrs = [
"public/pw_assert_log/assert_log.h",
"public_overrides/pw_assert_backend/assert_backend.h",
@@ -31,9 +34,6 @@
"public",
"public_overrides",
],
- srcs = [
- "assert_log.cc",
- ],
deps = [
"//pw_preprocessor",
],
diff --git a/pw_base64/BUILD b/pw_base64/BUILD
index f30a5bc..d1ea9af 100644
--- a/pw_base64/BUILD
+++ b/pw_base64/BUILD
@@ -39,8 +39,8 @@
pw_cc_test(
name = "base64_test",
srcs = [
- "base64_test_c.c",
"base64_test.cc",
+ "base64_test_c.c",
],
deps = [
":pw_base64",
diff --git a/pw_blob_store/BUILD b/pw_blob_store/BUILD
index 0ff8d86..c426cd9 100644
--- a/pw_blob_store/BUILD
+++ b/pw_blob_store/BUILD
@@ -24,7 +24,7 @@
pw_cc_library(
name = "pw_blob_store",
- srcs = [ "blob_store.cc" ],
+ srcs = ["blob_store.cc"],
hdrs = [
"public/pw_blob_store/blob_store.h",
],
@@ -69,6 +69,7 @@
"//pw_unit_test",
],
)
+
pw_cc_test(
name = "blob_store_deferred_write_test",
srcs = [
diff --git a/pw_blob_store/size_report/BUILD b/pw_blob_store/size_report/BUILD
index 4811b4f..ca27d3f 100644
--- a/pw_blob_store/size_report/BUILD
+++ b/pw_blob_store/size_report/BUILD
@@ -28,8 +28,8 @@
"//pw_assert",
"//pw_bloat:bloat_this_binary",
"//pw_kvs",
- "//pw_kvs:flash_test_partition",
"//pw_kvs:fake_flash_12_byte_partition",
+ "//pw_kvs:flash_test_partition",
"//pw_log",
],
)
@@ -42,8 +42,8 @@
"//pw_bloat:bloat_this_binary",
"//pw_blob_store",
"//pw_kvs",
- "//pw_kvs:flash_test_partition",
"//pw_kvs:fake_flash_12_byte_partition",
+ "//pw_kvs:flash_test_partition",
"//pw_log",
],
)
@@ -56,8 +56,8 @@
"//pw_bloat:bloat_this_binary",
"//pw_blob_store",
"//pw_kvs",
- "//pw_kvs:flash_test_partition",
"//pw_kvs:fake_flash_12_byte_partition",
+ "//pw_kvs:flash_test_partition",
"//pw_log",
],
)
diff --git a/pw_bytes/size_report/BUILD b/pw_bytes/size_report/BUILD
index d58feea..957c29b 100644
--- a/pw_bytes/size_report/BUILD
+++ b/pw_bytes/size_report/BUILD
@@ -28,4 +28,4 @@
"//pw_bloat:bloat_this_binary",
"//pw_bytes",
],
-)
\ No newline at end of file
+)
diff --git a/pw_chrono/BUILD b/pw_chrono/BUILD
index ea76d5b..cce73aa 100644
--- a/pw_chrono/BUILD
+++ b/pw_chrono/BUILD
@@ -35,14 +35,14 @@
pw_cc_library(
name = "system_clock_facade",
+ srcs = [
+ "system_clock.cc",
+ ],
hdrs = [
"public/pw_chrono/internal/system_clock_macros.h",
"public/pw_chrono/system_clock.h",
],
includes = ["public"],
- srcs = [
- "system_clock.cc"
- ],
deps = [
":epoch",
PW_CHRONO_SYSTEM_CLOCK_BACKEND + "_headers",
@@ -61,7 +61,7 @@
pw_cc_library(
name = "system_clock_backend",
deps = [
- PW_CHRONO_SYSTEM_CLOCK_BACKEND,
+ PW_CHRONO_SYSTEM_CLOCK_BACKEND,
],
)
diff --git a/pw_containers/BUILD b/pw_containers/BUILD
index 99926f4..d2030d4 100644
--- a/pw_containers/BUILD
+++ b/pw_containers/BUILD
@@ -26,14 +26,13 @@
name = "pw_containers",
deps = [
":flat_map",
- ":vector",
":intrusive_list",
+ ":vector",
],
)
pw_cc_library(
name = "intrusive_list",
- deps = [ "//pw_assert" ],
srcs = [
"intrusive_list.cc",
"public/pw_containers/internal/intrusive_list_impl.h",
@@ -42,6 +41,7 @@
"public/pw_containers/intrusive_list.h",
],
includes = ["public"],
+ deps = ["//pw_assert"],
)
pw_cc_library(
diff --git a/pw_cpu_exception_cortex_m/BUILD b/pw_cpu_exception_cortex_m/BUILD
index a39d11c..ace1574 100644
--- a/pw_cpu_exception_cortex_m/BUILD
+++ b/pw_cpu_exception_cortex_m/BUILD
@@ -25,30 +25,30 @@
pw_cc_library(
name = "support_armv7m",
- includes = ["public"],
- deps = [
- "//pw_preprocessor",
- "//pw_string",
- "//pw_log",
- ],
- hdrs = [ "public/pw_cpu_exception_cortex_m/cpu_state.h" ],
srcs = [
"cpu_state.cc",
"pw_cpu_exception_cortex_m_private/cortex_m_constants.h",
],
+ hdrs = ["public/pw_cpu_exception_cortex_m/cpu_state.h"],
+ includes = ["public"],
+ deps = [
+ "//pw_log",
+ "//pw_preprocessor",
+ "//pw_string",
+ ],
)
pw_cc_library(
name = "proto_dump_armv7m",
+ srcs = ["proto_dump.cc"],
+ hdrs = ["public/pw_cpu_exception_cortex_m/proto_dump.h"],
deps = [
- ":support_armv7m",
":cpu_state_protos",
+ ":support_armv7m",
"//pw_protobuf",
"//pw_status",
"//pw_stream",
],
- hdrs = ["public/pw_cpu_exception_cortex_m/proto_dump.h"],
- srcs = ["proto_dump.cc"],
)
proto_library(
@@ -61,6 +61,10 @@
# but provide preprocessor defines to enable/disable architecture specific code.
pw_cc_library(
name = "cpu_exception_armv7m",
+ srcs = [
+ "entry.cc",
+ "pw_cpu_exception_cortex_m_private/cortex_m_constants.h",
+ ],
deps = [
":proto_dump_armv7m",
":support_armv7m",
@@ -68,10 +72,6 @@
"//pw_cpu_exception",
"//pw_preprocessor",
],
- srcs = [
- "entry.cc",
- "pw_cpu_exception_cortex_m_private/cortex_m_constants.h",
- ],
)
pw_cc_test(
diff --git a/pw_hdlc/rpc_example/BUILD b/pw_hdlc/rpc_example/BUILD
index b2a48a2..e919349 100644
--- a/pw_hdlc/rpc_example/BUILD
+++ b/pw_hdlc/rpc_example/BUILD
@@ -31,8 +31,7 @@
deps = [
"//pw_hdlc",
"//pw_hdlc:pw_rpc",
- "//pw_rpc:server",
"//pw_log",
+ "//pw_rpc:server",
],
)
-
diff --git a/pw_hex_dump/BUILD b/pw_hex_dump/BUILD
index c0e21c9..9398c4b 100644
--- a/pw_hex_dump/BUILD
+++ b/pw_hex_dump/BUILD
@@ -24,11 +24,6 @@
pw_cc_library(
name = "pw_hex_dump",
- deps = [
- "//pw_bytes",
- "//pw_span",
- "//pw_status",
- ],
srcs = [
"hex_dump.cc",
],
@@ -36,9 +31,13 @@
"public/pw_hex_dump/hex_dump.h",
],
includes = ["public"],
+ deps = [
+ "//pw_bytes",
+ "//pw_span",
+ "//pw_status",
+ ],
)
-
pw_cc_test(
name = "hex_dump_test",
srcs = [
diff --git a/pw_i2c/BUILD b/pw_i2c/BUILD
index 2343bdc..8831e42 100644
--- a/pw_i2c/BUILD
+++ b/pw_i2c/BUILD
@@ -24,13 +24,13 @@
pw_cc_library(
name = "address",
+ srcs = [
+ "address.cc",
+ ],
hdrs = [
"public/pw_i2c/address.h",
],
includes = ["public"],
- srcs = [
- "address.cc",
- ],
deps = [
"//pw_assert",
],
diff --git a/pw_interrupt/BUILD b/pw_interrupt/BUILD
index b1f738b..b25443e 100644
--- a/pw_interrupt/BUILD
+++ b/pw_interrupt/BUILD
@@ -26,13 +26,13 @@
pw_cc_library(
name = "context_facade",
+ srcs = [
+ "context.cc",
+ ],
hdrs = [
"public/pw_interrupt/context.h",
],
includes = ["public"],
- srcs = [
- "context.cc"
- ],
deps = [
PW_INTERRUPT_CONTEXT_BACKEND + "_headers",
],
@@ -49,6 +49,6 @@
pw_cc_library(
name = "context_backend",
deps = [
- PW_INTERRUPT_CONTEXT_BACKEND,
+ PW_INTERRUPT_CONTEXT_BACKEND,
],
)
diff --git a/pw_interrupt_cortex_m/BUILD b/pw_interrupt_cortex_m/BUILD
index 02bd1f0..579a1e5 100644
--- a/pw_interrupt_cortex_m/BUILD
+++ b/pw_interrupt_cortex_m/BUILD
@@ -27,7 +27,7 @@
"public/pw_interrupt_cortex_m/context_inline.h",
"public_overrides/pw_interrupt_backend/context_inline.h",
],
- copts = [ "-DPW_INTERRUPT_CORTEX_M_ARMV7M=1" ],
+ copts = ["-DPW_INTERRUPT_CORTEX_M_ARMV7M=1"],
includes = [
"public",
"public_overrides",
@@ -36,7 +36,7 @@
pw_cc_library(
name = "context_armv7m",
- copts = [ "-DPW_INTERRUPT_CORTEX_M_ARMV7M=1" ],
+ copts = ["-DPW_INTERRUPT_CORTEX_M_ARMV7M=1"],
deps = [
":context_armv7m_headers",
"//pw_interrupt:context_facade",
@@ -49,7 +49,7 @@
"public/pw_interrupt_cortex_m/context.h",
"public_overrides/pw_interrupt_backend/context_backend.h",
],
- copts = [ "-DPW_INTERRUPT_CORTEX_M_ARMV8M=1" ],
+ copts = ["-DPW_INTERRUPT_CORTEX_M_ARMV8M=1"],
includes = [
"public",
"public_overrides",
@@ -58,7 +58,7 @@
pw_cc_library(
name = "context_armv8m",
- copts = [ "-DPW_INTERRUPT_CORTEX_M_ARMV8M=1" ],
+ copts = ["-DPW_INTERRUPT_CORTEX_M_ARMV8M=1"],
deps = [
":context_armv8m_headers",
"//pw_interrupt:context_facade",
diff --git a/pw_kvs/size_report/BUILD b/pw_kvs/size_report/BUILD
index bd752f3..3dc48b4 100644
--- a/pw_kvs/size_report/BUILD
+++ b/pw_kvs/size_report/BUILD
@@ -38,8 +38,8 @@
"//pw_assert",
"//pw_bloat:bloat_this_binary",
"//pw_kvs",
- "//pw_kvs:flash_test_partition",
"//pw_kvs:fake_flash_12_byte_partition",
+ "//pw_kvs:flash_test_partition",
"//pw_log",
],
)
@@ -51,8 +51,8 @@
"//pw_assert",
"//pw_bloat:bloat_this_binary",
"//pw_kvs",
- "//pw_kvs:flash_test_partition",
"//pw_kvs:fake_flash_12_byte_partition",
+ "//pw_kvs:flash_test_partition",
"//pw_log",
],
)
diff --git a/pw_log_multisink/BUILD b/pw_log_multisink/BUILD
index 979c752..8b87b64 100644
--- a/pw_log_multisink/BUILD
+++ b/pw_log_multisink/BUILD
@@ -24,8 +24,12 @@
pw_cc_library(
name = "pw_log_queue",
- srcs = [ "log_queue.cc" ],
- includes = [ "public" ],
+ srcs = ["log_queue.cc"],
+ hdrs = [
+ "public/pw_log_multisink/log_queue.h",
+ "public/pw_log_multisink/sink.h",
+ ],
+ includes = ["public"],
deps = [
"//pw_bytes",
"//pw_containers",
@@ -34,10 +38,6 @@
"//pw_ring_buffer",
"//pw_status",
],
- hdrs = [
- "public/pw_log_multisink/log_queue.h",
- "public/pw_log_multisink/sink.h",
- ]
)
pw_cc_test(
diff --git a/pw_log_rpc/BUILD b/pw_log_rpc/BUILD
index a779851..2acb090 100644
--- a/pw_log_rpc/BUILD
+++ b/pw_log_rpc/BUILD
@@ -24,15 +24,15 @@
pw_cc_library(
name = "pw_logs",
- srcs = [ "logs_rpc.cc" ],
- includes = [ "public" ],
+ srcs = ["logs_rpc.cc"],
+ hdrs = ["public/pw_log_rpc/logs_rpc.h"],
+ includes = ["public"],
deps = [
"//pw_bytes",
"//pw_result",
"//pw_ring_buffer",
"//pw_status",
],
- hdrs = [ "public/pw_log_rpc/logs_rpc.h" ]
)
pw_cc_test(
diff --git a/pw_log_sink/BUILD b/pw_log_sink/BUILD
index 19306ab..f10373d 100644
--- a/pw_log_sink/BUILD
+++ b/pw_log_sink/BUILD
@@ -24,8 +24,14 @@
pw_cc_library(
name = "pw_log_sink",
- srcs = [ "log_sink.cc" ],
- includes = [ "public" ],
+ srcs = ["log_sink.cc"],
+ hdrs = [
+ "public/pw_log_sink/log_sink.h",
+ "public/pw_log_sink/multisink_adapter.h",
+ "public/pw_log_sink/sink.h",
+ "public_overrides/pw_log_backend/log_backend.h",
+ ],
+ includes = ["public"],
deps = [
"//pw_bytes",
"//pw_log:facade",
@@ -34,20 +40,14 @@
"//pw_status",
"//pw_sync:interrupt_spin_lock",
],
- hdrs = [
- "public/pw_log_sink/log_sink.h",
- "public/pw_log_sink/multisink_adapter.h",
- "public/pw_log_sink/sink.h",
- "public_overrides/pw_log_backend/log_backend.h",
- ]
)
pw_cc_test(
name = "pw_log_sink_test",
- srcs = [ "log_sink_test.cc" ],
+ srcs = ["log_sink_test.cc"],
deps = [
":pw_log_sink",
"//pw_log:protos.pwpb",
"//pw_unit_test",
- ]
+ ],
)
diff --git a/pw_malloc/BUILD b/pw_malloc/BUILD
index f3ab4f3..b7548b0 100644
--- a/pw_malloc/BUILD
+++ b/pw_malloc/BUILD
@@ -20,6 +20,7 @@
package(default_visibility = ["//visibility:public"])
licenses(["notice"]) # Apache License 2.0
+
# TODO(pwbug/101): Need to add support for facades/backends to Bazel.
PW_MALLOC_BACKEND = "//pw_malloc_freelist"
diff --git a/pw_metric/BUILD b/pw_metric/BUILD
index 051fc27..50a0343 100644
--- a/pw_metric/BUILD
+++ b/pw_metric/BUILD
@@ -24,12 +24,12 @@
pw_cc_library(
name = "metric",
+ srcs = ["metric.cc"],
hdrs = [
- "public/pw_metric/metric.h",
"public/pw_metric/global.h",
+ "public/pw_metric/metric.h",
],
includes = ["public"],
- srcs = [ "metric.cc" ],
deps = [
"//pw_assert",
"//pw_containers",
@@ -41,10 +41,10 @@
pw_cc_library(
name = "global",
+ srcs = ["global.cc"],
hdrs = [
"public/pw_metric/global.h",
],
- srcs = [ "global.cc" ],
deps = [
":metric",
],
@@ -52,10 +52,10 @@
pw_cc_library(
name = "metric_service_nanopb",
+ srcs = ["metric_service_nanopb.cc"],
hdrs = [
"public/pw_metric/metric_service_nanopb.h",
],
- srcs = [ "metric_service_nanopb.cc" ],
deps = [
":metric",
],
diff --git a/pw_metric/size_report/BUILD b/pw_metric/size_report/BUILD
index f7693a9..2b53929 100644
--- a/pw_metric/size_report/BUILD
+++ b/pw_metric/size_report/BUILD
@@ -25,9 +25,9 @@
name = "base",
srcs = ["base.cc"],
deps = [
+ "//pw_assert",
"//pw_bloat:bloat_this_binary",
"//pw_log",
- "//pw_assert",
],
)
@@ -35,10 +35,10 @@
name = "one_metric",
srcs = ["one_metric.cc"],
deps = [
- "//pw_bloat:bloat_this_binary",
- "//pw_metric",
- "//pw_log",
"//pw_assert",
+ "//pw_bloat:bloat_this_binary",
+ "//pw_log",
+ "//pw_metric",
],
)
@@ -46,10 +46,10 @@
name = "dump",
srcs = ["dump.cc"],
deps = [
- "//pw_bloat:bloat_this_binary",
- "//pw_metric",
- "//pw_log",
"//pw_assert",
+ "//pw_bloat:bloat_this_binary",
+ "//pw_log",
+ "//pw_metric",
],
)
@@ -57,9 +57,9 @@
name = "more_metrics",
srcs = ["more_metrics.cc"],
deps = [
- "//pw_bloat:bloat_this_binary",
- "//pw_metric",
- "//pw_log",
"//pw_assert",
+ "//pw_bloat:bloat_this_binary",
+ "//pw_log",
+ "//pw_metric",
],
)
diff --git a/pw_multisink/BUILD b/pw_multisink/BUILD
index de50a3c..11f772c 100644
--- a/pw_multisink/BUILD
+++ b/pw_multisink/BUILD
@@ -24,8 +24,15 @@
pw_cc_library(
name = "pw_multisink",
- srcs = [ "drain.cc", "multisink.cc" ],
- includes = [ "public" ],
+ srcs = [
+ "drain.cc",
+ "multisink.cc",
+ ],
+ hdrs = [
+ "public/pw_multisink/drain.h",
+ "public/pw_multisink/multisink.h",
+ ],
+ includes = ["public"],
deps = [
"//pw_assert",
"//pw_bytes",
@@ -33,10 +40,6 @@
"//pw_ring_buffer",
"//pw_varint",
],
- hdrs = [
- "public/pw_multisink/drain.h",
- "public/pw_multisink/multisink.h",
- ]
)
pw_cc_test(
diff --git a/pw_persistent_ram/BUILD b/pw_persistent_ram/BUILD
index 1cde0cd..e1e89c4 100644
--- a/pw_persistent_ram/BUILD
+++ b/pw_persistent_ram/BUILD
@@ -24,12 +24,12 @@
pw_cc_library(
name = "pw_persistent_ram",
+ srcs = ["persistent_buffer.cc"],
hdrs = [
"public/pw_persistent_ram/persistent.h",
"public/pw_persistent_ram/persistent_buffer.h",
],
includes = ["public"],
- srcs = ["persistent_buffer.cc"],
deps = [
"//pw_assert",
"//pw_bytes",
@@ -56,7 +56,7 @@
],
deps = [
":pw_persistent_ram",
- "//pw_unit_test",
"//pw_random",
+ "//pw_unit_test",
],
)
diff --git a/pw_presubmit/docs.rst b/pw_presubmit/docs.rst
index 921cb35..2173f6e 100644
--- a/pw_presubmit/docs.rst
+++ b/pw_presubmit/docs.rst
@@ -238,4 +238,4 @@
---------------------
The ``pw_presubmit.format_code`` module formats supported source files using
external code format tools. The file ``format_code.py`` can be invoked directly
-from the command line or from ``pw`` as ``pw format``.
+from the command line or from ``pw`` as ``pw format``.
\ No newline at end of file
diff --git a/pw_result/size_report/BUILD b/pw_result/size_report/BUILD
index 1a67937..1c31ed8 100644
--- a/pw_result/size_report/BUILD
+++ b/pw_result/size_report/BUILD
@@ -25,8 +25,8 @@
name = "pointer_simple",
srcs = ["pointer_simple.cc"],
deps = [
- "//pw_result",
"//pw_log",
+ "//pw_result",
],
)
@@ -34,8 +34,8 @@
name = "result_simple",
srcs = ["result_simple.cc"],
deps = [
- "//pw_result",
"//pw_log",
+ "//pw_result",
],
)
@@ -43,8 +43,8 @@
name = "pointer_noinline",
srcs = ["pointer_noinline.cc"],
deps = [
- "//pw_result",
"//pw_log",
+ "//pw_result",
],
)
@@ -52,8 +52,8 @@
name = "result_noinline",
srcs = ["result_noinline.cc"],
deps = [
- "//pw_result",
"//pw_log",
+ "//pw_result",
],
)
@@ -61,8 +61,8 @@
name = "pointer_read",
srcs = ["pointer_read.cc"],
deps = [
- "//pw_result",
"//pw_log",
+ "//pw_result",
"//pw_span",
],
)
@@ -71,8 +71,8 @@
name = "result_read",
srcs = ["result_read.cc"],
deps = [
- "//pw_result",
"//pw_log",
+ "//pw_result",
"//pw_span",
],
)
diff --git a/pw_router/BUILD b/pw_router/BUILD
index f6f33c6..b284aff 100644
--- a/pw_router/BUILD
+++ b/pw_router/BUILD
@@ -24,8 +24,8 @@
pw_cc_library(
name = "static_router",
- hdrs = ["public/pw_router/static_router.h"],
srcs = ["static_router.cc"],
+ hdrs = ["public/pw_router/static_router.h"],
deps = [
":egress",
":packet_parser",
diff --git a/pw_status/BUILD b/pw_status/BUILD
index 6d28e38..c4089d2 100644
--- a/pw_status/BUILD
+++ b/pw_status/BUILD
@@ -36,8 +36,8 @@
pw_cc_test(
name = "status_test",
srcs = [
- "status_test_c.c",
"status_test.cc",
+ "status_test_c.c",
],
deps = [
":pw_status",
diff --git a/pw_sync/BUILD b/pw_sync/BUILD
index e0a4741..11e793a 100644
--- a/pw_sync/BUILD
+++ b/pw_sync/BUILD
@@ -24,20 +24,24 @@
# TODO(pwbug/101): Need to add support for facades/backends to Bazel.
PW_SYNC_BINARY_SEMAPHORE_BACKEND = "//pw_sync_stl:binary_semaphore"
+
PW_SYNC_COUNTING_SEMAPHORE_BACKEND = "//pw_sync_stl:counting_semaphore"
+
PW_SYNC_MUTEX_BACKEND = "//pw_sync_stl:mutex"
+
PW_SYNC_TIMED_MUTEX_BACKEND = "//pw_sync_stl:timed_mutex"
+
PW_SYNC_INTERRUPT_SPIN_LOCK_BACKEND = "//pw_sync_stl:interrupt_spin_lock"
pw_cc_library(
name = "binary_semaphore_facade",
+ srcs = [
+ "binary_semaphore.cc",
+ ],
hdrs = [
"public/pw_sync/binary_semaphore.h",
],
includes = ["public"],
- srcs = [
- "binary_semaphore.cc"
- ],
deps = [
PW_SYNC_BINARY_SEMAPHORE_BACKEND + "_headers",
"//pw_chrono:system_clock",
@@ -56,19 +60,19 @@
pw_cc_library(
name = "binary_semaphore_backend",
deps = [
- PW_SYNC_BINARY_SEMAPHORE_BACKEND,
+ PW_SYNC_BINARY_SEMAPHORE_BACKEND,
],
)
pw_cc_library(
name = "counting_semaphore_facade",
+ srcs = [
+ "counting_semaphore.cc",
+ ],
hdrs = [
"public/pw_sync/counting_semaphore.h",
],
includes = ["public"],
- srcs = [
- "counting_semaphore.cc"
- ],
deps = [
PW_SYNC_COUNTING_SEMAPHORE_BACKEND + "_headers",
"//pw_chrono:system_clock",
@@ -87,7 +91,7 @@
pw_cc_library(
name = "counting_semaphore_backend",
deps = [
- PW_SYNC_COUNTING_SEMAPHORE_BACKEND,
+ PW_SYNC_COUNTING_SEMAPHORE_BACKEND,
],
)
@@ -104,15 +108,15 @@
pw_cc_library(
name = "mutex_facade",
+ srcs = [
+ "mutex.cc",
+ ],
hdrs = [
"public/pw_sync/mutex.h",
],
includes = ["public"],
- srcs = [
- "mutex.cc"
- ],
deps = [
- ":lock_annotations",
+ ":lock_annotations",
"//pw_preprocessor",
PW_SYNC_MUTEX_BACKEND + "_headers",
],
@@ -129,21 +133,21 @@
pw_cc_library(
name = "mutex_backend",
deps = [
- PW_SYNC_MUTEX_BACKEND,
+ PW_SYNC_MUTEX_BACKEND,
],
)
pw_cc_library(
name = "timed_mutex_facade",
+ srcs = [
+ "timed_mutex.cc",
+ ],
hdrs = [
"public/pw_sync/timed_mutex.h",
],
includes = ["public"],
- srcs = [
- "timed_mutex.cc"
- ],
deps = [
- ":lock_annotations",
+ ":lock_annotations",
":mutex_facade",
"//pw_chrono:system_clock",
"//pw_preprocessor",
@@ -162,21 +166,21 @@
pw_cc_library(
name = "timed_mutex_backend",
deps = [
- PW_SYNC_TIMED_MUTEX_BACKEND,
+ PW_SYNC_TIMED_MUTEX_BACKEND,
],
)
pw_cc_library(
name = "interrupt_spin_lock_facade",
+ srcs = [
+ "interrupt_spin_lock.cc",
+ ],
hdrs = [
"public/pw_sync/interrupt_spin_lock.h",
],
includes = ["public"],
- srcs = [
- "interrupt_spin_lock.cc"
- ],
deps = [
- ":lock_annotations",
+ ":lock_annotations",
"//pw_preprocessor",
PW_SYNC_INTERRUPT_SPIN_LOCK_BACKEND + "_headers",
],
@@ -193,7 +197,7 @@
pw_cc_library(
name = "interrupt_spin_lock_backend",
deps = [
- PW_SYNC_INTERRUPT_SPIN_LOCK_BACKEND,
+ PW_SYNC_INTERRUPT_SPIN_LOCK_BACKEND,
],
)
diff --git a/pw_sync_embos/BUILD b/pw_sync_embos/BUILD
index da66e9e..1a18120 100644
--- a/pw_sync_embos/BUILD
+++ b/pw_sync_embos/BUILD
@@ -85,7 +85,6 @@
],
)
-
pw_cc_library(
name = "mutex_headers",
hdrs = [
@@ -138,13 +137,12 @@
],
deps = [
":timed_mutex_headers",
+ "//pw_chrono_embos:system_clock_headers",
"//pw_interrupt:context",
"//pw_sync:timed_mutex_facade",
- "//pw_chrono_embos:system_clock_headers",
],
)
-
pw_cc_library(
name = "interrupt_spin_lock_headers",
hdrs = [
@@ -171,4 +169,3 @@
"//pw_sync:interrupt_spin_lock_facade",
],
)
-
diff --git a/pw_sys_io_arduino/BUILD b/pw_sys_io_arduino/BUILD
index 3735a73..22f116d 100644
--- a/pw_sys_io_arduino/BUILD
+++ b/pw_sys_io_arduino/BUILD
@@ -29,5 +29,5 @@
"//pw_boot_armv7m",
"//pw_preprocessor",
"//pw_sys_io",
- ]
+ ],
)
diff --git a/pw_sys_io_baremetal_lm3s6965evb/BUILD b/pw_sys_io_baremetal_lm3s6965evb/BUILD
index 85fe01a..78bb467 100644
--- a/pw_sys_io_baremetal_lm3s6965evb/BUILD
+++ b/pw_sys_io_baremetal_lm3s6965evb/BUILD
@@ -14,7 +14,7 @@
load(
"//pw_build:pigweed.bzl",
- "pw_cc_library",
+ "pw_cc_library",
)
package(default_visibility = ["//visibility:public"])
@@ -23,11 +23,10 @@
pw_cc_library(
name = "pw_sys_io_baremetal_lm3s6965evb",
- hdrs = ["public/pw_sys_io_baremetal_lm3s6965evb/init.h"],
srcs = ["sys_io_baremetal.cc"],
- deps = [
+ hdrs = ["public/pw_sys_io_baremetal_lm3s6965evb/init.h"],
+ deps = [
"//pw_preprocessor",
- "//pw_sys_io",
- ],
+ "//pw_sys_io",
+ ],
)
-
diff --git a/pw_thread/BUILD b/pw_thread/BUILD
index 71957c0..37ab883 100644
--- a/pw_thread/BUILD
+++ b/pw_thread/BUILD
@@ -24,8 +24,11 @@
# TODO(pwbug/101): Need to add support for facades/backends to Bazel.
PW_THREAD_ID_BACKEND = "//pw_thread_stl:id"
+
PW_THREAD_SLEEP_BACKEND = "//pw_thread_stl:sleep"
+
PW_THREAD_THREAD_BACKEND = "//pw_thread_stl:thread"
+
PW_THREAD_YIELD_BACKEND = "//pw_thread_stl:yield"
pw_cc_library(
@@ -50,19 +53,19 @@
pw_cc_library(
name = "id_backend",
deps = [
- PW_THREAD_ID_BACKEND,
+ PW_THREAD_ID_BACKEND,
],
)
pw_cc_library(
name = "sleep_facade",
+ srcs = [
+ "sleep.cc",
+ ],
hdrs = [
"public/pw_thread/sleep.h",
],
includes = ["public"],
- srcs = [
- "sleep.cc"
- ],
deps = [
PW_THREAD_SLEEP_BACKEND + "_headers",
"//pw_chrono:system_clock",
@@ -81,7 +84,7 @@
pw_cc_library(
name = "sleep_backend",
deps = [
- PW_THREAD_SLEEP_BACKEND,
+ PW_THREAD_SLEEP_BACKEND,
],
)
@@ -99,20 +102,20 @@
pw_cc_library(
name = "thread",
- deps = [
- ":thread_facade",
- ":thread_core",
- PW_THREAD_THREAD_BACKEND + "_headers",
- ],
srcs = [
- "thread.cc"
+ "thread.cc",
+ ],
+ deps = [
+ ":thread_core",
+ ":thread_facade",
+ PW_THREAD_THREAD_BACKEND + "_headers",
],
)
pw_cc_library(
name = "thread_backend",
deps = [
- PW_THREAD_THREAD_BACKEND,
+ PW_THREAD_THREAD_BACKEND,
],
)
@@ -126,13 +129,13 @@
pw_cc_library(
name = "yield_facade",
+ srcs = [
+ "yield.cc",
+ ],
hdrs = [
"public/pw_thread/yield.h",
],
includes = ["public"],
- srcs = [
- "yield.cc"
- ],
deps = [
PW_THREAD_YIELD_BACKEND + "_headers",
"//pw_preprocessor",
@@ -150,7 +153,7 @@
pw_cc_library(
name = "yield_backend",
deps = [
- PW_THREAD_YIELD_BACKEND,
+ PW_THREAD_YIELD_BACKEND,
],
)
@@ -173,9 +176,9 @@
"thread_facade_test.cc",
],
deps = [
- ":thread",
":id",
":test_threads_header",
+ ":thread",
"//pw_chrono:system_clock",
"//pw_sync:binary_semaphore",
"//pw_unit_test",
diff --git a/pw_thread_embos/BUILD b/pw_thread_embos/BUILD
index dd35c30..3205087 100644
--- a/pw_thread_embos/BUILD
+++ b/pw_thread_embos/BUILD
@@ -67,9 +67,9 @@
],
deps = [
":sleep_headers",
- "//pw_chrono_embos:system_clock_headers",
"//pw_assert",
"//pw_chrono:system_clock",
+ "//pw_chrono_embos:system_clock_headers",
"//pw_thread:sleep_facade",
],
# TODO(pwbug/317): This should depend on embOS but our third parties
diff --git a/pw_thread_freertos/BUILD b/pw_thread_freertos/BUILD
index ab6c845..5b0895d 100644
--- a/pw_thread_freertos/BUILD
+++ b/pw_thread_freertos/BUILD
@@ -68,9 +68,9 @@
],
deps = [
":sleep_headers",
- "//pw_chrono_freertos:system_clock_headers",
"//pw_assert",
"//pw_chrono:system_clock",
+ "//pw_chrono_freertos:system_clock_headers",
"//pw_thread:sleep_facade",
],
# TODO(pwbug/317): This should depend on FreeRTOS but our third parties
@@ -81,9 +81,9 @@
pw_cc_library(
name = "thread_headers",
hdrs = [
+ "public/pw_thread_freertos/config.h",
"public/pw_thread_freertos/context.h",
"public/pw_thread_freertos/options.h",
- "public/pw_thread_freertos/config.h",
"public/pw_thread_freertos/thread_inline.h",
"public/pw_thread_freertos/thread_native.h",
"public_overrides/pw_thread_backend/thread_inline.h",
@@ -94,8 +94,9 @@
"public_overrides",
],
deps = [
- "//pw_assert",
":id",
+ "//pw_assert",
+ "//pw_sync:binary_semaphore",
"//pw_thread:thread_headers",
],
# TODO(pwbug/317): This should depend on FreeRTOS but our third parties
@@ -108,9 +109,9 @@
"thread.cc",
],
deps = [
- "//pw_assert",
":id",
":thread_headers",
+ "//pw_assert",
],
# TODO(pwbug/317): This should depend on FreeRTOS but our third parties
# currently do not have Bazel support.
@@ -118,44 +119,44 @@
pw_cc_library(
name = "dynamic_test_threads",
- deps = [
- "//pw_thread:thread_facade",
- "//pw_thread:test_threads_header",
- "//pw_chrono:system_clock",
- "//pw_thread:sleep",
- ],
srcs = [
"dynamic_test_threads.cc",
- ]
+ ],
+ deps = [
+ "//pw_chrono:system_clock",
+ "//pw_thread:sleep",
+ "//pw_thread:test_threads_header",
+ "//pw_thread:thread_facade",
+ ],
)
pw_cc_test(
name = "dynamic_thread_backend_test",
deps = [
- "//pw_thread:thread_facade_test",
":dynamic_test_threads",
- ]
+ "//pw_thread:thread_facade_test",
+ ],
)
pw_cc_library(
name = "static_test_threads",
- deps = [
- "//pw_thread:thread_facade",
- "//pw_thread:test_threads_header",
- "//pw_chrono:system_clock",
- "//pw_thread:sleep",
- ],
srcs = [
"static_test_threads.cc",
- ]
+ ],
+ deps = [
+ "//pw_chrono:system_clock",
+ "//pw_thread:sleep",
+ "//pw_thread:test_threads_header",
+ "//pw_thread:thread_facade",
+ ],
)
pw_cc_test(
name = "static_thread_backend_test",
deps = [
- "//pw_thread:thread_facade_test",
":static_test_threads",
- ]
+ "//pw_thread:thread_facade_test",
+ ],
)
pw_cc_library(
@@ -179,4 +180,3 @@
"//pw_thread:yield_facade",
],
)
-
diff --git a/pw_thread_stl/BUILD b/pw_thread_stl/BUILD
index e7624a6..6403ef2 100644
--- a/pw_thread_stl/BUILD
+++ b/pw_thread_stl/BUILD
@@ -93,21 +93,21 @@
pw_cc_library(
name = "test_threads",
- deps = [
- "//pw_thread:thread_facade",
- "//pw_thread:test_threads_header",
- ],
srcs = [
"test_threads.cc",
- ]
+ ],
+ deps = [
+ "//pw_thread:test_threads_header",
+ "//pw_thread:thread_facade",
+ ],
)
pw_cc_test(
name = "thread_backend_test",
deps = [
- "//pw_thread:thread_facade_test",
":test_threads",
- ]
+ "//pw_thread:thread_facade_test",
+ ],
)
pw_cc_library(
diff --git a/pw_thread_threadx/BUILD b/pw_thread_threadx/BUILD
index 427af06..a333d23 100644
--- a/pw_thread_threadx/BUILD
+++ b/pw_thread_threadx/BUILD
@@ -43,16 +43,16 @@
"//pw_thread:id_facade",
],
# TODO(pwbug/317): This should depend on ThreadX but our third parties
- # currently do not have Bazel support.
+ # currently do not have Bazel support.
)
# This target provides the ThreadX specific headers needs for thread creation.
pw_cc_library(
name = "thread_headers",
hdrs = [
+ "public/pw_thread_threadx/config.h",
"public/pw_thread_threadx/context.h",
"public/pw_thread_threadx/options.h",
- "public/pw_thread_threadx/config.h",
"public/pw_thread_threadx/thread_inline.h",
"public/pw_thread_threadx/thread_native.h",
"public_overrides/pw_thread_backend/thread_inline.h",
@@ -63,8 +63,8 @@
"public_overrides",
],
deps = [
- "//pw_assert",
":id",
+ "//pw_assert",
"//pw_thread:thread_headers",
],
# TODO(pwbug/317): This should depend on ThreadX but our third parties
@@ -77,9 +77,9 @@
"thread.cc",
],
deps = [
- "//pw_assert",
":id",
":thread_headers",
+ "//pw_assert",
],
# TODO(pwbug/317): This should depend on ThreadX but our third parties
# currently do not have Bazel support.
@@ -87,23 +87,23 @@
pw_cc_library(
name = "test_threads",
- deps = [
- "//pw_thread:thread_facade",
- "//pw_thread:test_threads_header",
- "//pw_chrono:system_clock",
- "//pw_thread:sleep",
- ],
srcs = [
"test_threads.cc",
- ]
+ ],
+ deps = [
+ "//pw_chrono:system_clock",
+ "//pw_thread:sleep",
+ "//pw_thread:test_threads_header",
+ "//pw_thread:thread_facade",
+ ],
)
pw_cc_test(
name = "thread_backend_test",
deps = [
- "//pw_thread:thread_facade_test",
":test_threads",
- ]
+ "//pw_thread:thread_facade_test",
+ ],
)
pw_cc_library(
@@ -128,13 +128,13 @@
],
deps = [
":sleep_headers",
- "//pw_chrono_threadx:system_clock_headers",
"//pw_assert",
"//pw_chrono:system_clock",
+ "//pw_chrono_threadx:system_clock_headers",
"//pw_thread:sleep_facade",
],
# TODO(pwbug/317): This should depend on ThreadX but our third parties
- # currently do not have Bazel support.
+ # currently do not have Bazel support.
)
pw_cc_library(
@@ -148,7 +148,7 @@
"public_overrides",
],
# TODO(pwbug/317): This should depend on ThreadX but our third parties
- # currently do not have Bazel support.
+ # currently do not have Bazel support.
)
pw_cc_library(
diff --git a/pw_tool/BUILD b/pw_tool/BUILD
index 304ec5b..c561a34 100644
--- a/pw_tool/BUILD
+++ b/pw_tool/BUILD
@@ -21,10 +21,9 @@
pw_cc_binary(
name = "pw_tool",
- srcs = [ "main.cc" ],
+ srcs = ["main.cc"],
deps = [
"//pw_log",
"//pw_polyfill",
- ]
+ ],
)
-
diff --git a/pw_trace/BUILD b/pw_trace/BUILD
index 24f5da2..cbd5be0 100644
--- a/pw_trace/BUILD
+++ b/pw_trace/BUILD
@@ -28,8 +28,8 @@
pw_cc_library(
name = "facade",
hdrs = [
- "public/pw_trace/trace.h",
"public/pw_trace/internal/trace_internal.h",
+ "public/pw_trace/trace.h",
],
includes = ["public"],
deps = [
@@ -67,13 +67,13 @@
pw_cc_test(
name = "trace_facade_test",
srcs = [
- "trace_facade_test.cc",
"pw_trace_test/fake_backend.h",
"pw_trace_test/public_overrides/pw_trace_backend/trace_backend.h",
+ "trace_facade_test.cc",
],
includes = [
"pw_trace_test",
- "pw_trace_test/public_overrides"
+ "pw_trace_test/public_overrides",
],
deps = [
":backend",
@@ -86,17 +86,17 @@
pw_cc_library(
name = "pw_trace_sample_app",
- srcs = [ "example/sample_app.cc" ],
- includes = [ "example/public" ],
- deps = [ "//pw_trace" ],
- hdrs = [ "example/public/pw_trace/example/sample_app.h" ]
+ srcs = ["example/sample_app.cc"],
+ hdrs = ["example/public/pw_trace/example/sample_app.h"],
+ includes = ["example/public"],
+ deps = ["//pw_trace"],
)
pw_cc_binary(
name = "trace_example_basic",
+ srcs = ["example/basic.cc"],
deps = [
":pw_trace_sample_app",
- "//pw_log"
+ "//pw_log",
],
- srcs = [ "example/basic.cc" ]
-)
\ No newline at end of file
+)
diff --git a/pw_trace_tokenized/BUILD b/pw_trace_tokenized/BUILD
index 1944d71..c9dd954 100644
--- a/pw_trace_tokenized/BUILD
+++ b/pw_trace_tokenized/BUILD
@@ -62,15 +62,15 @@
pw_cc_library(
name = "trace_rpc_service",
+ srcs = [
+ "trace_rpc_service_nanopb.cc",
+ ],
hdrs = [
"public/pw_trace_tokenized/trace_rpc_service_nanopb.h",
],
includes = [
"public",
],
- srcs = [
- "trace_rpc_service_nanopb.cc",
- ],
deps = [
"//pw_log",
"//pw_trace",
@@ -107,17 +107,17 @@
pw_cc_library(
name = "pw_trace_tokenized_buffer_log",
- hdrs = [
- "public/pw_trace_tokenized/trace_buffer_log.h",
- ],
srcs = [
"trace_buffer_log.cc",
],
+ hdrs = [
+ "public/pw_trace_tokenized/trace_buffer_log.h",
+ ],
deps = [
":trace_buffer_headers",
- "//pw_base64",
- "//pw_log",
- "//pw_string",
+ "//pw_base64",
+ "//pw_log",
+ "//pw_string",
],
)
@@ -138,7 +138,7 @@
],
includes = [
"pw_trace_test",
- "pw_trace_test/public_overrides"
+ "pw_trace_test/public_overrides",
],
deps = [
":backend",
@@ -179,61 +179,61 @@
pw_cc_library(
name = "pw_trace_host_trace_time",
- includes = [ "example/public" ],
- deps = [ "//pw_trace" ],
- srcs = [ "host_trace_time.cc" ]
+ srcs = ["host_trace_time.cc"],
+ includes = ["example/public"],
+ deps = ["//pw_trace"],
)
pw_cc_library(
name = "pw_trace_example_to_file",
- includes = [ "example/public" ],
- deps = [ "//pw_trace" ],
- hdrs = [ "example/public/pw_trace_tokenized/example/trace_to_file.h" ]
+ hdrs = ["example/public/pw_trace_tokenized/example/trace_to_file.h"],
+ includes = ["example/public"],
+ deps = ["//pw_trace"],
)
pw_cc_binary(
name = "trace_tokenized_example_basic",
+ srcs = ["example/basic.cc"],
deps = [
":pw_trace_example_to_file",
- "//pw_log",
"//dir_pw_trace",
"//dir_pw_trace:pw_trace_sample_app",
+ "//pw_log",
],
- srcs = [ "example/basic.cc" ]
)
pw_cc_binary(
name = "trace_tokenized_example_trigger",
+ srcs = ["example/trigger.cc"],
deps = [
":pw_trace_example_to_file",
- "//pw_log",
"//dir_pw_trace",
"//dir_pw_trace:pw_trace_sample_app",
+ "//pw_log",
],
- srcs = [ "example/trigger.cc" ]
)
pw_cc_binary(
name = "trace_tokenized_example_filter",
+ srcs = ["example/filter.cc"],
deps = [
":pw_trace_example_to_file",
- "//pw_log",
"//dir_pw_trace",
"//dir_pw_trace:pw_trace_sample_app",
+ "//pw_log",
],
- srcs = [ "example/filter.cc" ]
)
pw_cc_library(
name = "trace_tokenized_example_rpc",
+ srcs = ["example/rpc.cc"],
deps = [
":pw_trace_rpc_service",
"//dir_pw_rpc:server",
"//dir_pw_rpc:system_server",
- "//pw_log",
- "//pw_hdlc",
"//dir_pw_trace",
"//dir_pw_trace:pw_trace_sample_app",
+ "//pw_hdlc",
+ "//pw_log",
],
- srcs = [ "example/rpc.cc" ]
-)
\ No newline at end of file
+)
diff --git a/pw_unit_test/BUILD b/pw_unit_test/BUILD
index 7d94c19..99c9eec 100644
--- a/pw_unit_test/BUILD
+++ b/pw_unit_test/BUILD
@@ -88,14 +88,14 @@
pw_cc_library(
name = "rpc_service",
- hdrs = [
- "public/pw_unit_test/internal/rpc_event_handler.h",
- "public/pw_unit_test/unit_test_service.h",
- ],
srcs = [
"rpc_event_handler.cc",
"unit_test_service.cc",
],
+ hdrs = [
+ "public/pw_unit_test/internal/rpc_event_handler.h",
+ "public/pw_unit_test/unit_test_service.h",
+ ],
deps = [
":pw_unit_test",
"//pw_log",
diff --git a/pw_varint/BUILD b/pw_varint/BUILD
index 1104fdb..e4e35be 100644
--- a/pw_varint/BUILD
+++ b/pw_varint/BUILD
@@ -41,8 +41,8 @@
pw_cc_test(
name = "varint_test",
srcs = [
- "varint_test_c.c",
"varint_test.cc",
+ "varint_test_c.c",
],
deps = [
":pw_varint",
diff --git a/pw_web_ui/BUILD b/pw_web_ui/BUILD
index 118fba4..3a242a2 100644
--- a/pw_web_ui/BUILD
+++ b/pw_web_ui/BUILD
@@ -11,7 +11,10 @@
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations under
# the License.
-exports_files(["rollup.config.js"], visibility = ["//:__subpackages__"])
+exports_files(
+ ["rollup.config.js"],
+ visibility = ["//:__subpackages__"],
+)
alias(
name = "devserver",
diff --git a/pw_web_ui/src/frontend/BUILD b/pw_web_ui/src/frontend/BUILD
index 03da0b0..47f732c 100644
--- a/pw_web_ui/src/frontend/BUILD
+++ b/pw_web_ui/src/frontend/BUILD
@@ -14,7 +14,7 @@
package(default_visibility = ["//visibility:public"])
-load("@npm//@bazel/typescript:index.bzl", "ts_library", "ts_devserver")
+load("@npm//@bazel/typescript:index.bzl", "ts_devserver", "ts_library")
load("//pw_web_ui:web_bundle.bzl", "web_bundle")
ts_library(
@@ -24,21 +24,21 @@
"index.tsx",
],
deps = [
- "@npm//@types/react",
- "@npm//react",
- "@npm//@types/react-dom",
- "@npm//react-dom",
- "@npm//@material-ui/core",
"//pw_web_ui/src/transport:web_serial_transport_lib",
+ "@npm//@material-ui/core",
+ "@npm//@types/react",
+ "@npm//@types/react-dom",
+ "@npm//react",
+ "@npm//react-dom",
],
)
web_bundle(
name = "app_bundle",
+ entry_point = "index.tsx",
deps = [
":app_lib",
],
- entry_point = "index.tsx",
)
ts_devserver(
@@ -48,7 +48,7 @@
static_files = [
"index.html",
":app_bundle",
- ]
+ ],
)
ts_devserver(
@@ -58,5 +58,5 @@
static_files = [
"index.html",
":app_bundle",
- ]
+ ],
)
diff --git a/pw_web_ui/src/transport/BUILD b/pw_web_ui/src/transport/BUILD
index 14959a1..6a80546 100644
--- a/pw_web_ui/src/transport/BUILD
+++ b/pw_web_ui/src/transport/BUILD
@@ -36,8 +36,8 @@
],
deps = [
"//pw_web_ui/types:serial_lib",
- "@npm//jasmine",
"@npm//@types/jasmine",
+ "@npm//jasmine",
"@npm//rxjs",
],
)
@@ -62,24 +62,23 @@
deps = [
":serial_mock_lib",
":web_serial_transport_lib",
- "@npm//jasmine",
"@npm//@types/jasmine",
+ "@npm//jasmine",
"@npm//rxjs",
],
)
web_bundle(
name = "web_serial_transport_lib_test_bundle",
+ entry_point = "web_serial_transport_test.ts",
deps = [
":web_serial_transport_lib_test",
],
- entry_point = "web_serial_transport_test.ts",
)
-
karma_web_test(
name = "web_test",
srcs = [
- ":web_serial_transport_lib_test_bundle"
+ ":web_serial_transport_lib_test_bundle",
],
)
diff --git a/pw_web_ui/web_bundle.bzl b/pw_web_ui/web_bundle.bzl
index 0ac17ea..4aa9057 100644
--- a/pw_web_ui/web_bundle.bzl
+++ b/pw_web_ui/web_bundle.bzl
@@ -15,17 +15,17 @@
load("@npm//@bazel/rollup:index.bzl", "rollup_bundle")
def web_bundle(name, deps, entry_point):
- rollup_bundle(
- name = name,
- deps = deps + [
- "@npm//@rollup/plugin-node-resolve",
- "@npm//@rollup/plugin-commonjs",
- "@npm//rollup-plugin-node-builtins",
- "@npm//rollup-plugin-node-globals",
- "@npm//rollup-plugin-sourcemaps",
- ],
- entry_point = entry_point,
- config_file = "//pw_web_ui:rollup.config.js",
- sourcemap = "inline",
- format = "cjs"
- )
+ rollup_bundle(
+ name = name,
+ deps = deps + [
+ "@npm//@rollup/plugin-node-resolve",
+ "@npm//@rollup/plugin-commonjs",
+ "@npm//rollup-plugin-node-builtins",
+ "@npm//rollup-plugin-node-globals",
+ "@npm//rollup-plugin-sourcemaps",
+ ],
+ entry_point = entry_point,
+ config_file = "//pw_web_ui:rollup.config.js",
+ sourcemap = "inline",
+ format = "cjs",
+ )
diff --git a/targets/arduino/BUILD b/targets/arduino/BUILD
index 7514235..c351de2 100644
--- a/targets/arduino/BUILD
+++ b/targets/arduino/BUILD
@@ -36,7 +36,7 @@
name = "system_rpc_server",
srcs = ["system_rpc_server.cc"],
deps = [
- "//pw_rpc/system_server:facade",
"//pw_hdlc:pw_rpc",
+ "//pw_rpc/system_server:facade",
],
-)
\ No newline at end of file
+)
diff --git a/targets/host/BUILD b/targets/host/BUILD
index 20fd666..307a463 100644
--- a/targets/host/BUILD
+++ b/targets/host/BUILD
@@ -25,8 +25,7 @@
name = "system_rpc_server",
srcs = ["system_rpc_server.cc"],
deps = [
- "//pw_rpc/system_server:facade",
"//pw_hdlc:pw_rpc",
+ "//pw_rpc/system_server:facade",
],
)
-
diff --git a/targets/lm3s6965evb-qemu/BUILD b/targets/lm3s6965evb-qemu/BUILD
index bb15e06..eb05f9c 100644
--- a/targets/lm3s6965evb-qemu/BUILD
+++ b/targets/lm3s6965evb-qemu/BUILD
@@ -25,7 +25,7 @@
name = "pre_init",
srcs = [
"boot.cc",
- "vector_table.c"
+ "vector_table.c",
],
deps = [
"//pw_boot_armv7m",
diff --git a/targets/stm32f429i-disc1/BUILD b/targets/stm32f429i-disc1/BUILD
index b9cffcc..623f7b7 100644
--- a/targets/stm32f429i-disc1/BUILD
+++ b/targets/stm32f429i-disc1/BUILD
@@ -25,7 +25,7 @@
name = "pre_init",
srcs = [
"boot.cc",
- "vector_table.c"
+ "vector_table.c",
],
deps = [
"//pw_boot_armv7m",
@@ -39,7 +39,7 @@
name = "system_rpc_server",
srcs = ["system_rpc_server.cc"],
deps = [
- "//pw_rpc/system_server:facade",
"//pw_hdlc:pw_rpc",
+ "//pw_rpc/system_server:facade",
],
)
