Merge ~pushkarnk/ubuntu/+source/dnspython:merge-lp2043954-noble into ubuntu/+source/dnspython:debian/sid

Proposed by Pushkar Kulkarni
Status: Needs review
Proposed branch: ~pushkarnk/ubuntu/+source/dnspython:merge-lp2043954-noble
Merge into: ubuntu/+source/dnspython:debian/sid
Diff against target: 92 lines (+31/-7)
3 files modified
debian/changelog (+28/-0)
debian/control (+3/-5)
debian/tests/control (+0/-2)
Reviewer Review Type Date Requested Status
Gianfranco Costamagna (community) Approve
Utkarsh Gupta (community) Needs Information
git-ubuntu import Pending
Review via email: mp+455854@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Pushkar Kulkarni (pushkarnk) wrote :
Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

Hey there!

+ - Remove python3-sniffio from Suggests

I don't see this in the diff? And the reasoning for that? I do see https://git.launchpad.net/~pushkarnk/ubuntu/+source/dnspython/commit/?id=a92ad5ce3afe9654b0bace5f177686fa64ef631e - which tells me what's happening but it's still unclear why it was done? :)

review: Needs Information
Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

Once that's clear and you're ready for review (by marking the MP as "Needs Review"), I'll take that up and sponsor your upload. ;)

Revision history for this message
Pushkar Kulkarni (pushkarnk) wrote :

Tested a fresh install on noble. Also tested an upgrade. Both work well.

Revision history for this message
Pushkar Kulkarni (pushkarnk) wrote :

Ran autopkgtests against the PPA using:

autopkgtest --apt-upgrade --shell-fail --output-dir dnspython-ppa --setup-commands="sudo add-apt-repository -y -u -s ppa:pushkarnk/merges-ppa" dnspython -- lxd ubuntu-daily:noble

The tests passed.

Revision history for this message
Pushkar Kulkarni (pushkarnk) wrote :

The only change here is a new upstream version. So, I skipped running `check-mir` which was previously recommended.

Revision history for this message
Pushkar Kulkarni (pushkarnk) wrote :

> Hey there!
>
>
> + - Remove python3-sniffio from Suggests
>
> I don't see this in the diff? And the reasoning for that? I do see https://git
> .launchpad.net/~pushkarnk/ubuntu/+source/dnspython/commit/?id=a92ad5ce3afe9654
> b0bace5f177686fa64ef631e - which tells me what's happening but it's still
> unclear why it was done? :)

Hey, thanks for looking into this. I guess this is the chronology:

 - [1] moved some of the dependencies, including python3-sniffio to Suggests from Recommended (to avoid MIRs)
 - [2] is a debian commit which removed python3-sniffio as a dependency, because it also came in as a dependency of python3-trio
 - Now, every time we apply [1] over debian/sid, we need to also apply [3] to remove the redundant python3-sniffio dependency

Is there a better way to organize these commits?

[1] https://git.launchpad.net/ubuntu/+source/dnspython/commit/?id=55aaf9a022da3329f91c2f231735886af8fb04d2
[2] https://git.launchpad.net/ubuntu/+source/dnspython/commit/?id=ee715c17c7c379444767c369c7161139b53165d1
[3] https://git.launchpad.net/ubuntu/+source/dnspython/commit/?id=b6626db4a717dc211de816dd9b2b6e8673efcfcb

Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :

Sponsored!

review: Approve

Unmerged commits

ee6da07... by Pushkar Kulkarni

Update changelog

033f891... by Pushkar Kulkarni

update-maintainer

e633b75... by Pushkar Kulkarni

reconstruct-changelog

e743378... by Pushkar Kulkarni

merge-changelogs

ea53bd3... by Pushkar Kulkarni

Drop python-httpcore from Recommends to Suggests

Like the python-httpx dependency, which brings in a lot of transitive
dependencies not in the main archive, was earlier moved down from
Recommends to Suggests, the new httpcore dependency can also be
moved down to Suggests. The httpcore dependency was brought in
to work around an httpx limitation in the context of DNS over HTTPS.
The httpx dependency is also brought in for the same feature.

a92ad5c... by Pushkar Kulkarni

Remove python3-sniffio from Suggests

9c7acda... by Lukas Märdian

Drop python-h2 & httpx (DNS Over HTTPS, DOH) to Suggests (LP: 2004272)

This is to avoid a big tree of additional transitive dependencies (for now).
Foundations might agree to maintain all of those in the future and drop this
delta.

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 8075abd..b654f15 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,9 +1,31 @@
6+dnspython (2.4.2-1ubuntu1) noble; urgency=medium
7+
8+ * Merge from Debian unstable (LP: #2043954). Remaining changes:
9+ - Drop python-h2 & httpx (DNS Over HTTPS, DOH) to Suggests
10+ - Remove python3-sniffio from Suggests
11+ - Drop python-httpcore from Recommends to Suggests
12+
13+ -- Pushkar Kulkarni <pushkar.kulkarni@canonical.com> Mon, 20 Nov 2023 10:48:32 +0530
14+
15 dnspython (2.4.2-1) unstable; urgency=medium
16
17 * New upstream release
18
19 -- Scott Kitterman <scott@kitterman.com> Fri, 11 Aug 2023 21:35:53 -0400
20
21+dnspython (2.4.1-1ubuntu2) mantic; urgency=medium
22+
23+ * d/control: Move the httpcore dependency to Suggests (LP: #2030475)
24+
25+ -- Pushkar Kulkarni <pushkar.kulkarni@canonical.com> Mon, 07 Aug 2023 17:40:17 +0530
26+
27+dnspython (2.4.1-1ubuntu1) mantic; urgency=medium
28+
29+ * Merge from Debian unstable (LP: #2029461). Remaining changes:
30+ - Drop python-h2 & httpx (DNS Over HTTPS, DOH) to Suggests
31+
32+ -- Pushkar Kulkarni <pushkar.kulkarni@ubuntu.com> Thu, 03 Aug 2023 19:26:28 +0530
33+
34 dnspython (2.4.1-1) unstable; urgency=medium
35
36 * New upstream release
37@@ -36,6 +58,12 @@ dnspython (2.4.0~rc1-1) experimental; urgency=medium
38
39 -- Scott Kitterman <scott@kitterman.com> Thu, 06 Jul 2023 22:09:07 -0400
40
41+dnspython (2.3.0-1ubuntu1) lunar; urgency=medium
42+
43+ * Drop python-h2 & httpx (DNS Over HTTPS, DOH) to Suggests (LP: 2004272)
44+
45+ -- Lukas Märdian <slyon@ubuntu.com> Tue, 31 Jan 2023 14:37:06 +0100
46+
47 dnspython (2.3.0-1) unstable; urgency=medium
48
49 * Upload to unstable
50diff --git a/debian/control b/debian/control
51index 5c739ac..edf8111 100644
52--- a/debian/control
53+++ b/debian/control
54@@ -1,7 +1,8 @@
55 Source: dnspython
56 Section: python
57 Priority: optional
58-Maintainer: Debian Python Team <team+python@tracker.debian.org>
59+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
60+XSBC-Original-Maintainer: Debian Python Team <team+python@tracker.debian.org>
61 Uploaders: Robert S. Edmonds <edmonds@debian.org>,
62 Scott Kitterman <scott@kitterman.com>,
63 Matthew Grant <matthewgrant5@gmail.com>
64@@ -30,13 +31,10 @@ Architecture: all
65 Depends: netbase, ${misc:Depends}, ${python3:Depends}
66 Recommends: ca-certificates,
67 python3-cryptography,
68- python3-h2,
69- python3-httpcore (>=0.17.3),
70- python3-httpx,
71 python3-idna,
72 python3-requests,
73 python3-requests-toolbelt
74-Suggests: python3-trio, python3-aioquic
75+Suggests: python3-trio, python3-aioquic, python3-h2, python3-httpx, python3-httpcore (>=0.17.3)
76 Description: DNS toolkit for Python 3
77 dnspython is a DNS toolkit for Python. It supports almost all record types. It
78 can be used for queries, zone transfers, and dynamic updates. It supports TSIG
79diff --git a/debian/tests/control b/debian/tests/control
80index b102c2d..497c969 100644
81--- a/debian/tests/control
82+++ b/debian/tests/control
83@@ -3,9 +3,7 @@ Depends: ca-certificates,
84 python3-aioquic,
85 python3-all,
86 python3-cryptography,
87- python3-h2,
88 python3-httpcore,
89- python3-httpx,
90 python3-idna,
91 python3-pytest,
92 python3-requests,

Subscribers

People subscribed via source and target branches