mirror of
https://git.launchpad.net/~ubuntu-release/britney/+git/britney2-ubuntu
synced 2025-05-13 11:31:38 +00:00
Fix pep8 issues in check_target_suite_source_pkg_consistency
Signed-off-by: Niels Thykier <niels@thykier.net>
This commit is contained in:
parent
c9d523da93
commit
5a38e00996
@ -336,6 +336,7 @@ def old_libraries(mi_factory, suite_info, outofsync_arches=frozenset()):
|
|||||||
removals.append(mi_factory.generate_removal_for_cruft_item(pkg.pkg_id))
|
removals.append(mi_factory.generate_removal_for_cruft_item(pkg.pkg_id))
|
||||||
return removals
|
return removals
|
||||||
|
|
||||||
|
|
||||||
def check_target_suite_source_pkg_consistency(suite_info, comment, logger):
|
def check_target_suite_source_pkg_consistency(suite_info, comment, logger):
|
||||||
sources_t = suite_info.target_suite.sources
|
sources_t = suite_info.target_suite.sources
|
||||||
binaries_t = suite_info.target_suite.binaries
|
binaries_t = suite_info.target_suite.binaries
|
||||||
@ -348,21 +349,24 @@ def check_target_suite_source_pkg_consistency(suite_info, comment, logger):
|
|||||||
pkg = binaries_t[arch][pkg_name]
|
pkg = binaries_t[arch][pkg_name]
|
||||||
src = pkg.source
|
src = pkg.source
|
||||||
|
|
||||||
if src not in sources_t:
|
if src not in sources_t: # pragma: no cover
|
||||||
issues_found = True
|
issues_found = True
|
||||||
logger.error("inconsistency found (%s): src %s not in target, target has pkg %s with source %s"%(comment,src,pkg_name,src))
|
logger.error("inconsistency found (%s): src %s not in target, target has pkg %s with source %s" % (
|
||||||
|
comment, src, pkg_name, src))
|
||||||
|
|
||||||
for src in sources_t:
|
for src in sources_t:
|
||||||
source_data = sources_t[src]
|
source_data = sources_t[src]
|
||||||
for pkg_id in source_data.binaries:
|
for pkg_id in source_data.binaries:
|
||||||
binary, _, parch = pkg_id
|
binary, _, parch = pkg_id
|
||||||
if binary not in binaries_t[parch]:
|
if binary not in binaries_t[parch]: # pragma: no cover
|
||||||
issues_found = True
|
issues_found = True
|
||||||
logger.error("inconsistency found (%s): binary %s from source %s not in binaries_t[%s]"%(comment,binary,src,parch))
|
logger.error("inconsistency found (%s): binary %s from source %s not in binaries_t[%s]" % (
|
||||||
|
comment, binary, src, parch))
|
||||||
|
|
||||||
if issues_found:
|
if issues_found:
|
||||||
raise AssertionError("inconsistencies found in target suite")
|
raise AssertionError("inconsistencies found in target suite")
|
||||||
|
|
||||||
|
|
||||||
def is_nuninst_asgood_generous(constraints, architectures, old, new, break_arches=frozenset()):
|
def is_nuninst_asgood_generous(constraints, architectures, old, new, break_arches=frozenset()):
|
||||||
"""Compares the nuninst counters and constraints to see if they improved
|
"""Compares the nuninst counters and constraints to see if they improved
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user