britney.py: Remove unused named parameter

The "include_hijacked" parameter of "_compute_groups" was always
false, so there was little point in having it.

Signed-off-by: Niels Thykier <niels@thykier.net>
debian
Niels Thykier 9 years ago
parent 22cef671a5
commit efd3359aba

@ -1855,7 +1855,7 @@ class Britney(object):
def _compute_groups(self, source_name, suite, migration_architecture, def _compute_groups(self, source_name, suite, migration_architecture,
is_removal, include_hijacked=False, is_removal,
allow_smooth_updates=True, allow_smooth_updates=True,
removals=frozenset()): removals=frozenset()):
"""Compute the groups of binaries being migrated by item """Compute the groups of binaries being migrated by item
@ -1874,8 +1874,6 @@ class Britney(object):
architecture). [Same as item.architecture, where available] architecture). [Same as item.architecture, where available]
* "is_removal" is a boolean determining if this is a removal * "is_removal" is a boolean determining if this is a removal
or not [Same as item.is_removal, where available] or not [Same as item.is_removal, where available]
* "include_hijacked" determines whether hijacked binaries should
be included in results or not. (defaults: False)
* "allow_smooth_updates" is a boolean determing whether smooth- * "allow_smooth_updates" is a boolean determing whether smooth-
updates are permitted in this migration. When set to False, updates are permitted in this migration. When set to False,
the "smoothbins" return value will always be the empty set. the "smoothbins" return value will always be the empty set.
@ -1932,8 +1930,8 @@ class Britney(object):
if migration_architecture == 'source' and is_removal and binary not in binaries_t[parch][0]: if migration_architecture == 'source' and is_removal and binary not in binaries_t[parch][0]:
continue continue
if (not include_hijacked # Do not include hijacked binaries
and binaries_t[parch][0][binary][SOURCE] != source_name): if binaries_t[parch][0][binary][SOURCE] != source_name:
continue continue
bins.append(pkg_id) bins.append(pkg_id)
@ -2014,8 +2012,7 @@ class Britney(object):
if migration_architecture not in ['source', parch]: if migration_architecture not in ['source', parch]:
continue continue
if (not include_hijacked if self.binaries[suite][parch][0][binary][SOURCE] != source_name:
and self.binaries[suite][parch][0][binary][SOURCE] != source_name):
# This binary package has been hijacked by some other source. # This binary package has been hijacked by some other source.
# So don't add it as part of this update. # So don't add it as part of this update.
# #

Loading…
Cancel
Save