Merge ~apw/casper:main into casper:main

Proposed by Andy Whitcroft
Status: Merged
Approved by: Łukasz Zemczak
Approved revision: a4214f92fed09080c98d206e58397d22a11f9448
Merged at revision: a4214f92fed09080c98d206e58397d22a11f9448
Proposed branch: ~apw/casper:main
Merge into: casper:main
Diff against target: 27 lines (+7/-1)
2 files modified
debian/changelog (+6/-0)
debian/tests/boot (+1/-1)
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
Review via email: mp+394748@code.launchpad.net

Commit message

Fix version handling for forward backport kernel versions; these terminate in +21.04.1 which confuses the tests.

To post a comment you must log in.
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

The new regex seems fine, and it works for both the +21.04 scenario and regular versioning.

review: Approve

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 fd390fd..dfb8051 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,9 @@
6+casper (1.456) hirsute; urgency=medium
7+
8+ * Support kernels with "+" in the version string (LP: #1905357).
9+
10+ -- Andrea Righi <andrea.righi@canonical.com> Wed, 02 Dec 2020 12:04:34 +0000
11+
12 casper (1.455) groovy; urgency=medium
13
14 * Use a tmpfs in tests; overlay is not compatible with zfs.
15diff --git a/debian/tests/boot b/debian/tests/boot
16index 1054dd9..c0c3a27 100755
17--- a/debian/tests/boot
18+++ b/debian/tests/boot
19@@ -29,7 +29,7 @@ set -e
20
21 basecmdline="boot=casper console=ttyS0"
22
23-kvers="$(dpkg -l linux-image-generic | awk '/^ii/ { print $3 }' | sed -e 's/\.[0-9]\+$//; s/\.\([0-9]\+\)$/-\1/')"-generic
24+kvers="$(dpkg -l linux-image-generic | awk '/^ii/ { print $3 }' | sed -e 's/+.*//; s/\.[0-9]\+$//; s/\.\([0-9]\+\)$/-\1/')"-generic
25 kernel=$(echo /boot/vmlinu*-"$kvers")
26
27 mkinitramfs -o casper-initrd "$kvers"

Subscribers

People subscribed via source and target branches