From d3fa43b0aad66b7e72edf7959e80c1e3f92a63fb Mon Sep 17 00:00:00 2001 From: Sylvestre Ledru Date: Fri, 1 Jan 2016 22:44:57 +0100 Subject: [PATCH 1/3] Use the list of authors from github --- AUTHORS | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/AUTHORS b/AUTHORS index a4e183d..1618b6f 100644 --- a/AUTHORS +++ b/AUTHORS @@ -1,6 +1,3 @@ -Paul R. Tagliamonte -Leo Cavaille -Sylvestre Ledru -Matthias Klumpp -Jon Severinsson -Clement Schreiner +See on github: +https://github.com/opencollab/debile/graphs/contributors + From 6ce763bbb51aead21a907bc191533e575432bbec Mon Sep 17 00:00:00 2001 From: Sylvestre Ledru Date: Mon, 1 Aug 2016 10:07:15 +0200 Subject: [PATCH 2/3] print test --- debile/slave/runners/pep8.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/debile/slave/runners/pep8.py b/debile/slave/runners/pep8.py index bb2ded7..e6641b2 100644 --- a/debile/slave/runners/pep8.py +++ b/debile/slave/runners/pep8.py @@ -28,6 +28,12 @@ def pep8(dsc, analysis): run_command(["dpkg-source", "-x", dsc, "source-pep8"]) with cd('source-pep8'): out, _, ret = run_command(['pep8', '.']) + print dsc + print "out : " + print out + print "ret : " + print ret + failed = ret != 0 for issue in parse_pep8(out.splitlines()): From 6d55580ef36597d9a8bfb38a252b3f8446c92769 Mon Sep 17 00:00:00 2001 From: Sylvestre Ledru Date: Mon, 1 Aug 2016 11:05:15 +0200 Subject: [PATCH 3/3] manage the case when out is empty --- debile/slave/runners/pep8.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/debile/slave/runners/pep8.py b/debile/slave/runners/pep8.py index e6641b2..6a4a5e5 100644 --- a/debile/slave/runners/pep8.py +++ b/debile/slave/runners/pep8.py @@ -35,9 +35,11 @@ def pep8(dsc, analysis): print ret failed = ret != 0 - - for issue in parse_pep8(out.splitlines()): - analysis.results.append(issue) + if out is not None: + for issue in parse_pep8(out.splitlines()): + analysis.results.append(issue) + else: + print "Output of pep8 checker command is None!" return (analysis, out, failed, None, None)