aboutsummaryrefslogtreecommitdiff
path: root/debian/patches/pep8-accommodations.diff
diff options
context:
space:
mode:
Diffstat (limited to 'debian/patches/pep8-accommodations.diff')
-rw-r--r--debian/patches/pep8-accommodations.diff35
1 files changed, 35 insertions, 0 deletions
diff --git a/debian/patches/pep8-accommodations.diff b/debian/patches/pep8-accommodations.diff
new file mode 100644
index 0000000..68ee5e4
--- /dev/null
+++ b/debian/patches/pep8-accommodations.diff
@@ -0,0 +1,35 @@
+Description: Various accommodations so that the PEP 8 tests don't cause
+ autopkgtests to fail. These include fixing the --exclude argument, adding
+ more files to the exclude pattern, and ignoring E402, which is bogus; see
+ https://github.com/PyCQA/pep8/issues/394. I also fixed one legitimate
+ looking violation.
+Author: Barry Warsaw <barry@ubuntu.com>
+Bug: https://bugs.launchpad.net/aptdaemon/+bug/1486646
+
+--- a/tests/test_pep8.py
++++ b/tests/test_pep8.py
+@@ -30,9 +30,11 @@
+
+ def test(self):
+ """Check if the source code matches the PEP8 style conventions."""
+- subprocess.check_call(["pep8", "--statistics", "--show-source",
+- "--show-pep8", "--exclude", "pkenums.py",
+- "aptdaemon", "tests"])
++ subprocess.check_call([
++ "pep8", "--statistics", "--show-source",
++ "--show-pep8", "--exclude",
++ "pkenums.py,aptdaemon,tests,debian,doc,.pc,gtk3-demo.py,setup.py",
++ "--ignore=E402"])
+
+
+ if __name__ == "__main__":
+--- a/apport/aptdaemon.py
++++ b/apport/aptdaemon.py
+@@ -28,6 +28,7 @@
+
+ apt_pkg.init()
+
++
+ def add_info(report):
+ """Collect and append additional information about a crash.
+