Skip to content

Commit de948b8

Browse files
committed
[build] Fix macOS-cross build
This is needed since we don't have an x64 macOS clang-tidy binary, we should only fetch that when we actually use it. Also add "manual" tags for other binaries so we only fetch them when needed.
1 parent 988f711 commit de948b8

File tree

3 files changed

+5
-0
lines changed

3 files changed

+5
-0
lines changed

BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ alias(
3232
alias(
3333
name = "format",
3434
actual = "//build/deps/formatters:format",
35+
tags = ["manual"],
3536
)
3637

3738
npm_link_all_packages(name = "node_modules")

build/deps/formatters/BUILD

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ native_binary(
5151
"//conditions:default": "@clang-format-linux-amd64//file:file",
5252
}),
5353
out = "clang-format",
54+
tags = ["manual"],
5455
visibility = ["//visibility:public"],
5556
)
5657

@@ -63,6 +64,7 @@ native_binary(
6364
"//conditions:default": "@ruff-linux-amd64//:file",
6465
}),
6566
out = "ruff",
67+
tags = ["manual"],
6668
visibility = ["//visibility:public"],
6769
)
6870

@@ -77,6 +79,7 @@ native_binary(
7779
"//conditions:default": "@buildifier-linux-amd64//file:file",
7880
}),
7981
out = "buildifier",
82+
tags = ["manual"],
8083
visibility = ["//visibility:public"],
8184
)
8285

tools/BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,5 +32,6 @@ native_binary(
3232
},
3333
),
3434
out = "clang_tidy",
35+
tags = ["manual"],
3536
visibility = ["//visibility:public"],
3637
)

0 commit comments

Comments
 (0)