commit 22c20743e76b437518525064e2d2c7f814d8d6c9
author Cindy Liu <hcindyl@google.com> Fri Sep 01 12:20:51 2023 -0700
committer Cindy Liu <hcindyl@google.com> Fri Sep 01 12:20:51 2023 -0700
tree 85a13216037a1c3c77d7c3e84e6497ad44db4731
parent c494ecaaeb435b7282abf6b2a5aad76976dbf7cf

Update kelvin_binary rule tag attribute

Bazel observes the tags of the leaf target -- a target is built if
a depended target is built. We need to pass the tags attribute to all
the emit rules so wilecard omission of `tags=["manual"]` works properly

Bug: 298661906
Change-Id: Ia3baf6b9d4aa1fe2eaf48436f2da684f9d6b7c63

build_tools/bazel/kelvin.bzl

diff --git a/build_tools/bazel/kelvin.bzl b/build_tools/bazel/kelvin.bzl
index 03c1756..000d985 100644
--- a/build_tools/bazel/kelvin.bzl
+++ b/build_tools/bazel/kelvin.bzl
@@ -146,6 +146,7 @@
         name,
         srcs,
         is_riscv_test = False,
+        tags = [],
         **kwargs):
     """A helper macro for generating binary artifacts for the kelvin core.
 
@@ -175,6 +176,7 @@
         name = name,
         srcs = srcs,
         linker_script = "//crt:kelvin.ld",
+        tags = tags,
         **kwargs
     )
 
@@ -183,11 +185,13 @@
         name = "{}.bin".format(name),
         srcs = [name],
         output_group = "bin_file",
+        tags = tags,
     )
     native.filegroup(
         name = "{}.elf".format(name),
         srcs = [name],
         output_group = "elf_file",
+        tags = tags,
     )
 
 def kelvin_test(