From cbe3179f5c1c96c0e649c02a0b14f893ae2540c4 Mon Sep 17 00:00:00 2001 From: "Steven E. Harris" Date: Tue, 12 Nov 2024 13:45:14 -0500 Subject: [PATCH] Correct comparison against stamping policy name The three valid stamping policies are named as follows: - Allow - Force - Prevent When checking whether to stamp injected values for tagged fields, we were looking mistakenly for a value of "Never", which was the previous name for the policy now known as "Prevent". Correct that sentinel value to allow rule callers to prevent stamping of injected values. --- cue/cue.bzl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cue/cue.bzl b/cue/cue.bzl index 3d10599..6ce1adb 100644 --- a/cue/cue.bzl +++ b/cue/cue.bzl @@ -33,7 +33,7 @@ CUEInstanceInfo = provider( def _replacer_if_stamping(stamping_policy): # NB: We can't access the "_cue_config" attribute here. - return Label("//tools/cmd/replace-stamps") if stamping_policy != "Never" else None + return Label("//tools/cmd/replace-stamps") if stamping_policy != "Prevent" else None def _add_common_source_consuming_attrs_to(attrs): attrs.update({