Merge ~xnox/grub:focal-fix-postinst-empty-device into ~ubuntu-core-dev/grub/+git/ubuntu:focal

Proposed by Dimitri John Ledkov
Status: Merged
Merged at revision: 1448fc1dc18e068af87069b0041f71dec03719dd
Proposed branch: ~xnox/grub:focal-fix-postinst-empty-device
Merge into: ~ubuntu-core-dev/grub/+git/ubuntu:focal
Diff against target: 56 lines (+24/-2)
3 files modified
debian/changelog (+8/-0)
debian/grub-multi-install (+8/-1)
debian/postinst.in (+8/-1)
Reviewer Review Type Date Requested Status
Michael Vogt Pending
Julian Andres Klode Pending
Ubuntu Core Development Team Pending
Review via email: mp+391439@code.launchpad.net

Commit message

postinst.in, grub-multi-install: fix logic of skipping installing onto any device, if one chose to not install bootloader on any device. LP: #1896608

To post a comment you must log in.

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 181b526..25d552d 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,11 @@
6+grub2 (2.04-1ubuntu26.6) UNRELEASED; urgency=medium
7+
8+ * postinst.in, grub-multi-install: fix logic of skipping installing onto
9+ any device, if one chose to not install bootloader on any device. LP:
10+ #1896608
11+
12+ -- Dimitri John Ledkov <xnox@ubuntu.com> Mon, 28 Sep 2020 14:11:41 +0100
13+
14 grub2 (2.04-1ubuntu26.5) focal; urgency=medium
15
16 * ubuntu-linuxefi-arm64.patch: Fix build on armhf (LP: #1862279)
17diff --git a/debian/grub-multi-install b/debian/grub-multi-install
18index 9bd4089..bedc700 100755
19--- a/debian/grub-multi-install
20+++ b/debian/grub-multi-install
21@@ -402,7 +402,14 @@ while :; do
22 db_fset grub-efi/install_devices_empty seen false
23 fi
24 else
25- exit 1 # noninteractive
26+ # if question was seen we are done
27+ # Otherwise, abort
28+ db_fget grub-efi/install_devices_empty seen
29+ if [ "$RET" = true ]; then
30+ break
31+ else
32+ exit 1
33+ fi
34 fi
35 else
36 break
37diff --git a/debian/postinst.in b/debian/postinst.in
38index e6dfe36..cedc137 100644
39--- a/debian/postinst.in
40+++ b/debian/postinst.in
41@@ -685,7 +685,14 @@ case "$1" in
42 db_fset grub-pc/install_devices_empty seen false
43 fi
44 else
45- exit 1 # noninteractive
46+ # if question was seen we are done
47+ # Otherwise, abort
48+ db_fget grub-pc/install_devices_empty seen
49+ if [ "$RET" = true ]; then
50+ break
51+ else
52+ exit 1
53+ fi
54 fi
55 else
56 break

Subscribers

People subscribed via source and target branches