Merge ~hectorcao/ubuntu/+source/fetchmail:lp2139104 into ubuntu/+source/fetchmail:ubuntu/devel

Proposed by Hector CAO
Status: Needs review
Proposed branch: ~hectorcao/ubuntu/+source/fetchmail:lp2139104
Merge into: ubuntu/+source/fetchmail:ubuntu/devel
Diff against target: 46 lines (+11/-4)
2 files modified
debian/changelog (+7/-0)
debian/tests/mock-pop3-server.py (+4/-4)
Reviewer Review Type Date Requested Status
Christian Ehrhardt (community) Needs Fixing
git-ubuntu import Pending
Review via email: mp+499724@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Christian Ehrhardt (paelzer) wrote :

- please refer to https://docs.python.org/3/whatsnew/3.14.html#pep-765-control-flow-in-finally-blocks

- please rebase on the latest version, remember tell your sponsors to use -v$latestversion

- the fix IMHO should be fixing it for real, maybe assign retval=0, =1 in the except and then return after the finally block

- report debian bug OR salsa PR to fix this there, refer to it in your commit

review: Needs Fixing
Revision history for this message
Hector CAO (hectorcao) wrote :

Hello @paelzer,

- I rebased the MR on ubuntu/devel
- I refreshed the fix to not use allow-stderr
- I reported the issue to Debian

Please take a look again

64c4558... by Hector CAO

merge-changelogs

dc3a476... by Hector CAO

reconstruct-changelog

e8602f2... by Hector CAO

changelog 6.6.2-2

Unmerged commits

e8602f2... by Hector CAO

changelog 6.6.2-2

dc3a476... by Hector CAO

reconstruct-changelog

64c4558... by Hector CAO

merge-changelogs

Diff calculation failed

Calculating the branch diff failed. You can manually schedule an update if required.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 900e653..9e66a01 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+fetchmail (6.5.4-0ubuntu4) resolute; urgency=medium
7+
8+ * d/t/mock-pop3-server.py : fix return value in except/finally block
9+ (LP: #2139104)
10+
11+ -- Hector Cao <hector.cao@canonical.com> Fri, 06 Feb 2026 15:26:31 +0100
12+
13 fetchmail (6.5.4-0ubuntu3) resolute; urgency=medium
14
15 * No-change mass rebuild for Ubuntu 26.04 (LP: #2132257)
16diff --git a/debian/tests/mock-pop3-server.py b/debian/tests/mock-pop3-server.py
17index 159069b..b1af47b 100644
18--- a/debian/tests/mock-pop3-server.py
19+++ b/debian/tests/mock-pop3-server.py
20@@ -172,6 +172,7 @@ def serve(messages_filename, host=DEFAULT_HOST, port=DEFAULT_PORT):
21
22 mailbox = Mailbox(messages_filename)
23
24+ ret = 0
25 try:
26 dbg("* mock-pop3-server ready on {}:{}".format(host, port))
27 while True:
28@@ -190,15 +191,14 @@ def serve(messages_filename, host=DEFAULT_HOST, port=DEFAULT_PORT):
29 server.close()
30 except KeyboardInterrupt:
31 dbg("* mock-pop3-server interrupted")
32- return 1
33+ ret = 1
34 except SystemExit:
35 dbg("* mock-pop3-server exiting")
36- return 1
37+ ret = 1
38 finally:
39 sock.shutdown(socket.SHUT_RDWR)
40 sock.close()
41- return 0
42-
43+ return ret
44
45 if __name__ == "__main__":
46 fd, message_filename = mkstemp(prefix='message-', suffix='.txt')

Subscribers

People subscribed via source and target branches