bazel-contrib
diff --git a/‎CHANGELOG.md
Copy file name to clipboardExpand all lines: CHANGELOG.md
+2Lines changed: 2 additions & 0 deletions b/‎CHANGELOG.md
Copy file name to clipboardExpand all lines: CHANGELOG.md
+2Lines changed: 2 additions & 0 deletions
diff --git a/‎gazelle/modules_mapping/def.bzl
Copy file name to clipboardExpand all lines: gazelle/modules_mapping/def.bzl
+8-4Lines changed: 8 additions & 4 deletions b/‎gazelle/modules_mapping/def.bzl
Copy file name to clipboardExpand all lines: gazelle/modules_mapping/def.bzl
+8-4Lines changed: 8 additions & 4 deletions
diff --git a/‎gazelle/modules_mapping/generator.py
Copy file name to clipboardExpand all lines: gazelle/modules_mapping/generator.py
+1Lines changed: 1 addition & 0 deletions b/‎gazelle/modules_mapping/generator.py
Copy file name to clipboardExpand all lines: gazelle/modules_mapping/generator.py
+1Lines changed: 1 addition & 0 deletions
@@ -59,6 +59,8 @@ Unreleased changes template.
 * (pypi) The `ppc64le` is now pointing to the right target in the `platforms` package.
 * (gazelle) No longer incorrectly merge `py_binary` targets during partial updates in 
   `file` generation mode. Fixed in [#2619](https://github.com/bazelbuild/rules_python/pull/2619).
+* (gazelle) Don't collapse depsets to a list or into args when generating the modules mapping file.
+  Support spilling modules mapping args into a params file.
 
 {#v0-0-0-added}
 ### Added
 
@@ -25,18 +25,22 @@ module name doesn't match the wheel distribution name.
 
 def _modules_mapping_impl(ctx):
     modules_mapping = ctx.actions.declare_file(ctx.attr.modules_mapping_name)
-    args = ctx.actions.args()
     all_wheels = depset(
         [whl for whl in ctx.files.wheels],
         transitive = [dep[DefaultInfo].files for dep in ctx.attr.wheels] + [dep[DefaultInfo].data_runfiles.files for dep in ctx.attr.wheels],
     )
-    args.add("--output_file", modules_mapping.path)
+
+    args = ctx.actions.args()
+    args.use_param_file(param_file_arg = "@%s")
+    args.set_param_file_format(format = "multiline")
     if ctx.attr.include_stub_packages:
         args.add("--include_stub_packages")
+    args.add("--output_file", modules_mapping)
     args.add_all("--exclude_patterns", ctx.attr.exclude_patterns)
-    args.add_all("--wheels", [whl.path for whl in all_wheels.to_list()])
+    args.add_all("--wheels", all_wheels)
+
     ctx.actions.run(
-        inputs = all_wheels.to_list(),
+        inputs = all_wheels,
         outputs = [modules_mapping],
         executable = ctx.executable._generator,
         arguments = [args],
 
@@ -152,6 +152,7 @@ def data_has_purelib_or_platlib(path):
     parser = argparse.ArgumentParser(
         prog="generator",
         description="Generates the modules mapping used by the Gazelle manifest.",
+        fromfile_prefix_chars="@",
     )
     parser.add_argument("--output_file", type=str)
     parser.add_argument("--include_stub_packages", action="store_true")
Original file line number	Diff line number	Diff line change
`@@ -152,6 +152,7 @@ def data_has_purelib_or_platlib(path):`
`152`	`152`	`parser = argparse.ArgumentParser(`
`153`	`153`	`prog="generator",`
`154`	`154`	`description="Generates the modules mapping used by the Gazelle manifest.",`
	`155`	`+ fromfile_prefix_chars="@",`
`155`	`156`	`)`
`156`	`157`	`parser.add_argument("--output_file", type=str)`
`157`	`158`	`parser.add_argument("--include_stub_packages", action="store_true")`