fixup! fix: allow disabling exec_tools toolchain from looking up an i…

…nterpreter
bazelbuild · Sep 6, 2024 · b79b9bb · b79b9bb
1 parent e6873ce
commit b79b9bb
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/python/private/BUILD.bazel b/python/private/BUILD.bazel
@@ -214,6 +214,7 @@ bzl_library(
     srcs = ["py_exec_tools_toolchain.bzl"],
     deps = [
         ":py_exec_tools_info_bzl",
+        ":sentinel_bzl",
         ":toolchain_types_bzl",
         "//python/private/common:providers_bzl",
         "@bazel_skylib//lib:paths",

diff --git a/python/private/sentinel.bzl b/python/private/sentinel.bzl
@@ -18,9 +18,13 @@ Label attributes with defaults cannot accept None, otherwise they fall
 back to using the default. A sentinel allows detecting an intended None value.
 """
 
-SentinelInfo = provider()
+SentinelInfo = provider(
+    doc = "Indicates this was the sentinel target.",
+    fields = [],
+)
 
 def _sentinel_impl(ctx):
+    _ = ctx  # @unused
     return [SentinelInfo()]
 
 sentinel = rule(implementation = _sentinel_impl)
diff --git a/tests/py_exec_tools_toolchain/py_exec_tools_toolchain_tests.bzl b/tests/py_exec_tools_toolchain/py_exec_tools_toolchain_tests.bzl
@@ -15,7 +15,7 @@
 
 load("@rules_testing//lib:analysis_test.bzl", "analysis_test")
 load("@rules_testing//lib:test_suite.bzl", "test_suite")
-load("//python/private:py_exec_tools_toolchain.bzl", "py_exec_tools_toolchain")
+load("//python/private:py_exec_tools_toolchain.bzl", "py_exec_tools_toolchain")  # buildifier: disable=bzl-visibility
 
 _tests = []