Hi Matthias,

>>> Or you remove the autodep8 test from debian/control.
>> Indeed that is what I changed in 1.1.11-2 which should be in both sid
>> and bullseye by now -- I changed the autopkgtest definition and added
>> custom test scripts reflecting the situation.
>> 
>> All tests are green so far now [3]. Where did you get your log snippet from?
> 
> http://autopkgtest.ubuntu.com/packages/p/python-virustotal-api/focal/amd64

But version 1.1.11-2 (without the ubuntu1 patch) also passes its autopkgtest in 
the list on that page (row 3)?

> that's what I changed, it's still in the control file.
> 
> http://launchpadlibrarian.net/462719068/python-virustotal-api_1.1.11-2_1.1.11-2ubuntu1.diff.gz

This would also only cause the CI to run the regular autopkgtest in 
debian/tests, instead of the failing Python-specific ones that assume a 
particular mapping from import name to package name. My change in 1.1.11-2 does 
the same:

diff --git a/debian/control b/debian/control
index 83a6cfe..6032eda 100644
--- a/debian/control
+++ b/debian/control
@@ -9,7 +9,7 @@ Build-Depends: debhelper-compat (= 12),
                python3-setuptools,
                python3-requests
 Standards-Version: 4.5.0
-Testsuite: autopkgtest-pkg-python
+Testsuite: autopkgtest
 Vcs-Git: https://salsa.debian.org/debian/python-virustotal-api.git
 Vcs-Browser: https://salsa.debian.org/debian/python-virustotal-api
 Homepage: https://github.com/blacktop/virustotal-api

Plus adding a separate, explicit test setup in d/tests, of course.

I do not see where removing the Testsuite line would change anything compared 
to 1.1.11-2, given that the autopkgtest for that’s version passes even on 
Ubuntu’s CI. Am I missing something?

Cheers
Sascha

Attachment: signature.asc
Description: Message signed with OpenPGP

Reply via email to