Skip to content

Commit

Permalink
Fix bazel rules for layering_check and parse_headers with clang (#162)
Browse files Browse the repository at this point in the history
Signed-off-by: Michael Carroll <[email protected]>
  • Loading branch information
mjcarroll authored Nov 15, 2024
1 parent 7daf8e7 commit cae001e
Show file tree
Hide file tree
Showing 3 changed files with 24 additions and 6 deletions.
13 changes: 10 additions & 3 deletions BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,10 @@ cc_library(
"test/integration/implptr/implptr_test_classes.hh",
],
includes = ["test/integration/implptr"],
deps = [":ImplPtr"],
deps = [
":Export",
":ImplPtr",
],
)

cc_test(
Expand All @@ -122,6 +125,7 @@ cc_test(
srcs = ["test/integration/implptr/ImplPtr_TEST.cc"],
deps = [
":implptr_test_classes",
"@googletest//:gtest",
"@googletest//:gtest_main",
],
)
Expand All @@ -131,6 +135,7 @@ cc_test(
srcs = ["src/Environment_TEST.cc"],
deps = [
":Environment",
"@googletest//:gtest",
"@googletest//:gtest_main",
],
)
Expand All @@ -140,7 +145,7 @@ cc_binary(
srcs = ["test/integration/subprocess/subprocess_main.cc"],
deps = [
":Environment",
"//cli",
"//cli:cli11",
],
)

Expand All @@ -149,8 +154,10 @@ cc_test(
srcs = ["test/integration/subprocess_TEST.cc"],
local_defines = ['SUBPROCESS_EXECUTABLE_PATH=\\"subprocess_main\\"'],
deps = [
":gz-utils",
":Environment",
":Subprocess",
":subprocess_main",
"@googletest//:gtest",
"@googletest//:gtest_main",
],
)
Expand Down
5 changes: 4 additions & 1 deletion cli/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ cc_library(
"-parse_headers",
],
includes = ["include/vendored-cli"],
visibility = ["//visibility:public"],
)

cc_library(
Expand Down Expand Up @@ -45,7 +46,9 @@ cc_test(
name = "cli_TEST",
srcs = ["src/cli_TEST.cc"],
deps = [
":cli",
":GzFormatter",
":cli11",
"@googletest//:gtest",
"@googletest//:gtest_main",
],
)
12 changes: 10 additions & 2 deletions log/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,13 @@ cc_library(
],
includes = ["include"],
visibility = ["//visibility:public"],
deps = [":SplitSink"],
deps = [
":Export",
":SplitSink",
"//:Config",
"//:ImplPtr",
"@spdlog",
],
)

cc_library(
Expand All @@ -61,7 +67,9 @@ cc_test(
srcs = ["src/SplitSink_TEST.cc"],
visibility = ["//visibility:public"],
deps = [
":log",
":SplitSink",
"@googletest//:gtest",
"@googletest//:gtest_main",
"@spdlog",
],
)

0 comments on commit cae001e

Please sign in to comment.