Merge lp:~mvo/software-properties/dep8 into lp:software-properties

[Michael Vogt]

add dep8 testing header

[Martin Pitt (pitti)]

Please note that you also need to add a "XS-Testsuite: autopkgtest" header to the source stanza in debian/control for this to work.

17 +(cd tests ; xvfb-run make )

Why do you run this in a subshell? That shouldn't be necessary, but of course does not actually hurt either.

review: Needs Fixing

[lp:~mvo/software-properties/dep8]

794. By Michael Vogt on 2012-09-10

debian/control: add missing XS-Testsuite header

795. By Michael Vogt on 2012-09-10

debian/tests/run-tests: no need to run a subshell, thanks to Martin Pitt

[Michael Vogt (mvo)]

Thanks Martin, I fixed both issues and it should be ready for re-review or merge.

review: Resubmit

[Martin Pitt (pitti)]

LGTM, thanks! Please upload.

review: Approve

=== modified file 'debian/control'
--- debian/control	2012-07-18 13:12:13 +0000
+++ debian/control	2012-09-10 07:35:23 +0000
@@ -13,6 +13,7 @@
                dh-translations
 Standards-Version: 3.9.3
 Vcs-Bzr: http://code.launchpad.net/~ubuntu-core-dev/software-properties/main
+XS-Testsuite: autopkgtest
 
 Package: python-software-properties
 Section: python

=== added directory 'debian/tests'
=== added file 'debian/tests/control'
--- debian/tests/control	1970-01-01 00:00:00 +0000
+++ debian/tests/control	2012-09-10 07:35:23 +0000
@@ -0,0 +1,2 @@
+Tests: run-tests
+Depends: @, xvfb

=== added file 'debian/tests/run-tests'
--- debian/tests/run-tests	1970-01-01 00:00:00 +0000
+++ debian/tests/run-tests	2012-09-10 07:35:23 +0000
@@ -0,0 +1,7 @@
+#!/bin/sh
+
+set -e
+
+cd tests
+xvfb-run make
+