Remove unnecessary requires for archiver_flags

keith · keith · commit 82f0a1551c4d · 2026-03-20T17:46:13.000-07:00
These args don't use this variable, so it doesn't need to confuse things
here (realistically if this isn't set this action will fail too)
diff --git a/cc/toolchains/args/archiver_flags/BUILD b/cc/toolchains/args/archiver_flags/BUILD
@@ -47,7 +47,6 @@ cc_args(
         ],
         "//conditions:default": ["rcsD"],
     }),
-    requires_not_none = "//cc/toolchains/variables:output_execpath",
 )
 
 cc_args(
diff --git a/tests/rule_based_toolchain/legacy_features_as_args/goldens/macos/archiver_flags.textproto b/tests/rule_based_toolchain/legacy_features_as_args/goldens/macos/archiver_flags.textproto
@@ -2,7 +2,6 @@ enabled: false
 flag_sets {
   actions: "c++-link-static-library"
   flag_groups {
-    expand_if_available: "output_execpath"
     flags: "-D"
     flags: "-no_warning_for_no_symbols"
     flags: "-static"
diff --git a/tests/rule_based_toolchain/legacy_features_as_args/goldens/unix/archiver_flags.textproto b/tests/rule_based_toolchain/legacy_features_as_args/goldens/unix/archiver_flags.textproto
@@ -2,7 +2,6 @@ enabled: false
 flag_sets {
   actions: "c++-link-static-library"
   flag_groups {
-    expand_if_available: "output_execpath"
     flags: "rcsD"
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,6 @@ cc_args(`
`47`	`47`	`],`
`48`	`48`	`"//conditions:default": ["rcsD"],`
`49`	`49`	`}),`
`50`		`- requires_not_none = "//cc/toolchains/variables:output_execpath",`
`51`	`50`	`)`
`52`	`51`
`53`	`52`	`cc_args(`
Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,6 @@ enabled: false`
`2`	`2`	`flag_sets {`
`3`	`3`	`actions: "c++-link-static-library"`
`4`	`4`	`flag_groups {`
`5`		`- expand_if_available: "output_execpath"`
`6`	`5`	`flags: "rcsD"`
`7`	`6`	`}`
`8`	`7`	`}`