On Wed, 29 Oct 2014 02:29:07 -0700 Zac Medico <zmed...@gentoo.org> wrote:
> The fixes various forms of buggy behavior involving virtuals handling > and EAPI 5 subslots. The code at fault is part of the support for > bug #141118 (virtuals lookahead). This code was written long before > subslots existed, and because of them it needs to be updated now. > > Previously, the relevant virtuals code would only select one virtual > from each slot for lookahead. Now, it needs to select one from each > slot/subslot pair. The buggy forms of behavior fixed by this change > can be difficult to reproduce, since they only express themselves > under some rare circumstances. > > In addition to the _dep_check_composite_db fixes, there is also a > related fix inside _expand_new_virtuals, which is required so that > _slot_operator_update_probe can reliably check whether or not a > particular input atom was selected. Without this change, the > _dep_check_composite_db changes will cause the test from commit > d3be49fe6827aa1974856dffe6d5a1aca80a7bed to fail in a way that is > very similar to the failure reported in bug #526160. > > Fixes: d3be49fe6827 ("depgraph: fix bug #526160") > X-Gentoo-Bug: 526160 > X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=526160 > --- > pym/_emerge/depgraph.py | 118 > ++++++++++++++++++------ > pym/portage/dep/dep_check.py | 8 +- > pym/portage/tests/resolver/test_virtual_slot.py | 11 ++- 3 files > changed, 106 insertions(+), 31 deletions(-) > not that I'll pretend to understand all the code changes, but looks good merge away -- Brian Dolbec <dolsen>