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/y2016/control_loops/superstructure/BUILD b/y2016/control_loops/superstructure/BUILD
index aecfd98..cb07ebe 100644
--- a/y2016/control_loops/superstructure/BUILD
+++ b/y2016/control_loops/superstructure/BUILD
@@ -8,6 +8,7 @@
"superstructure_goal.fbs",
],
gen_reflections = 1,
+ target_compatible_with = ["@platforms//os:linux"],
)
flatbuffer_cc_library(
@@ -19,6 +20,7 @@
includes = [
"//frc971/control_loops:control_loops_fbs_includes",
],
+ target_compatible_with = ["@platforms//os:linux"],
)
flatbuffer_cc_library(
@@ -27,6 +29,7 @@
"superstructure_output.fbs",
],
gen_reflections = 1,
+ target_compatible_with = ["@platforms//os:linux"],
)
flatbuffer_cc_library(
@@ -38,6 +41,7 @@
includes = [
"//frc971/control_loops:control_loops_fbs_includes",
],
+ target_compatible_with = ["@platforms//os:linux"],
)
genrule(
@@ -49,6 +53,7 @@
"integral_intake_plant.cc",
],
cmd = "$(location //y2016/control_loops/python:intake) $(OUTS)",
+ target_compatible_with = ["@platforms//os:linux"],
tools = [
"//y2016/control_loops/python:intake",
],
@@ -64,6 +69,7 @@
"integral_arm_plant.cc",
],
cmd = "$(location //y2016/control_loops/python:arm) $(OUTS)",
+ target_compatible_with = ["@platforms//os:linux"],
tools = [
"//y2016/control_loops/python:arm",
],
@@ -84,6 +90,7 @@
"integral_arm_plant.h",
"integral_intake_plant.h",
],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
"//frc971/control_loops:state_feedback_loop",
],
@@ -99,6 +106,7 @@
"superstructure.h",
"superstructure_controls.h",
],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
":superstructure_goal_fbs",
":superstructure_output_fbs",
@@ -123,6 +131,7 @@
"superstructure_lib_test.cc",
],
data = ["//y2016:config"],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
":superstructure_goal_fbs",
":superstructure_lib",
@@ -143,6 +152,7 @@
srcs = [
"superstructure_main.cc",
],
+ target_compatible_with = ["@platforms//os:linux"],
deps = [
":superstructure_lib",
"//aos:init",