Merge ~zhsj/ubuntu/+source/dhcpcd5:ubuntu/mantic into ubuntu/+source/dhcpcd5:ubuntu/devel

Proposed by Shengjing Zhu
Status: Merged
Merge reported by: Robie Basak
Merged at revision: c48de24c136a369b86b187b5bd0cc3976ddb2d90
Proposed branch: ~zhsj/ubuntu/+source/dhcpcd5:ubuntu/mantic
Merge into: ubuntu/+source/dhcpcd5:ubuntu/devel
Diff against target: 283 lines (+120/-25)
10 files modified
debian/changelog (+17/-0)
debian/control (+3/-4)
debian/patches/0009_allow_newfstatat_syscall.patch (+25/-0)
debian/patches/0010_miss_seccomp_on_ppc64el.patch (+29/-0)
debian/patches/series (+2/-0)
debian/tests/common-ntp-servers-from-dhcp (+5/-0)
debian/tests/control (+0/-8)
debian/upstream/signing-key.asc (+37/-0)
debian/watch (+2/-3)
dev/null (+0/-10)
Reviewer Review Type Date Requested Status
Ubuntu Sponsors Pending
git-ubuntu import Pending
Review via email: mp+443474@code.launchpad.net

This proposal supersedes a proposal from 2023-05-24.

To post a comment you must log in.
Revision history for this message
Shengjing Zhu (zhsj) wrote :
Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

This has landed and can be closed now.

Revision history for this message
Shengjing Zhu (zhsj) wrote :

@athos-ribeiro do you know how to close the merge proposal? i only see the delete option.

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 9b47e66..0653f8c 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,20 @@
6+dhcpcd5 (9.4.1-21ubuntu1) mantic; urgency=medium
7+
8+ [ Martin-Éric Racine ]
9+ * Migrate debian/watch to Github in cadence with upstream.
10+
11+ [ Shengjing Zhu ]
12+ * Fix autopkgtests (Closes: #1036098)
13+ + Use bind-interfaces in dnsmasq conf
14+ + Wait until ntp server is reloaded.
15+ * Backport seccomp patch to allow newfstatat syscall (Closes: #1034465)
16+ * Backport seccomp patch for missing SECCOMP_AUDIT_ARCH on ppc64el (Closes: #1036662)
17+ * Drop incompatible ntpd integration. (Closes: #1036092)
18+ No longer working after ntpd was replaced by ntpsec.
19+ * Remove Conflicts/Replaces dhcp-client (Closes: #1036085)
20+
21+ -- Shengjing Zhu <shengjing.zhu@canonical.com> Wed, 24 May 2023 16:05:40 +0800
22+
23 dhcpcd5 (9.4.1-21) unstable; urgency=medium
24
25 * Document migration to Predictable Network Interface Names in NEWS.Debian.
26diff --git a/debian/control b/debian/control
27index 0d417e6..999f016 100644
28--- a/debian/control
29+++ b/debian/control
30@@ -2,7 +2,8 @@ Source: dhcpcd5
31 Section: net
32 Priority: optional
33 Homepage: https://roy.marples.name/projects/dhcpcd
34-Maintainer: Martin-Éric Racine <martin-eric.racine@iki.fi>
35+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
36+XSBC-Original-Maintainer: Martin-Éric Racine <martin-eric.racine@iki.fi>
37 Build-Depends: debhelper-compat (= 13),
38 libudev-dev [linux-any],
39 pkg-config
40@@ -13,10 +14,8 @@ Vcs-Git: https://salsa.debian.org/debian/dhcpcd5.git
41
42 Package: dhcpcd-base
43 Architecture: any
44-Conflicts: dhcp-client
45 Provides: dhcp-client
46-Replaces: dhcp-client,
47- dhcpcd5 (<< 9.4.1-2)
48+Replaces: dhcpcd5 (<< 9.4.1-2)
49 Breaks: dhcpcd5 (<< 9.4.1-2)
50 Depends: adduser,
51 ${misc:Depends},
52diff --git a/debian/hooks/66-ntp.conf b/debian/hooks/66-ntp.conf
53deleted file mode 100644
54index 4fa88b0..0000000
55--- a/debian/hooks/66-ntp.conf
56+++ /dev/null
57@@ -1,34 +0,0 @@
58-# vi: ft=sh
59-
60-SERVERFILE_IPV4="/run/ntp.conf.dhcp"
61-SERVERFILE_IPV6="$SERVERFILE_IPV4"
62-
63-add_servers() {
64- # exit early if there's nothing to do
65- if [ "$new_ntp_servers" = "$old_ntp_servers" ]; then
66- return
67- fi
68-
69- insert_servers iburst
70- reload_config
71-}
72-
73-remove_servers() {
74- SERVERFILE=$SERVERFILE_IPV4
75-
76- # exit early if there's nothing to do
77- if [ ! -f $SERVERFILE ]; then
78- return
79- fi
80-
81- withdraw_servers
82- reload_config
83-}
84-
85-reload_config() {
86- invoke-rc.d ntp try-restart || :
87-}
88-
89-if [ -e /etc/ntp.conf ]; then
90- handle_reason
91-fi
92diff --git a/debian/patches/0009_allow_newfstatat_syscall.patch b/debian/patches/0009_allow_newfstatat_syscall.patch
93new file mode 100644
94index 0000000..93f6d1b
95--- /dev/null
96+++ b/debian/patches/0009_allow_newfstatat_syscall.patch
97@@ -0,0 +1,25 @@
98+From 38befd4e867583002b96ec39df733585d74c4ff5 Mon Sep 17 00:00:00 2001
99+From: Roy Marples <roy@marples.name>
100+Date: Fri, 26 Aug 2022 09:24:50 +0100
101+Subject: [PATCH] privsep: Allow newfstatat syscall as well
102+
103+Allows newer glibc variants to work apparently.
104+As reported in #84 and #89.
105+---
106+ src/privsep-linux.c | 3 +++
107+ 1 file changed, 3 insertions(+)
108+
109+diff --git a/src/privsep-linux.c b/src/privsep-linux.c
110+index 4d5c195d..66aeb490 100644
111+--- a/src/privsep-linux.c
112++++ b/src/privsep-linux.c
113+@@ -353,6 +353,9 @@ static struct sock_filter ps_seccomp_filter[] = {
114+ #ifdef __NR_nanosleep
115+ SECCOMP_ALLOW(__NR_nanosleep), /* XXX should use ppoll instead */
116+ #endif
117++#ifdef __NR_newfstatat
118++ SECCOMP_ALLOW(__NR_newfstatat),
119++#endif
120+ #ifdef __NR_ppoll
121+ SECCOMP_ALLOW(__NR_ppoll),
122+ #endif
123diff --git a/debian/patches/0010_miss_seccomp_on_ppc64el.patch b/debian/patches/0010_miss_seccomp_on_ppc64el.patch
124new file mode 100644
125index 0000000..969a191
126--- /dev/null
127+++ b/debian/patches/0010_miss_seccomp_on_ppc64el.patch
128@@ -0,0 +1,29 @@
129+From 7a2d9767585ed2c407d4985bd2d81552034fb90a Mon Sep 17 00:00:00 2001
130+From: CHEN Xiangyu <xiangyu.chen@aol.com>
131+Date: Thu, 9 Feb 2023 18:41:52 +0800
132+Subject: [PATCH] privsep-linux: fix SECCOMP_AUDIT_ARCH missing ppc64le (#181)
133+
134+when dhcpcd running on ppc64le platform, it would be killed by SIGSYS.
135+
136+Signed-off-by: Xiangyu Chen <xiangyu.chen@windriver.com>
137+---
138+ src/privsep-linux.c | 6 +++++-
139+ 1 file changed, 5 insertions(+), 1 deletion(-)
140+
141+diff --git a/src/privsep-linux.c b/src/privsep-linux.c
142+index 7372d26b..6a301950 100644
143+--- a/src/privsep-linux.c
144++++ b/src/privsep-linux.c
145+@@ -232,7 +232,11 @@ ps_root_sendnetlink(struct dhcpcd_ctx *ctx, int protocol, struct msghdr *msg)
146+ #elif defined(__or1k__)
147+ # define SECCOMP_AUDIT_ARCH AUDIT_ARCH_OPENRISC
148+ #elif defined(__powerpc64__)
149+-# define SECCOMP_AUDIT_ARCH AUDIT_ARCH_PPC64
150++# if (BYTE_ORDER == LITTLE_ENDIAN)
151++# define SECCOMP_AUDIT_ARCH AUDIT_ARCH_PPC64LE
152++# else
153++# define SECCOMP_AUDIT_ARCH AUDIT_ARCH_PPC64
154++# endif
155+ #elif defined(__powerpc__)
156+ # define SECCOMP_AUDIT_ARCH AUDIT_ARCH_PPC
157+ #elif defined(__riscv)
158diff --git a/debian/patches/series b/debian/patches/series
159index 476dd9e..0e7ab06 100644
160--- a/debian/patches/series
161+++ b/debian/patches/series
162@@ -6,3 +6,5 @@
163 0006_skip_hooks_with_tilde.patch
164 0007_set_buffer_lenght_to_0.patch
165 0008_fix_incorrect_assert.patch
166+0009_allow_newfstatat_syscall.patch
167+0010_miss_seccomp_on_ppc64el.patch
168diff --git a/debian/tests/common-ntp-servers-from-dhcp b/debian/tests/common-ntp-servers-from-dhcp
169index 98848ef..bbb5dbc 100644
170--- a/debian/tests/common-ntp-servers-from-dhcp
171+++ b/debian/tests/common-ntp-servers-from-dhcp
172@@ -24,6 +24,7 @@ dnsmasq_config() {
173 echo "Preparing dnsmasq configuration..."
174 cat <<EOF > /etc/dnsmasq.conf
175 interface=veth0
176+bind-interfaces
177 dhcp-range=192.168.1.100,192.168.1.150,12h
178 dhcp-range=fdae:9322:f1cc::9,fdae:9322:f1cc::99,64,12h
179 dhcp-option=option:ntp-server,$ntp_server_ipv4
180@@ -117,9 +118,13 @@ run_test() {
181 sleep 5
182
183 configure_interface
184+ # wait until ntp server is reloaded with new configuration
185+ sleep 5
186 check_ntp_server_obtained
187
188 deconfigure_interface
189+ # wait until ntp server is reloaded with new configuration
190+ sleep 5
191 check_ntp_server_gone
192
193 check_conf_hash
194diff --git a/debian/tests/control b/debian/tests/control
195index bda624b..0c34d92 100644
196--- a/debian/tests/control
197+++ b/debian/tests/control
198@@ -13,14 +13,6 @@ Depends: dnsmasq,
199 @
200 Restrictions: isolation-machine, needs-root, breaks-testbed
201
202-Tests: ntpd-ntp-servers-from-dhcp
203-Depends: dnsmasq,
204- ethtool,
205- iproute2,
206- ntp,
207- @
208-Restrictions: isolation-machine, needs-root, breaks-testbed
209-
210 Tests: openntpd-ntp-servers-from-dhcp
211 Depends: dnsmasq,
212 ethtool,
213diff --git a/debian/tests/ntpd-ntp-servers-from-dhcp b/debian/tests/ntpd-ntp-servers-from-dhcp
214deleted file mode 100644
215index b1663a2..0000000
216--- a/debian/tests/ntpd-ntp-servers-from-dhcp
217+++ /dev/null
218@@ -1,10 +0,0 @@
219-#!/bin/sh
220-set -e
221-
222-added_servers_ipv4="/run/ntp.conf.dhcp"
223-added_servers_ipv6="$added_servers_ipv4"
224-server_list_cmd="ntpq -nwc lpeers"
225-
226-. debian/tests/common-ntp-servers-from-dhcp
227-
228-run_test
229diff --git a/debian/upstream/signing-key.asc b/debian/upstream/signing-key.asc
230new file mode 100644
231index 0000000..44d4252
232--- /dev/null
233+++ b/debian/upstream/signing-key.asc
234@@ -0,0 +1,37 @@
235+-----BEGIN PGP PUBLIC KEY BLOCK-----
236+
237+mQENBEkrCoIBCACyCfZ4qEaFPi33OT1EYDziIOb5KnvW3E9zP7O8oHGmhzi+v11X
238+AsShyiQCFaKD3QgQ/Ra5TV6mjfjV2fPjSXp6wVy9ojvmIFpvIgu1AQJIqkYl0jpV
239+JFNT1n9RcI3RkZdYYbxkncmp3M8NMhWo0Txpuw6dx2c7gLeQp0PQiwJUPiLS1rUw
240+KQLFGnPfvMu4koCoxeyYydijR8XuMmEk3KkYxqDKw81IljwhvOiJa9AVik01GMnE
241+C+9xQcrdaa5+0yqPt3n/9WrGk+yC2oLiSjGgJHNB7niU716gf5HVUYrqEdiuVw4+
242+scSjEdCOmRoWcq6D84EIkoXXX0gurSLcHh7xABEBAAG0HlJveSBNYXJwbGVzIDxy
243+b3lAbWFycGxlcy5uYW1lPokBUQQTAQgAOxYhBKeF7SdVlV2ek+pZ9ll/l+qa1FVJ
244+BQJkRp31AhsDBQsJCAcCAiICBhUKCQgLAgQWAgMBAh4HAheAAAoJEFl/l+qa1FVJ
245+0u8H/2DUrOEsJYpCSRJTvVX+swpFlsnfq55z0/27QwbPmtHak61WTzfr8psBki4g
246+xKjpyVTpmvXhJVMzCVoMmUCRbYui2BCJ562yM7puxT3e6H8cCdjoezymkhPp9g2A
247+Xm2QuHvhVvSi1spilgDm5S3o7E6qeoz2NDkItnXuwYvc3uKAUIg9hfSRjXDopV+H
248+bo0G4pDI010hr3eRRmcZgPcvL4mzxeLasV9ZVFLCQ78C9h2+SzmkcOXE/SAFNT3J
249+sg7Z5ferEninNiFIDVSDFg9eunA/uhY45waggtjUSjsgCS4PcJnXsDz/dUCyUvNH
250+PHi48tImjP9oFXCQcMWYUZyIk9O0JVJveSBNYXJwbGVzIChOZXRCU0QpIDxyb3lA
251+TmV0QlNELm9yZz6JATgEEwECACIFAkuAGkkCGwMGCwkIBwMCBhUIAgkKCwQWAgMB
252+Ah4BAheAAAoJEFl/l+qa1FVJHnMH/2zHxpWdNcfMCkOvUo9ZVGjS2K57fKCIb2EH
253+whyQPqFcY4OtFYNCJC75s4imY5ym24PCeFxxXG8GLMqgVtK5Bbrnx0UMI68x6UzP
254++2ap+SBDFE7FaudCAbdX48ABkrHdLA7VzB+Bk/AbpqEIKz86o8NFVFdAxMLdwrLd
255+TjGG3figbGEOqBIlFog5c+9ijn8uvhRYW3dIb6b4hQ6laOfmZwKAThVhx3Dk9RBH
256+9DkHe7jD8+PJ05zdCgAa0cx7w7C/eMcpNNakPLASpgzCEmVCobgpuoEcZXmEv8hJ
257+ZYQ3bzzKEIHhzQNd5Ack7r7r5MrFspD6y5Q419DybL3NiU9gcdi5AQ0ESSsK+gEI
258+AMOYyNnU8QvuP68M3ODgzy3NgqwUViZCTXKiaJhpCgQWTvVYHFMN/DSCBKIWjpJn
259+B11rVlzeRRO6sjOBl9wiIOCcvv20Soc2sRdysTH7K4stgLFIFXIqT9QxGeaJwe+l
260+oSwjtbrCZW7G90H4DMfhM/VbEZjv817434XNIAd3fn2sykOnwm9B8vMdy8jk1/vj
261++/lmU/YYFkHLKausbkr+FHRaBLJyBBin/Po5D953BaSe/ATN2IGHSfPo0P4hqJ/G
262+8Tg8QTeBysDk1UpxePY4Sas/rlkmyi/4ssn65BwsUe9W4BxOjX4XO5Pkg368CVo8
263+/rk13uGqzqFFFkShj1BaRtEAEQEAAYkBNgQYAQgAIAIbDBYhBKeF7SdVlV2ek+pZ
264+9ll/l+qa1FVJBQJkRpwjAAoJEFl/l+qa1FVJ5o4H/0M8J+BaAIRcmq6IPnRknvU9
265+FiIXCXliuPOZm0a6Vc+U+a5rr8ErSJXzAK/IqtOzHn7KUrlHT/V6w5VRSmIFjR5e
266+JVXhdcYB9PywxBjr6ffD7v2X4EGhQCTNzrbwmsXP9PF5ES4s++hZa3XO0fbfxnwO
267+cu+4jJAENupk9lxduXlU2FIoKKctGpzW8RRmM8b5Afi0XLKfR8w+eL7HXtsIfYqo
268+QMSOuCupmNd+iWS+RZXm0yl30qnLzKs+ZMUvWgkkTZEUFR29yYnPLBQMcq0c7QVo
269+iCC8dtq0EwWROJeo+dVtczXZORQ3VPM880fBqy2Ppu61lL8OMUlLxmrxqPFakcY=
270+=wWGO
271+-----END PGP PUBLIC KEY BLOCK-----
272diff --git a/debian/watch b/debian/watch
273index 4b892e9..9ff12d3 100644
274--- a/debian/watch
275+++ b/debian/watch
276@@ -1,5 +1,4 @@
277 version=4
278-https://roy.marples.name/downloads/dhcpcd/dhcpcd-([0-9\.]*)\.(?:zip|tgz|tbz|txz|(?:tar\.(?:gz|bz2|xz))) \
279+opts="mode=git,pgpmode=gittag" \
280+https://github.com/NetworkConfiguration/dhcpcd.git refs/tags/v([\d\.]+) \
281 debian uupdate --no-symlink
282-#opts="mode=git,pgpmode=gittag" \
283-#https://github.com/NetworkConfiguration/dhcpcd.git refs/tags/dhcpcd-([\d\.]+) \

Subscribers

People subscribed via source and target branches