Merge ~mfo/ubuntu/+source/crash:merge-lp2047861-noble into ubuntu/+source/crash:debian/sid

Proposed by Mauricio Faria de Oliveira
Status: Needs review
Proposed branch: ~mfo/ubuntu/+source/crash:merge-lp2047861-noble
Merge into: ubuntu/+source/crash:debian/sid
Diff against target: 253 lines (+166/-4)
4 files modified
debian/changelog (+145/-0)
debian/control (+2/-1)
debian/tests/control (+2/-2)
debian/tests/live (+17/-1)
Reviewer Review Type Date Requested Status
Sergio Durigan Junior (community) Approve
Review via email: mp+458192@code.launchpad.net

Description of the change

Build/tests from PPA in bug 2047861.

To post a comment you must log in.
Revision history for this message
Mauricio Faria de Oliveira (mfo) wrote :

Autopkgtests (live system form) and manual tests (dump file form) PASS in amd64, arm64, ppc64el, and s390x (bug 2047861 comment #8).

Packages in ppa:mfo/lp2047861-noble-crash

Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thanks, Mauricio.

The merge LGTM. Autopkgtest results are OK, and your manual test results make me very confident that no apparent regression is being introduced. Thanks for those!

I'm leaving a small comment regarding the changelog entry, but otherwise this is good to be uploaded. Let me know when you address the comment and I can upload right away.

Thanks again.

review: Needs Information
Revision history for this message
Mauricio Faria de Oliveira (mfo) wrote :

Thanks for the review, Sergio!

The changelog entry now has 'included upstream in 8.0.4' for the dropped patches.

Thanks for catching and explaining that; I incorrectly assumed it could be taken as a default reason for dropping patches on merges, and will make sure to always address that in the future.

I amended the changelog commit and force pushed.

Thanks again!

Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thanks, Mauricio. That's perfect now!

Uploaded:

$ dput crash_8.0.4-1ubuntu1_source.changes
Trying to upload package to ubuntu
Checking signature on .changes
gpg: /home/sergio/work/crash/crash_8.0.4-1ubuntu1_source.changes: Valid signature from 106DA1C8C3CBBF14
Checking signature on .dsc
gpg: /home/sergio/work/crash/crash_8.0.4-1ubuntu1.dsc: Valid signature from 106DA1C8C3CBBF14
Package includes an .orig.tar.gz file although the debian revision suggests
that it might not be required. Multiple uploads of the .orig.tar.gz may be
rejected by the upload queue management software.
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading crash_8.0.4-1ubuntu1.dsc: done.
  Uploading crash_8.0.4.orig.tar.gz: done.
  Uploading crash_8.0.4-1ubuntu1.debian.tar.xz: done.
  Uploading crash_8.0.4-1ubuntu1_source.buildinfo: done.
  Uploading crash_8.0.4-1ubuntu1_source.changes: done.
Successfully uploaded packages.

review: Approve

Unmerged commits

e65e59e... by Mauricio Faria de Oliveira

changelog

4b126a0... by Mauricio Faria de Oliveira

update-maintainer

e66561b... by Mauricio Faria de Oliveira

reconstruct-changelog

2c01457... by Mauricio Faria de Oliveira

merge-changelogs

3e12e6a... by Mauricio Faria de Oliveira

d/t/control: Run autopkg test with allow-stderr.

[8.0.0-1ubuntu1]

f1c6eb3... by Mauricio Faria de Oliveira

d/t/control: Add linux-libc-dev dependency for the autopkg test. This package gets usually broken with kernel upgrades, so let it already show in the autopkg test.

[7.2.9-2ubuntu3]

97d45ae... by Mauricio Faria de Oliveira

d/t/live: Fix test, as if will return 0 when no cases were true.

[7.2.8-1ubuntu1]

f5cd9e8... by Mauricio Faria de Oliveira

d/t/live: Fix logic issue in "live" autopkgtest introduced in the last upload.

[7.2.6-1ubuntu2]

389dbd7... by Mauricio Faria de Oliveira

d/t/live: If the "live" autopkgtest fails with a recommendation to try /proc/kcore instead of the default, attempt that before failing the test. LP: #1858958.

[7.2.6-1ubuntu1]

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 fa99cd7..591c4c4 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,24 @@
6+crash (8.0.4-1ubuntu1) noble; urgency=medium
7+
8+ * Merge with Debian unstable. Remaining changes: (LP: #2047861)
9+ - d/t/live: If the "live" autopkgtest fails with a recommendation to
10+ try /proc/kcore instead of the default, attempt that before
11+ failing the test. LP 1858958.
12+ - d/t/live: Fix logic issue in "live" autopkgtest introduced in the
13+ last upload.
14+ - d/t/live: Fix test, as if will return 0 when no cases were true.
15+ - d/t/control: Add linux-libc-dev dependency for the autopkg test.
16+ This package gets usually broken with kernel upgrades, so let it
17+ already show in the autopkg test.
18+ - d/t/control: Run autopkg test with allow-stderr.
19+ * Drop changes:
20+ - d/p/0002-Fix-compilation-error-due-to-new-strlcpy-function.patch
21+ [ included upstream in 8.0.4 ]
22+ - d/p/0003-Support-module-memory-layout-change-on-Linux-6.4.patch
23+ [ included upstream in 8.0.4 ]
24+
25+ -- Mauricio Faria de Oliveira <mfo@canonical.com> Tue, 02 Jan 2024 19:09:28 -0300
26+
27 crash (8.0.4-1) unstable; urgency=medium
28
29 * New upstream (Closes: #1057009)
30@@ -73,6 +94,31 @@ crash (8.0.4-1) unstable; urgency=medium
31
32 -- Troy Heber <troyh@debian.org> Mon, 27 Nov 2023 14:43:03 -0700
33
34+crash (8.0.3+ds1-3ubuntu1) noble; urgency=medium
35+
36+ * Merge with Debian unstable. Remaining changes: (LP: #2042357)
37+ - d/t/live: If the "live" autopkgtest fails with a recommendation to
38+ try /proc/kcore instead of the default, attempt that before
39+ failing the test. LP 1858958.
40+ - d/t/live: Fix logic issue in "live" autopkgtest introduced in the
41+ last upload.
42+ - d/t/live: Fix test, as if will return 0 when no cases were true.
43+ - d/t/control: Add linux-libc-dev dependency for the autopkg test.
44+ This package gets usually broken with kernel upgrades, so let it
45+ already show in the autopkg test.
46+ - d/t/control: Run autopkg test with allow-stderr.
47+ * Drop changes:
48+ - d/rules: Build without lto. Fails to build gdb on ppc64el. That
49+ should be fixed, once gdb is updated to a more recent version
50+ (e.g. 10.x). [This line is already commented out since Mantic.]
51+ * New changes:
52+ - d/p/0002-Fix-compilation-error-due-to-new-strlcpy-function.patch:
53+ Fix FTBFS due to strlcpy() introduced in glibc 2.38.
54+ - d/p/0003-Support-module-memory-layout-change-on-Linux-6.4.patch:
55+ Fix fatal 'crash: invalid structure member offset: module_core_size'
56+
57+ -- Mauricio Faria de Oliveira <mfo@canonical.com> Tue, 07 Nov 2023 15:00:36 -0300
58+
59 crash (8.0.3+ds1-3) unstable; urgency=medium
60
61 * New upload to resolve missing gdb
62@@ -92,6 +138,16 @@ crash (8.0.3-1) unstable; urgency=medium
63
64 -- Troy Heber <troyh@debian.org> Sat, 28 Oct 2023 12:21:36 -0600
65
66+crash (8.0.2-1ubuntu1) mantic; urgency=medium
67+
68+ * Merge with Debian; remaining changes:
69+ - Add linux-libc-dev dependency for the autopkg test. This package
70+ gets usually broken with kernel upgrades, so let it already show
71+ in the autopkg test.
72+ - Run autopkg test with allow-stderr.
73+
74+ -- Matthias Klose <doko@ubuntu.com> Thu, 06 Jul 2023 16:49:16 +0200
75+
76 crash (8.0.2-1) unstable; urgency=medium
77 * New upstream
78 * commit f1cd581d1c4afa5b8ffdfaa6a3ea9f545fe4ec91
79@@ -2242,6 +2298,18 @@ crash (8.0.1-1) UNRELEASED; urgency=medium
80
81 -- Troy Heber <troyh@debian.org> Thu, 17 Nov 2022 10:27:25 -0700
82
83+crash (8.0.0-1ubuntu1) jammy; urgency=medium
84+
85+ * Merge with Debian; remaining changes:
86+ - Build without lto. Fails to build gdb on ppc64el. That should be fixed,
87+ once gdb is updated to a more recent version (e.g. 10.x).
88+ - Add linux-libc-dev dependency for the autopkg test. This package
89+ gets usually broken with kernel upgrades, so let it already show
90+ in the autopkg test.
91+ * Run autopkg test with allow-stderr.
92+
93+ -- Matthias Klose <doko@ubuntu.com> Wed, 23 Mar 2022 15:07:01 +0100
94+
95 crash (8.0.0-1) unstable; urgency=medium
96 * New upstream (Closes: #950544)
97 * Add lintian override for zlib in embedded gdb
98@@ -3374,6 +3442,17 @@ crash (8.0.0-1) unstable; urgency=medium
99
100 -- Troy Heber <troyh@debian.org> Wed, 02 Feb 2022 09:19:00 -0700
101
102+crash (7.3.0-1ubuntu1) impish; urgency=medium
103+
104+ * Merge with Debian; remaining changes:
105+ - Build without lto. Fails to build gdb on ppc64el. That should be fixed,
106+ once gdb is updated to a more recent version (e.g. 10.x).
107+ - Add linux-libc-dev dependency for the autopkg test. This package
108+ gets usually broken with kernel upgrades, so let it already show
109+ in the autopkg test.
110+
111+ -- dann frazier <dann.frazier@canonical.com> Thu, 08 Jul 2021 09:16:21 -0600
112+
113 crash (7.3.0-1) experimental; urgency=medium
114
115 * Add support for new lockless righbuffer that Linux 5.10 introduced.
116@@ -3555,6 +3634,27 @@ crash (7.3.0-1) experimental; urgency=medium
117
118 -- Troy Heber <troyh@debian.org> Wed, 07 Jul 2021 15:31:12 -0600
119
120+crash (7.2.9-2ubuntu3) impish; urgency=medium
121+
122+ * Add linux-libc-dev dependency for the autopkg test. This package
123+ gets usually broken with kernel upgrades, so let it already show
124+ in the autopkg test.
125+
126+ -- Matthias Klose <doko@ubuntu.com> Mon, 21 Jun 2021 09:54:43 +0200
127+
128+crash (7.2.9-2ubuntu2) hirsute; urgency=medium
129+
130+ * Build without lto. Fails to build gdb on ppc64el. That should be fixed,
131+ once gdb is updated to a more recent version (e.g. 10.x).
132+
133+ -- Matthias Klose <doko@ubuntu.com> Thu, 25 Mar 2021 10:04:06 +0100
134+
135+crash (7.2.9-2ubuntu1) hirsute; urgency=medium
136+
137+ * Merge with Debian; remaining changes:
138+
139+ -- Matthias Klose <doko@ubuntu.com> Thu, 25 Mar 2021 09:08:04 +0100
140+
141 crash (7.2.9-2) unstable; urgency=medium
142
143 * Backporting future patches to support printk in kernel 5.10+ based on:
144@@ -3810,6 +3910,17 @@ crash (7.2.9-1) unstable; urgency=medium
145
146 -- Troy Heber <troyh@debian.org> Thu, 03 Dec 2020 09:53:49 -0700
147
148+crash (7.2.8-1ubuntu1) focal; urgency=low
149+
150+ * Merge from Debian unstable. Remaining changes:
151+ - Update the maintainer field for Ubuntu.
152+ - If the "live" autopkgtest fails with a recommendation to try
153+ /proc/kcore instead of the default, attempt that before failing
154+ the test. LP: #1858958.
155+ * Fix test, as if will return 0 when no cases were true.
156+
157+ -- Thadeu Lima de Souza Cascardo <cascardo@canonical.com> Wed, 12 Feb 2020 22:12:51 +0000
158+
159 crash (7.2.8-1) unstable; urgency=medium
160
161 * New upstream (Closes: #950544)
162@@ -4139,6 +4250,40 @@ crash (7.2.7-1) UNRELEASED; urgency=medium
163
164 -- Troy Heber <troyh@debian.org> Tue, 11 Feb 2020 19:19:28 -0700
165
166+crash (7.2.6-1ubuntu4) focal; urgency=medium
167+
168+ * No-change rebuild with fixed binutils on arm64.
169+
170+ -- Matthias Klose <doko@ubuntu.com> Sat, 08 Feb 2020 10:57:37 +0000
171+
172+crash (7.2.6-1ubuntu3) focal; urgency=medium
173+
174+ * Rebuild with updated binutils, fixing an "illegal insruction" crash
175+ on arm64. LP: #1859226.
176+ * Update the maintainer field for Ubuntu.
177+
178+ -- dann frazier <dannf@ubuntu.com> Sat, 11 Jan 2020 09:36:27 -0700
179+
180+crash (7.2.6-1ubuntu2) focal; urgency=medium
181+
182+ * Fix logic issue in "live" autopkgtest introduced in the last upload.
183+
184+ -- dann frazier <dannf@ubuntu.com> Thu, 09 Jan 2020 19:24:17 -0700
185+
186+crash (7.2.6-1ubuntu1) focal; urgency=medium
187+
188+ * If the "live" autopkgtest fails with a recommendation to try
189+ /proc/kcore instead of the default, attempt that before failing
190+ the test. LP: #1858958.
191+
192+ -- dann frazier <dannf@ubuntu.com> Thu, 09 Jan 2020 14:21:45 -0700
193+
194+crash (7.2.6-1build1) eoan; urgency=medium
195+
196+ * No-change upload with strops.h and sys/strops.h removed in glibc.
197+
198+ -- Matthias Klose <doko@ubuntu.com> Thu, 05 Sep 2019 10:47:07 +0000
199+
200 crash (7.2.6-1) unstable; urgency=medium
201
202 * Two fixes for the Xen hypervisor; the first fixes a bug seen with Xen
203diff --git a/debian/control b/debian/control
204index cd27d75..beb893f 100644
205--- a/debian/control
206+++ b/debian/control
207@@ -1,7 +1,8 @@
208 Source: crash
209 Section: utils
210 Priority: optional
211-Maintainer: Troy Heber <troyh@debian.org>
212+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
213+XSBC-Original-Maintainer: Troy Heber <troyh@debian.org>
214 Uploaders:
215 Build-Depends: debhelper (>= 13), dpkg-dev (>= 1.16.1), quilt (>= 0.47), binutils, binutils-dev, bison, gawk, flex, zlib1g-dev, libncurses5-dev, liblzo2-dev, libsnappy-dev, libzstd-dev, texinfo
216 Standards-Version: 4.6.2.0
217diff --git a/debian/tests/control b/debian/tests/control
218index 6a60b96..45bd121 100644
219--- a/debian/tests/control
220+++ b/debian/tests/control
221@@ -1,3 +1,3 @@
222 Tests: live
223-Restrictions: needs-root isolation-machine
224-Depends: @, lsb-release, software-properties-common, sudo
225+Restrictions: needs-root isolation-machine allow-stderr
226+Depends: @, lsb-release, software-properties-common, sudo, linux-libc-dev
227diff --git a/debian/tests/live b/debian/tests/live
228index df59b62..b7e0cfc 100644
229--- a/debian/tests/live
230+++ b/debian/tests/live
231@@ -18,5 +18,21 @@ elif [ "$(lsb_release -is)" = "Ubuntu" ]; then
232 sudo DEBIAN_FRONTEND=noninteractive apt-get install -y -q linux-image-$(uname -r)-dbgsym
233 fi
234
235+crash_out="$(mktemp)"
236+cleanup() {
237+ rm -f $crash_out
238+}
239+trap cleanup EXIT
240+
241 echo "Testing crash on live kernel"
242-crash -st /usr/lib/debug/boot/vmlinux-$(uname -r) 2>&1
243+if crash -st /usr/lib/debug/boot/vmlinux-$(uname -r) > $crash_out 2>&1; then
244+ exit 0
245+else
246+ ret=$?
247+fi
248+
249+if ! grep -q "please use /proc/kcore" $crash_out; then
250+ exit $ret
251+fi
252+echo "Retrying using /proc/kcore (LP: #1858958)"
253+crash -st /usr/lib/debug/boot/vmlinux-$(uname -r) /proc/kcore 2>&1

Subscribers

People subscribed via source and target branches