[Qa-jenkins-scm] [jenkins.debian.net] 04/05: reproducible: breakages: rename alien_rbuild to alien_log and do a fancy recursion to check both RBUILD_PATH and LOGS_PATH

[Holger Levsen]

This is an automated email from the git hooks/post-receive script.

holger pushed a commit to branch master
in repository jenkins.debian.net.

commit 4630b5cdaab642db42a1b6ac5689afe203b3987d
Author: Mattia Rizzolo <mattia at mapreri.org>
Date:   Sat May 30 03:29:17 2015 +0200

    reproducible: breakages: rename alien_rbuild to alien_log and do a fancy recursion to check both RBUILD_PATH and LOGS_PATH
---
 bin/reproducible_breakages.py | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/bin/reproducible_breakages.py b/bin/reproducible_breakages.py
index 2f26eb3..5d4da63 100755
--- a/bin/reproducible_breakages.py
+++ b/bin/reproducible_breakages.py
@@ -127,10 +127,15 @@ def pbuilder_dep_fail():
     return bad_pkgs
 
 
-def alien_rbuild():
-    log.info('running alien_rbuild check...')
+def alien_log(directory=None):
+    if directory == None:
+        bad_files = []
+        for path in RBUILD_PATH, LOGS_PATH:
+            bad_files.extend(alien_log(path))
+        return bad_files
+    log.info('running alien_log check over ' + directory + '...')
     query = '''SELECT s.name
-               FROM sources AS s JOIN results AS r on r.package_id=s.id
+               FROM sources AS s JOIN results AS r ON r.package_id=s.id
                WHERE r.status != "" AND s.name="{pkg}" AND s.suite="{suite}"
                AND s.architecture="{arch}"
                ORDER BY s.name ASC, s.suite DESC'''
@@ -229,8 +234,8 @@ def _gen_section(header, pkgs, entries=None):
 def gen_html():
     html = ''
     # files that should not be there (e.g. removed package without cleanup)
-    html += _gen_section('rbuild file that should not be there', None,
-                         entries=alien_rbuild())
+    html += _gen_section('log files that should not be there', None,
+                         entries=alien_log())
     html += _gen_section('debbindiff files that should not be there:', None,
                          entries=alien_dbd())
     html += _gen_section('rb-pkg pages that should not be there:', None,

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/qa/jenkins.debian.net.git