refactor: Add missing uses of DefaultInfo (bazel-contrib#2972)

keith · web-flow · commit e03b63c725cb · 2025-06-11T22:09:44.000Z
Required for compatibility with bazelbuild/bazel#20183
diff --git a/python/private/common.bzl b/python/private/common.bzl
@@ -425,7 +425,7 @@ def create_py_info(
         else:
             # TODO(b/228692666): Remove this once non-PyInfo targets are no
             # longer supported in `deps`.
-            files = target.files.to_list()
+            files = target[DefaultInfo].files.to_list()
             for f in files:
                 if f.extension == "py":
                     py_info.transitive_sources.add(f)
@@ -449,7 +449,7 @@ def create_py_info(
                 info = _get_py_info(target)
                 py_info.merge_uses_shared_libraries(info.uses_shared_libraries)
             else:
-                files = target.files.to_list()
+                files = target[DefaultInfo].files.to_list()
                 for f in files:
                     py_info.merge_uses_shared_libraries(cc_helper.is_valid_shared_library_artifact(f))
                     if py_info.get_uses_shared_libraries():
diff --git a/python/private/py_wheel.bzl b/python/private/py_wheel.bzl
@@ -480,7 +480,7 @@ def _py_wheel_impl(ctx):
         args.add("--no_compress")
 
     for target, filename in ctx.attr.extra_distinfo_files.items():
-        target_files = target.files.to_list()
+        target_files = target[DefaultInfo].files.to_list()
         if len(target_files) != 1:
             fail(
                 "Multi-file target listed in extra_distinfo_files %s",
@@ -493,7 +493,7 @@ def _py_wheel_impl(ctx):
         )
 
     for target, filename in ctx.attr.data_files.items():
-        target_files = target.files.to_list()
+        target_files = target[DefaultInfo].files.to_list()
         if len(target_files) != 1:
             fail(
                 "Multi-file target listed in data_files %s",
diff --git a/python/uv/private/uv_toolchain.bzl b/python/uv/private/uv_toolchain.bzl
@@ -24,7 +24,7 @@ def _uv_toolchain_impl(ctx):
     uv = ctx.attr.uv
 
     default_info = DefaultInfo(
-        files = uv.files,
+        files = uv[DefaultInfo].files,
         runfiles = uv[DefaultInfo].default_runfiles,
     )
     uv_toolchain_info = UvToolchainInfo(
diff --git a/sphinxdocs/private/sphinx.bzl b/sphinxdocs/private/sphinx.bzl
@@ -386,7 +386,7 @@ def _sphinx_source_tree_impl(ctx):
         _relocate(orig_file)
 
     for src_target, dest in ctx.attr.renamed_srcs.items():
-        src_files = src_target.files.to_list()
+        src_files = src_target[DefaultInfo].files.to_list()
         if len(src_files) != 1:
             fail("A single file must be specified to be renamed. Target {} " +
                  "generate {} files: {}".format(

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ def _uv_toolchain_impl(ctx):`
`24`	`24`	`uv = ctx.attr.uv`
`25`	`25`
`26`	`26`	`default_info = DefaultInfo(`
`27`		`- files = uv.files,`
	`27`	`+ files = uv[DefaultInfo].files,`
`28`	`28`	`runfiles = uv[DefaultInfo].default_runfiles,`
`29`	`29`	`)`
`30`	`30`	`uv_toolchain_info = UvToolchainInfo(`