[Python-apps-team] Bug#972793: hg-git: autopkgtest needs update for new version of dulwich: lots of deprecation *warnings*

[Paul Gevers]

Source: hg-git
Version: 0.9.0-1
Severity: serious
X-Debbugs-CC: debian-ci at lists.debian.org, dulwich at packages.debian.org
Tags: sid bullseye
User: debian-ci at lists.debian.org
Usertags: needs-update
Control: affects -1 src:dulwich

Dear maintainer(s),

With a recent upload of dulwich the autopkgtest of hg-git fails in
testing when that autopkgtest is run with the binary packages of dulwich
from unstable. It passes when run with only packages from testing. In
tabular form:

                       pass            fail
dulwich                from testing    0.20.6-1.1
hg-git                 from testing    0.9.0-1
all others             from testing    from testing

I copied some of the output at the bottom of this report.

Currently this regression is blocking the migration of dulwich to
testing [1]. Of course, dulwich shouldn't just break your autopkgtest
(or even worse, your package), but it seems to me that the change in
dulwich was intended and your package needs to update to the new
situation. Also, failing on deprecation warnings should really be
avoided in autopkgtesting (that's more something for upstreams test
framework), please disable deprecation warnings if possible.

More information about this bug and the reason for filing it can be found on
https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation

Paul

[1] https://qa.debian.org/excuses.php?package=dulwich

https://ci.debian.net/data/autopkgtest/testing/amd64/h/hg-git/7682446/log.gz

--- /tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/tests/test-renames.t
+++
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/tests/test-renames.t.err
@@ -371,6 +371,16 @@
   adding objects
   added 2 commits with 2 trees and 3 blobs
   updating reference refs/heads/master
+
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/hggit/git_handler.py:411:
DeprecationWarning: Use SendPackResult.refs instead.
+    if remote_name and new_refs:
+  /usr/lib/python3/dist-packages/mercurial/pycompat.py:362:
DeprecationWarning: Use SendPackResult.refs instead.
+    iteritems = lambda x: x.items()
+
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/hggit/git_handler.py:431:
DeprecationWarning: Use SendPackResult.refs instead.
+    if old_refs == new_refs:
+
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/hggit/git_handler.py:434:
DeprecationWarning: Use SendPackResult.refs instead.
+    elif len(new_refs) > len(old_refs):
+
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/hggit/git_handler.py:436:
DeprecationWarning: Use SendPackResult.refs instead.
+    elif len(old_refs) > len(new_refs):

   $ cd ../gitrepo
   $ git log master --pretty=oneline

ERROR: test-renames.t output changed
!.
---
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/tests/test-file-removal.t
+++
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/tests/test-file-removal.t.err
@@ -168,6 +168,12 @@
   searching for changes
   adding objects
   added 9 commits with 8 trees and 5 blobs
+
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/hggit/git_handler.py:431:
DeprecationWarning: Use SendPackResult.refs instead.
+    if old_refs == new_refs:
+
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/hggit/git_handler.py:434:
DeprecationWarning: Use SendPackResult.refs instead.
+    elif len(new_refs) > len(old_refs):
+
/tmp/autopkgtest-lxc.1m_ql5en/downtmp/build.IYS/src/hggit/git_handler.py:435:
DeprecationWarning: Use SendPackResult.refs instead.
+    ret = 1 + (len(new_refs) - len(old_refs))

   $ cd ..
   $ git --git-dir=gitrepo2 log --pretty=medium

ERROR: test-file-removal.t output changed
!.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://alioth-lists.debian.net/pipermail/python-apps-team/attachments/20201023/0b0dea75/attachment.sig>