XPost: linux.debian.bugs.dist

control: tag -1 +patch

Hello Ondřej,

On Sun, Aug 06, 2017 at 03:07:24PM +0200, Ondřej Nový wrote:

In 5.6.30. Testsuite
...
Currently, the only defined value is autopkgtest.

Which is not true, because we have autodep8. Look to: https://anonscm.debian.org/git/lintian/lintian.git/tree/checks/testsuite.pm#n60

Thanks. Seeking seconds for the patch below.

And if you want to use autodep8, you need to explicitly add this to d/control.

Actually, this depends on which autodep8 module you want to use. I know
that my elpa module runs the tests even if the Testsuite: field is
missing, for example.

diff --git a/policy/ch-controlfields.rst b/policy/ch-controlfields.rst
index 61f2b23..2bc7a07 100644
--- a/policy/ch-controlfields.rst
+++ b/policy/ch-controlfields.rst
@@ -1009,12 +1009,12 @@ reference whose name matches ``refs/dgit/*``. See the manual page of

Simple field containing a comma-separated list of values allowing test
execution environments to discover packages which provide tests.
-Currently, the only defined value is ``autopkgtest``.

-This field is automatically added to Debian source control files by
-``dpkg`` when a ``debian/tests/control`` file is present in the source -package. This field may also be used in source package control files if -needed in other situations.
+This field is automatically added to Debian binary control files by
+``dpkg``, with the value ``autopkgtest``, when a
+``debian/tests/control`` file is present in the source package. This
+field may also be used in source package control files if needed in
+other situations.

.. _s5.7:

--
Sean Whitton

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEEm5FwB64DDjbk/CSLaVt65L8GYkAFAlmh0NQACgkQaV

XPost: linux.debian.bugs.dist

Sean Whitton <spwhitton@spwhitton.name> writes:

diff --git a/policy/ch-controlfields.rst b/policy/ch-controlfields.rst
index 61f2b23..2bc7a07 100644
--- a/policy/ch-controlfields.rst
+++ b/policy/ch-controlfields.rst
@@ -1009,12 +1009,12 @@ reference whose name matches ``refs/dgit/*``. See the manual page of

Simple field containing a comma-separated list of values allowing test
execution environments to discover packages which provide tests.
-Currently, the only defined value is ``autopkgtest``.

-This field is automatically added to Debian source control files by -``dpkg`` when a ``debian/tests/control`` file is present in the source -package. This field may also be used in source package control files if -needed in other situations.
+This field is automatically added to Debian binary control files by +``dpkg``, with the value ``autopkgtest``, when a
+``debian/tests/control`` file is present in the source package. This
+field may also be used in source package control files if needed in
+other situations.

.. _s5.7:

This seems okay, so seconded, although it would be nice to be more
specific about when this is used and what its allowable values are. But
I'm also okay with leaving that for future work.

--
Russ Allbery (rra@debian.org) <http://www.eyrie.org/~eagle/>

--- SoupGate-Win32 v1.05
* Origin: fsxNet Usenet Gateway (21:1/5)

XPost: linux.debian.bugs.dist

On Sat, 26 Aug 2017 12:49:46 -0700, Sean Whitton wrote:

diff --git a/policy/ch-controlfields.rst b/policy/ch-controlfields.rst
index 61f2b23..2bc7a07 100644
--- a/policy/ch-controlfields.rst
+++ b/policy/ch-controlfields.rst
@@ -1009,12 +1009,12 @@ reference whose name matches ``refs/dgit/*``. See the manual page of

Simple field containing a comma-separated list of values allowing test
execution environments to discover packages which provide tests.
-Currently, the only defined value is ``autopkgtest``.

-This field is automatically added to Debian source control files by -``dpkg`` when a ``debian/tests/control`` file is present in the source -package. This field may also be used in source package control files if -needed in other situations.
+This field is automatically added to Debian binary control files by +``dpkg``, with the value ``autopkgtest``, when a
+``debian/tests/control`` file is present in the source package. This
+field may also be used in source package control files if needed in
+other situations.

.. _s5.7:

Seconded.


Cheers,
gregor

--
.''`. https://info.comodo.priv.at/ - Debian Developer https://www.debian.org
: :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D 85FA BB3A 6801 8649 AA06
`. `' Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
`-

-----BEGIN PGP SIGNATURE-----

iQKTBAEBCgB9FiEE0eExbpOnYKgQTYX6uzpoAYZJqgYFAlmjgBNfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldEQx RTEzMTZFOTNBNzYwQTgxMDREODVGQUJCM0E2ODAxODY0OUFBMDYACgkQuzpoAYZJ qgZKFRAAq3P8+nbj9SsgZAgp2h0AxeDpBcgpSKuEkzgSiRFRYSUEapHd/hFqaWVa Fk4i5A/nyQebyI+Nctg1zhHBlIQ7MFRoIyIGkE6FIEbh9IuVSnDusTF4Sf89A2+t PtLlssh0Yg45n5sJLtSUSIeqWQQl9GVg5WhKmn5T84ZAuJqNL19Cze7Tc0fC9nXQ aUzIWUX6jIg3TGjQclbLGZYzWLRDne+TiFm3xF42ei1lxVQkxGVGQEoT7iMTVEFc zqkSnfmgouIcd9pGssPH0WLhotj2Xmqqzc0G8b+pgdGK0vdWYeComzKnXqR4nXu9 RlEiiXkBetvuQ4rmMOu0UAJbWEv8k0nDVjA6P758KZgvzyKG0OdvG6LpugYpftCc EkLYCFgEn5HgLAc5n3PthNhfRvQD7H5EkERWWe0J4ylIvSDGd2fo25rFFhvg2mCD GGxI6x5OsdD+aYyyp5wDgih7J1czMJibcjQ0lZff6eHSSY6K9i3cgMIcshP2EKox GopcsuBU/tqxAol65laeNCaF5FscB98NLwW0b2D8tY3g7+7hw+FIk+lBJEzcw0NV CiIyVRF2JsHDNPo4cnqYAyB79C+7otLSjpTbg5spQhByh/jJZLBRM8oKpV5Zbnx8 s+7N/H0Av9JVmKgZCavAB8tbTFfql9oOCvFMZFEHtBm0kcqZzmU=
=2D2R
-----END PGP SIGNATURE-----

--- SoupGate-Win32 v1.05
* Origin: fsxNet Usenet Gateway (21:1/5)

XPost: linux.debian.bugs.dist

Hi,

2017-08-26 21:49 GMT+02:00 Sean Whitton <spwhitton@spwhitton.name>:

Actually, this depends on which autodep8 module you want to use. I know
that my elpa module runs the tests even if the Testsuite: field is
missing, for example.

i think this is true for local build environment only, but not for ci.debian.net. If you want your autodep8 tests to run on Debian CI, you
need to explicitly enable them.

--
Best regards
Ondřej Nový

Email: novy@ondrej.org
PGP: 3D98 3C52 EB85 980C 46A5 6090 3573 1255 9D1E 064B

<div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote">2017-08-26 21:49 GMT+02:00 Sean Whitton <span dir="ltr">&lt;<a href="mailto:spwhitton@spwhitton.name" target="_blank">spwhitton@spwhitton.name</a>&gt;</span>:<br><blockquote
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Actually, this depends on which autodep8 module you want to use.  I know<br>
that my elpa module runs the tests even if the Testsuite: field is<br>
missing, for example.<br></blockquote><div><br></div><div>i think this is true for local build environment only, but not for <a href="http://ci.debian.net">ci.debian.net</a>. If you want your autodep8 tests to run on Debian CI, you need to explicitly
enable them.</div><div> </div></div>-- <br><div class="gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>B

XPost: linux.debian.bugs.dist

On Wed, Aug 30, 2017 at 08:53:17PM +0200, Mattia Rizzolo wrote:

On Wed, Aug 30, 2017 at 12:56:30PM +0200, Ondrej Novy wrote:

2017-08-26 21:49 GMT+02:00 Sean Whitton <spwhitton@spwhitton.name>:

Actually, this depends on which autodep8 module you want to use. I know that my elpa module runs the tests even if the Testsuite: field is missing, for example.

i think this is true for local build environment only, but not for ci.debian.net. If you want your autodep8 tests to run on Debian CI, you need to explicitly enable them.

Indeed. Alternatively, you need to reach out to the ci.d.n admins: they maintain a manual list of packages for which to run a specific autodep8
thing (which is what has been done for all the things currently in
autodep8 to have them enabled right away for all packages).

actually it's plain whitelist of package names. autodep8 figures out
what to do by itself.

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEst7mYDbECCn80PEM/A2xu81GC94FAlmnD0MACgkQ/A2xu81G C94ThBAAkHbv4C5aOrqdXVrFITU/s++aWsb/7oCOcvgzAymVZusZYn4SRqncH7xf ue8h6Ay58Oi1qE4e2VFrl/T7CI7dYnxi3EQN/j7ZKfLn4+i4w/gzlLDrpssJavoD huzwfuThHVWTWZJZakipL8+bViXWyng4Ar/Cz4AsvO0fTZlxlL171O9SIWqsZznN m+lJupZs/VDCCyvu5jCeNhHoJYWSpwH5ETKMI8wh7nZxlgauuK6PojAG8VZufRrz czmDNviclBztyO57rEtIu1hbkKfKMzAkQEOO2humofn3jijm/Nxqt3/E6DoE6W3t 5yo+0qKT9m/ZZle+ZOJFZmp07FB1XVsQPhz9iD+LYIVmKwYv79nlXGax4xzCJnVS TlTWAHZSvwBGiQold6dIP4gf8xjaOd+VRT8w1OHqW4hAt9Brr7XSSOzTJwe3gnms z2I3uXEEWfrjaABfTkmd+MEaEQ31WxQRXxSO3Fnxvy3gEbGTol582UZ7b3zqW+xL tbyOWLYIhPkoDXIoMWIiwisoA6fJbsjOTWreR1nLQ526gfiCq28WssnwbgPZYgdK k7c/bow3RABqYsUijr56Ao1SeGdbiq7oFL/F6Ow1htDCTTNwDdVB65Qdyg3lPIhc teTDfc7D3PCY1EnMPtW0+sTRAsleF17GUbCjEiXmmFyuoq0j4Kw=
=PIyu
-----END PGP SIGNATURE-----

--- SoupGate-Win32 v1.05
* Origin: fsxNet Usenet Gateway (21:1/5)

XPost: linux.debian.bugs.dist

On Wed, Aug 30, 2017 at 12:56:30PM +0200, Ondrej Novy wrote:

2017-08-26 21:49 GMT+02:00 Sean Whitton <spwhitton@spwhitton.name>:

Actually, this depends on which autodep8 module you want to use. I know that my elpa module runs the tests even if the Testsuite: field is
missing, for example.

i think this is true for local build environment only, but not for ci.debian.net. If you want your autodep8 tests to run on Debian CI, you
need to explicitly enable them.

Indeed. Alternatively, you need to reach out to the ci.d.n admins: they maintain a manual list of packages for which to run a specific autodep8
thing (which is what has been done for all the things currently in
autodep8 to have them enabled right away for all packages).

--
regards,
Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18 4D18 4B04 3FCD B944 4540 .''`.
more about me: https://mapreri.org : :' : Launchpad user: https://launchpad.net/~mapreri `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia `-

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEEi3hoeGwz5cZMTQpICBa54Yx2K60FAlmnCZgACgkQCBa54Yx2 K63euhAAgxZx+J7nbSlnENUoeROUc4ZD2iuH1MmeHm25GHVQw/AbJI9nIhNQ+pn6 kaCZisYnB60nhc4nrs1/vxMRlIdjlHcwXgD1x5iPXuSX8AYUF3K6nzzhS1+VJAFO EghVifrFa8BpBkRJe/AW4nJs58gLsDEtEn5HQBnLqbiygL8TtSvNyGoFwtSF6Lfn ZdKBKXAuSXbhJOUslIoWAf5W/8dUA7CHU+46NqtWPSvkNJm2YegMhzBMGtftPBRR myY+cT23ppNw9gZSecaki2j19A3egWGeZ6y4fKeZD/Sx7zCgtUhpZ+rHWmlg7Cid WuGSOlvbr9CcpLpxarkFxJTM5n2fWQ4M0kzZPPbMGyX6rlQ5QxKnpumN02sW3tQN xOp5bP5oPQBjY4no34CPeWxL6vuuDJOQupp4pKnOCtLgIV5ghdUKGruZe6OEGIQm Kqw6OUPmnqV0E0WYr0z30twQ8PIsWHBeP+Cdjgv2QYjjdZX60jm2uXz9g76+txwN 0RdhG6AfCpgn851hq6xlDNYF/OejC8vnRWBzqegqGTec0Fkxtrc