diff options
| author |  Greg Estren <gregce@google.com> | 2016-06-01 23:22:48 +0000 |
|---|---|---|
| committer |  Dmitry Lomov <dslomov@google.com> | 2016-06-02 11:44:02 +0000 |
| commit | 7971e674fadbb41123271a86a2c4418e8c0f42a6 (patch) | |
| tree | 3a9d3c16aaa4397228ed2614843a95794eb687b8 /src/java_tools | |
| parent | 8c77715e6fa8f536dd16c7b5af14d367e1e0e96c (diff) | |
Dynamic configurations: trim top-level targets, too.
Right now, configuration trimming happens in
ConfiguredTargetFunction.computeDependencies. This means only
the deps of other targets get trimmed.
With this change, every ConfiguredTarget gets its configuration
accurately trimmed, regardless of where it comes from or what
it's used for.
In practice, there could still be other code paths that
instantiate ConfiguredTargetValue.key without pre-trimming.
We'll have to tackle those as we hit them.
Also cleaned up some symbol naming in BuildView.update to try
to make the logic flow clearer.
TESTED: BuildViewTest#testNewActionsAreDifferentAndDontConflict now
passes with dynamic configs (among others)
--
MOS_MIGRATED_REVID=123807892
Diffstat (limited to 'src/java_tools')
0 files changed, 0 insertions, 0 deletions