Switch everything to platforms
This patch switches the codebase over from using the "cpu"
mechanism to using bazel platforms. See
https://docs.bazel.build/versions/master/platforms.html for some more
information.
Most of the substantial changes are in //tools. Instead of using
`cc_toolchain_suite` rules, we now use regular `toolchain` rules that
are registered in the WORKSPACE. That also means that bazel now uses
the target platform to select the compiler.
All --cpu=* arguments should now be --config=* arguments. For example,
`--cpu=roborio` should now be `--config=roborio`. The CI script and
all documentation has been updated to reflect that.
The remainder of the changes revolve around tagging all targets with
`target_compatible_with`. The old mechanism allowed us to specify
repo-wide defaults. The new mechanism does not. That means every
target that didn't have any compatibility specified, now requires
compatibility with `@platforms//os:linux`.
I used buildozer for the vast majority of `target_compatible_with`
changes. buildozer automatically buildifies any BUILD files it
touches. That means this patch also contains a few non-functional
changes that I was too lazy to remove.
Change-Id: I66d6e6ad9161520ee397597cdb492585820a3acd
diff --git a/y2017/control_loops/superstructure/BUILD b/y2017/control_loops/superstructure/BUILD
index 7107e23..fa835c1 100644
--- a/y2017/control_loops/superstructure/BUILD
+++ b/y2017/control_loops/superstructure/BUILD
@@ -11,6 +11,7 @@
includes = [
"//frc971/control_loops:control_loops_fbs_includes",
],
+ target_compatible_with = ["@platforms//os:linux"],
)
flatbuffer_cc_library(
@@ -22,6 +23,7 @@
includes = [
"//frc971/control_loops:control_loops_fbs_includes",
],
+ target_compatible_with = ["@platforms//os:linux"],
)
flatbuffer_cc_library(
@@ -34,6 +36,7 @@
"//frc971/control_loops:control_loops_fbs_includes",
"//frc971/control_loops:profiled_subsystem_fbs_includes",
],
+ target_compatible_with = ["@platforms//os:linux"],
)
flatbuffer_cc_library(
@@ -42,6 +45,7 @@
"superstructure_output.fbs",
],
gen_reflections = 1,
+ target_compatible_with = ["@platforms//os:linux"],
)
cc_library(
@@ -52,6 +56,7 @@
hdrs = [
"superstructure.h",
],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
":superstructure_goal_fbs",
":superstructure_output_fbs",
@@ -74,6 +79,7 @@
"superstructure_lib_test.cc",
],
data = ["//y2017:config"],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
":superstructure_goal_fbs",
":superstructure_lib",
@@ -98,6 +104,7 @@
srcs = [
"superstructure_main.cc",
],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
":superstructure_lib",
"//aos:init",
@@ -113,6 +120,7 @@
hdrs = [
"vision_time_adjuster.h",
],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
"//aos/containers:ring_buffer",
"//frc971/control_loops:control_loops_fbs",
@@ -129,6 +137,7 @@
"vision_time_adjuster_test.cc",
],
data = ["//y2017:config"],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
":vision_time_adjuster",
"//aos/events:simulated_event_loop",
@@ -143,6 +152,7 @@
hdrs = [
"vision_distance_average.h",
],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
"//aos/containers:ring_buffer",
"//aos/time",
@@ -155,6 +165,7 @@
srcs = [
"vision_distance_average_test.cc",
],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
":vision_distance_average",
"//aos:flatbuffers",