Only traverse rules definitely providing JavaInfo in android_neverlink_aspect

sfreilich · copybara-github · commit 149c537b897e · 2026-01-28T12:25:52.000-08:00
This improves build performance by avoiding creating aspect targets for transitive dependencies that are not Java dependencies at all. The tradeoff is that it does require the participating rules to include `JavaInfo` in `rule.provides`.

PiperOrigin-RevId: 862345672
Change-Id: I3bd0665b73349b5ad83102cd7cf0aa4a150a67c9
diff --git a/rules/android_neverlink_aspect.bzl b/rules/android_neverlink_aspect.bzl
@@ -72,4 +72,5 @@ def _collect_transitive_neverlink_libs(ctx, deps, runtime_jars):
 android_neverlink_aspect = aspect(
     implementation = _android_neverlink_aspect_impl,
     attr_aspects = _ATTRS,
+    required_providers = [JavaInfo],
 )

Original file line number	Diff line number	Diff line change
`@@ -72,4 +72,5 @@ def _collect_transitive_neverlink_libs(ctx, deps, runtime_jars):`
`72`	`72`	`android_neverlink_aspect = aspect(`
`73`	`73`	`implementation = _android_neverlink_aspect_impl,`
`74`	`74`	`attr_aspects = _ATTRS,`
	`75`	`+ required_providers = [JavaInfo],`
`75`	`76`	`)`