tree b7397ff126872174629bf816020a10da298b3fb1
parent fcdd61859c9d11dd86c395956071c4b7e0647ded
author Takuto Ikuta <tikuta@google.com> 1782812455 +0900
committer gn-scoped@luci-project-accounts.iam.gserviceaccount.com <gn-scoped@luci-project-accounts.iam.gserviceaccount.com> 1782887321 -0700

Reland "Propagate public_deps outputs via dependency_output for non-binary targets"

This relands the change with a fix to prevent propagating dependency
outputs of data-only targets (like bundle_data) via public_deps.

Data-only targets are runtime dependencies and should not be propagated
as implicit (build-time) dependencies, to avoid having unnecessary
inputs.

Also I sent deps reduction to chromium as https://crrev.com/c/8023463 to
avoid the limit of maximum allowed number of files in remote link.

Original change's description:
> Revert "Propagate public_deps outputs via dependency_output for non-binary targets"
>
> This reverts commit 3bccda11b7bc16d89994f34da1e6604889f88f86.
>
> Reason for revert: This doesn't pass chromium's CQ.
>
> Failure Link:
> https://ci.chromium.org/ui/p/chromium/builders/try/linux-chromeos-rel/2928808?
>
> Original change's description:
> > Propagate public_deps outputs via dependency_output for non-binary targets
> >
> > For non-binary targets (like actions, groups, etc.), their dependency
> > outputs (stamp or phony targets) now transitively depend on the
> > dependency outputs of their public_deps.
> >
> > This ensures that dependents of the current target will also implicitly
> > depend on the outputs of its public_deps, preventing the inflation of
> > implicit inputs on each build line.
> >
> > Also updated various unit tests and added a new complex test case to
> > verify this behavior.
> >
> > This removes the necessity of some redundant BUILD.gn config changes
> > like
> > https://crrev.com/c/8003676/8/front_end/models/ai_assistance/skills/BUILD.gn
> > to propagate outputs of actions to indirect dependents.
> >
> > Bug: 513105742
> > Change-Id: I14de8cea4e4add977cacdc88386737aedda16f86
> > Reviewed-on: https://gn-review.googlesource.com/c/gn/+/23520
> > Commit-Queue: Takuto Ikuta <tikuta@google.com>
> > Reviewed-by: Matt Stark <msta@google.com>
>
> Bug: 513105742
> Change-Id: I35977d90e1f59bbd37fdd3d3c1dc5deedbb3d417
> Reviewed-on: https://gn-review.googlesource.com/c/gn/+/23541
> Reviewed-by: Junji Watanabe <jwata@google.com>
> Commit-Queue: Takuto Ikuta <tikuta@google.com>

# Not skipping CQ checks because this is a reland.

Bug: 513105742
Change-Id: I1599385b8902a3fe0a471afd30975a85f4520670
Reviewed-on: https://gn-review.googlesource.com/c/gn/+/23560
Reviewed-by: Junji Watanabe <jwata@google.com>
Commit-Queue: Takuto Ikuta <tikuta@google.com>