commit dada107facf6a76af9d974fa6b17af76fa85e680
author Philipp Schrader <philipp.schrader@gmail.com> Tue Nov 24 11:34:46 2020 -0800
committer Philipp Schrader <philipp.schrader@gmail.com> Sat Dec 12 15:48:00 2020 -0800
tree 0e5b9c456e38573b94ade3d8b15c18f13b07d15b
parent fbbd93b5a87a6c01c24d750fedac568bc6ac8cbe

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

motors/peripheral/BUILD

diff --git a/motors/peripheral/BUILD b/motors/peripheral/BUILD
index c8b63e9..a42d502 100644
--- a/motors/peripheral/BUILD
+++ b/motors/peripheral/BUILD
@@ -1,5 +1,3 @@
-load("//tools:environments.bzl", "mcu_cpus")
-
 cc_library(
     name = "adc",
     srcs = [
@@ -8,7 +6,7 @@
     hdrs = [
         "adc.h",
     ],
-    restricted_to = mcu_cpus,
+    target_compatible_with = ["@platforms//os:none"],
     visibility = ["//visibility:public"],
     deps = [
         ":configuration",
@@ -22,7 +20,7 @@
     hdrs = [
         "configuration.h",
     ],
-    restricted_to = mcu_cpus,
+    target_compatible_with = ["@platforms//os:none"],
     visibility = ["//visibility:public"],
 )
 
@@ -34,7 +32,7 @@
     hdrs = [
         "can.h",
     ],
-    restricted_to = mcu_cpus,
+    target_compatible_with = ["@platforms//os:none"],
     visibility = ["//visibility:public"],
     deps = [
         "//motors:util",
@@ -45,7 +43,6 @@
 cc_library(
     name = "uart_buffer",
     hdrs = ["uart_buffer.h"],
-    compatible_with = mcu_cpus,
     visibility = ["//visibility:public"],
     deps = [
         "//third_party/GSL",
@@ -57,6 +54,7 @@
     srcs = [
         "uart_buffer_test.cc",
     ],
+    target_compatible_with = ["@platforms//os:linux"],
     deps = [
         ":uart_buffer",
         "//aos/testing:googletest",
@@ -67,7 +65,7 @@
     name = "uart",
     srcs = ["uart.cc"],
     hdrs = ["uart.h"],
-    restricted_to = mcu_cpus,
+    target_compatible_with = ["@platforms//os:none"],
     visibility = ["//visibility:public"],
     deps = [
         ":uart_buffer",
@@ -82,7 +80,7 @@
     name = "adc_dma",
     srcs = ["adc_dma.cc"],
     hdrs = ["adc_dma.h"],
-    restricted_to = mcu_cpus,
+    target_compatible_with = ["@platforms//os:none"],
     visibility = ["//visibility:public"],
     deps = [
         ":configuration",
@@ -95,7 +93,7 @@
     name = "spi",
     srcs = ["spi.cc"],
     hdrs = ["spi.h"],
-    restricted_to = mcu_cpus,
+    target_compatible_with = ["@platforms//os:none"],
     visibility = ["//visibility:public"],
     deps = [
         ":uart_buffer",