diff options
author | Greg Estren <gregce@google.com> | 2016-05-31 20:31:19 +0000 |
---|---|---|
committer | Dmitry Lomov <dslomov@google.com> | 2016-06-01 08:14:00 +0000 |
commit | 6c11b66b13c03fd125b4d215266d30288395dfb9 (patch) | |
tree | 44f21c7a28ba1b2e8fe915a33d1fa7eb3798a918 /site/blog/index.md | |
parent | 9ab8c9da0c3acf48dd029f02b5ffb20f3de59cb3 (diff) |
Fix a bug in SkyframeExecutor.getConfigurations() for dynamic
configurations:
That calls getDynamicConfigOptions() to get the right BuildOptions
for a target, but that fails to trim the option's FragmentOptions.
That means, for example, a BuildConfiguration could get constructed
with just the CppConfiguration fragment but with a BuildOptions
that still has PythonOptions.
This isn't just sloppy. It breaks Bazel tests that use
SkyframeExecutor.getConfiguredTargetForTesting (which follows this
code path) to compare against the production code path (which
already properly trims deps). So two configured targets that should be
equal won't be because their BuildConfigurations won't match.
This is also a prerequisite change for trimming top-level configurations,
coming up soon.
TESTED: BuildViewTest#testNewActionsAreDifferentAndDontConflict (and other
tests) pass with this change + the still-pending top-level-trimming change.
--
MOS_MIGRATED_REVID=123676990
Diffstat (limited to 'site/blog/index.md')
0 files changed, 0 insertions, 0 deletions