build: switch to build settings for static build
This switches the way to select static linking for C builds from a
constraint_setting to a build_setting. The benefit of this is that the
transition to static linking no longer depends on the architecture,
because this setting is not part of the platform anymore.
The alternative would be to select the matching static platform based on
the architecture in the transition implementation. However, I did not
find a good way to obtain the architecture in a transition
implementation. You can access the platform through the settings
parameter, but it's not clear how to extract the architecture from that.
Something that would work is to define an attribute for the architecture
on the rule where the transition is used, and fill this attribute using
a select() at the point where the rule is instantiated. But this
requires changes to all rules which use the transition, which is ugly.
Change-Id: I8962e24b561b8d1724bac785cb3739cdc434d47e
Reviewed-on: https://review.monogon.dev/c/monogon/+/4172
Tested-by: Jenkins CI
Reviewed-by: Lorenz Brun <lorenz@monogon.tech>
diff --git a/build/platforms/BUILD.bazel b/build/platforms/BUILD.bazel
index 217350a..293af9b 100644
--- a/build/platforms/BUILD.bazel
+++ b/build/platforms/BUILD.bazel
@@ -18,18 +18,6 @@
visibility = ["//visibility:public"],
)
-# Linux x86_64 platform with static linking
-# (i.e. Metropolis node, scratch containers...).
-platform(
- name = "linux_amd64_static",
- constraint_values = [
- "@platforms//os:linux",
- "@platforms//cpu:x86_64",
- "//build/platforms/linkmode:musl-static",
- ],
- visibility = ["//visibility:public"],
-)
-
platform(
name = "remote_amd64",
constraint_values = [
diff --git a/build/platforms/linkmode/BUILD.bazel b/build/platforms/linkmode/BUILD.bazel
index 8a7f99c..f749e9d 100644
--- a/build/platforms/linkmode/BUILD.bazel
+++ b/build/platforms/linkmode/BUILD.bazel
@@ -1,14 +1,7 @@
-package(
- default_visibility = ["//visibility:public"],
-)
+load("@bazel_skylib//rules:common_settings.bzl", "bool_flag")
-# Minimal Metropolis node environment. No dynamic linker,
-# so everything has to be statically linked.
-constraint_value(
- name = "musl-static",
- constraint_setting = ":linkmode",
-)
-
-constraint_setting(
- name = "linkmode",
+bool_flag(
+ name = "static",
+ build_setting_default = False,
+ visibility = ["//visibility:public"],
)