diff --git a/dak/check_archive.py b/dak/check_archive.py index 348344bc0e2f5e8aade64551d3fb9b39fda0ac83..62e111a5f45bc5a9dc1ceb4b34daf597700dcf18 100644 --- a/dak/check_archive.py +++ b/dak/check_archive.py @@ -219,8 +219,8 @@ def check_override(): q = session.execute(""" SELECT DISTINCT b.package FROM binaries b, bin_associations ba WHERE b.id = ba.bin AND ba.suite = :suiteid AND NOT EXISTS - (SELECT 1 FROM override o WHERE o.suite = :suiteid AND o.package = b.package)""" - % {'suiteid': suite.suite_id}) + (SELECT 1 FROM override o WHERE o.suite = :suiteid AND o.package = b.package)""", + {'suiteid': suite.suite_id}) for j in q.fetchall(): print(j[0]) @@ -228,8 +228,8 @@ SELECT DISTINCT b.package FROM binaries b, bin_associations ba q = session.execute(""" SELECT DISTINCT s.source FROM source s, src_associations sa WHERE s.id = sa.source AND sa.suite = :suiteid AND NOT EXISTS - (SELECT 1 FROM override o WHERE o.suite = :suiteid and o.package = s.source)""" - % {'suiteid': suite.suite_id}) + (SELECT 1 FROM override o WHERE o.suite = :suiteid and o.package = s.source)""", + {'suiteid': suite.suite_id}) for j in q.fetchall(): print(j[0]) diff --git a/dak/process_policy.py b/dak/process_policy.py index a274a8f66023af5633e175789cdc4f603e9b2080..f3c3b7a0377171d9101570e9eb06659f356c864c 100644 --- a/dak/process_policy.py +++ b/dak/process_policy.py @@ -177,7 +177,7 @@ def comment_accept(upload, srcqueue, comments, transaction): source_component = source_component_func(upload.source) if upload.target_suite.suite_name in cnf.value_list('Dinstall::ThrowAwayNewBinarySuites') and \ source_component.component_name in cnf.value_list('Dinstall::ThrowAwayNewBinaryComponents'): - throw_away_binaries = True + throw_away_binaries = True for suite in all_target_suites: debug_suite = suite.debug_suite diff --git a/daklib/checks.py b/daklib/checks.py index 53c757fcdef08fc8745dce20fa91088531f5bbfc..558b2fbc3c52ee3d5f7ff890444410004b1f691a 100644 --- a/daklib/checks.py +++ b/daklib/checks.py @@ -476,7 +476,7 @@ class BinaryCheck(Check): raise Reject('{0}: APT could not parse {1} field'.format(fn, field)) for group in depends: if not allow_alternatives and len(group) != 1: - raise Reject('{0}: {1}: alternatives are not allowed'.format(fn)) + raise Reject('{0}: {1}: alternatives are not allowed'.format(fn, field)) for dep_pkg, dep_ver, dep_rel in group: if dep_rel not in allow_relations: raise Reject('{}: {}: depends on {}, but only relations {} are allowed for this field'.format(fn, field, " ".join(dep_pkg, dep_rel, dep_ver), allow_relations)) @@ -557,7 +557,7 @@ class SourceCheck(Check): if is_orig: upstream_match = re_field_version_upstream.match(version) if not upstream_match: - raise Reject('{0}: Source package includes upstream tarball, but {0} has no Debian revision.'.format(filename, version)) + raise Reject('{0}: Source package includes upstream tarball, but {1} has no Debian revision.'.format(filename, version)) version = upstream_match.group('upstream') version_match = re_field_version.match(version) version_without_epoch = version_match.group('without_epoch')