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/osbase/build/def.bzl b/osbase/build/def.bzl
index b047d8b..c8ca9dd 100644
--- a/osbase/build/def.bzl
+++ b/osbase/build/def.bzl
@@ -6,7 +6,7 @@
"""
return {
"@io_bazel_rules_go//go/config:static": True,
- "//command_line_option:platforms": "//build/platforms:linux_amd64_static",
+ "//build/platforms/linkmode:static": True,
}
build_static_transition = transition(
@@ -14,7 +14,7 @@
inputs = [],
outputs = [
"@io_bazel_rules_go//go/config:static",
- "//command_line_option:platforms",
+ "//build/platforms/linkmode:static",
],
)
@@ -63,9 +63,7 @@
"@io_bazel_rules_go//go/config:race": False,
"@io_bazel_rules_go//go/config:pure": False,
"@io_bazel_rules_go//go/config:static": False,
-
- # Note: this toolchain is not actually used to perform the build.
- "//command_line_option:platforms": "//build/platforms:linux_amd64_static",
+ "//build/platforms/linkmode:static": False,
}
def _ignore_unused_configuration_impl(_settings, _attr):