Revert the change from bug 828136, since it prevents solving of some blockers unless --update and --deep are specified as reported in bug 833014.
Bug: https://bugs.gentoo.org/833014 Reverts: a7289ac0eaaa0d435bf6d9bfb2724a6b39adcbee Signed-off-by: Zac Medico <zmed...@gentoo.org> --- lib/portage/dep/dep_check.py | 6 +----- .../tests/resolver/test_installkernel.py | 20 +------------------ .../resolver/test_unecessary_slot_upgrade.py | 11 ---------- 3 files changed, 2 insertions(+), 35 deletions(-) diff --git a/lib/portage/dep/dep_check.py b/lib/portage/dep/dep_check.py index 8ca4c0b9d..9fccda08b 100644 --- a/lib/portage/dep/dep_check.py +++ b/lib/portage/dep/dep_check.py @@ -376,7 +376,6 @@ def dep_zapdeps( # c) contains masked installed packages # d) is the first item - no_new_slots = [] preferred_in_graph = [] preferred_installed = preferred_in_graph preferred_any_slot = preferred_in_graph @@ -392,7 +391,6 @@ def dep_zapdeps( # unsat_use_* must come after preferred_non_installed # for correct ordering in cases like || ( foo[a] foo[b] ). choice_bins = ( - no_new_slots, preferred_in_graph, preferred_non_installed, unsat_use_in_graph, @@ -691,9 +689,7 @@ def dep_zapdeps( other.append(this_choice) else: if all_use_satisfied: - if new_slot_count == 0 and not want_update: - no_new_slots.append(this_choice) - elif all_in_graph: + if all_in_graph: preferred_in_graph.append(this_choice) elif all_installed: if all_installed_slots: diff --git a/lib/portage/tests/resolver/test_installkernel.py b/lib/portage/tests/resolver/test_installkernel.py index b73bbe5bb..5909b53aa 100644 --- a/lib/portage/tests/resolver/test_installkernel.py +++ b/lib/portage/tests/resolver/test_installkernel.py @@ -58,25 +58,8 @@ class InstallKernelTestCase(TestCase): ), ], ), - # Demonstrate bug 833014, where the calculation fails unless + # Test bug 833014, where the calculation failed unless # --update and --deep are specified. - ResolverPlaygroundTestCase( - [ - "sys-kernel/installkernel-systemd-boot", - "sys-kernel/gentoo-kernel-bin", - ], - ambiguous_merge_order=True, - success=False, - mergelist=[ - "sys-kernel/installkernel-systemd-boot-1", - "sys-kernel/gentoo-kernel-bin-5.15.23", - "virtual/dist-kernel-5.15.23", - ( - "!sys-kernel/installkernel-gentoo", - "!sys-kernel/installkernel-systemd-boot", - ), - ], - ), ResolverPlaygroundTestCase( [ "sys-kernel/installkernel-systemd-boot", @@ -84,7 +67,6 @@ class InstallKernelTestCase(TestCase): ], ambiguous_merge_order=True, success=True, - options={"--deep": True, "--update": True}, mergelist=[ "virtual/dist-kernel-5.15.23", "sys-kernel/installkernel-systemd-boot-1", diff --git a/lib/portage/tests/resolver/test_unecessary_slot_upgrade.py b/lib/portage/tests/resolver/test_unecessary_slot_upgrade.py index f8b8b346a..a89ebdb67 100644 --- a/lib/portage/tests/resolver/test_unecessary_slot_upgrade.py +++ b/lib/portage/tests/resolver/test_unecessary_slot_upgrade.py @@ -26,13 +26,6 @@ class UnnecessarySlotrUpgradeTestCase(TestCase): test_cases = ( # Test bug 828136, where an unnecessary python slot upgrade # was triggered. - ResolverPlaygroundTestCase( - [ - "app-misc/a", - ], - success=True, - mergelist=("app-misc/a-1",), - ), ResolverPlaygroundTestCase( [ "app-misc/a", @@ -42,10 +35,6 @@ class UnnecessarySlotrUpgradeTestCase(TestCase): "dev-lang/python-3.10", "app-misc/a-1", ), - options={ - "--deep": True, - "--update": True, - }, ), ) -- 2.34.1