diff options
| author | David Kalnischkies <david@kalnischkies.de> | 2024-05-16 14:51:28 +0000 |
|---|---|---|
| committer | David Kalnischkies <david@kalnischkies.de> | 2024-05-16 20:12:51 +0000 |
| commit | f404c8e06e07c2a89efb0f1ba997a7f1c0c472db (patch) | |
| tree | d7e55ba92e7436c20d6108875670fe82f731097f /test/integration/framework | |
| parent | b1c3039dae19553bff19615fbb4d116aeaa644a7 (diff) | |
Do not save new if we have already installed providers
Companion of the previous commit that uses the same reasoning, but from
the other side of the logic (as we start from reverse deps here) so that
we are dropping the useless NEW from the solution.
Ideally this code would not live in -private, but in -pkg, but there is
not really a pre-existent good central place for this currently as a
sort-of fixup for our default resolver given this is the result of
resolver decision to not do something which had ripple effects that
aren't reverted along with it.
It is a bit unlikely that this is fixing all instances that this bug
has given we have dealt with a few instances of it already as they are
relatively obscure situations to clean up after and the cost of being
wrong (= autoremover generating breaks) is much higher than being
correct (= no pointless install – that is likely installed later anyhow
after the temporary automatic hold is lifted). And by virtue of being
the result of mostly temporary situations its also unlikely to be
debugable in hindsight months later; so if there is another situation,
its better to deal with it in a new bugreport.
Closes: #839546
Diffstat (limited to 'test/integration/framework')
0 files changed, 0 insertions, 0 deletions
