Remove an unnecessary if-statement

Signed-off-by: Niels Thykier <niels@thykier.net>
ubuntu/rebased
Niels Thykier 6 years ago
parent 9971714e27
commit 68407c53c0
No known key found for this signature in database
GPG Key ID: A65B78DBE67C7AAC

@ -239,21 +239,20 @@ class InstallabilitySolver(object):
oldcons = set(chain.from_iterable(universe.negative_dependencies_of(r) for r in rms)) oldcons = set(chain.from_iterable(universe.negative_dependencies_of(r) for r in rms))
newcons = set(chain.from_iterable(universe.negative_dependencies_of(a) for a in adds)) newcons = set(chain.from_iterable(universe.negative_dependencies_of(a) for a in adds))
oldcons -= newcons oldcons -= newcons
if oldcons: # Some of the old binaries have "conflicts" that will
# Some of the old binaries have "conflicts" that will # be removed.
# be removed. for o in ifilter_only(ptable, oldcons):
for o in ifilter_only(ptable, oldcons): # "key" removes a conflict with one of
# "key" removes a conflict with one of # "other"'s binaries, so it is probably a good
# "other"'s binaries, so it is probably a good # idea to migrate "key" before "other"
# idea to migrate "key" before "other" other = ptable[o]
other = ptable[o] if other == key:
if other == key: # "Self-conflicts" => ignore
# "Self-conflicts" => ignore continue
continue if debug_solver and other not in order[key]['before']: # pragma: no cover
if debug_solver and other not in order[key]['before']: # pragma: no cover self.logger.debug("Conflict induced order: %s before %s", key, other)
self.logger.debug("Conflict induced order: %s before %s", key, other) order[key]['before'].add(other)
order[key]['before'].add(other) order[other]['after'].add(key)
order[other]['after'].add(key)
self._compute_group_order_rms(rms, order, key, ptable, going_out) self._compute_group_order_rms(rms, order, key, ptable, going_out)
self._compute_group_order_adds(adds, order, key, ptable, going_out, going_in) self._compute_group_order_adds(adds, order, key, ptable, going_out, going_in)

Loading…
Cancel
Save