grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH] grub-reboot: Warn when "for the next boot only" promise cannot b


From: dann frazier
Subject: [PATCH] grub-reboot: Warn when "for the next boot only" promise cannot be kept
Date: Fri, 17 Aug 2018 17:33:44 -0600
User-agent: Mutt/1.10.1 (2018-07-13)

The "for the next boot only" property of grub-reboot is dependent upon
GRUB being able to clear the next_entry variable in the environment
block. However, GRUB cannot write to devices using the diskfilter
and lvm abstractions.

Ref: https://lists.gnu.org/archive/html/grub-devel/2009-12/msg00276.html
Ref: https://bugs.launchpad.net/bugs/788298

Signed-off-by: dann frazier <address@hidden>
---
 util/grub-reboot.in | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/util/grub-reboot.in b/util/grub-reboot.in
index 4d4f75704..ef3b5c049 100644
--- a/util/grub-reboot.in
+++ b/util/grub-reboot.in
@@ -20,6 +20,7 @@
 address@hidden@
 address@hidden@
 address@hidden@
address@hidden@
 sysconfdir="@sysconfdir@"
 address@hidden@
 address@hidden@
@@ -32,6 +33,7 @@ fi
 self=`basename $0`
 
 grub_editenv=${bindir}/@grub_editenv@
+grub_probe=${sbindir}/@grub_probe@
 rootdir=
 bootdir=
 grubdir=`echo "/@bootdirname@/@grubdirname@" | sed 's,//*,/,g'`
@@ -56,6 +58,8 @@ submenus or sub-submenus require specifying the submenu 
components and then the
 menu item component. The titles should be separated using the greater-than
 character (>) with no extra spaces. Depending on your shell some characters 
including > may need escaping. More information about this is available
 in the GRUB Manual in the section about the 'default' command. "; echo
+    echo
+    gettext "NOTE: In cases where GRUB cannot write to the environment block, 
such as when it is stored on an MDRAID or LVM device, the chosen boot menu 
entry will remain the default even after reboot. "; echo
     echo
     gettext "Report bugs to <address@hidden>."; echo
 }
@@ -131,6 +135,18 @@ fi
 
 grubdir=`echo "${bootdir}/@grubdirname@" | sed 's,//*,/,g'`
 
+abstractions=`$grub_probe --target=abstraction ${grubdir}/grubenv`
+for abstraction in $abstractions; do
+    case "$abstraction" in
+       diskfilter | lvm)
+           gettext_printf "\nWARNING: Detected GRUB environment block on 
$abstraction device\n"
+           gettext_printf "%s will remain the default boot entry until 
manually cleared with:\n" "${entry}"
+           gettext_printf "    grub-editenv ${grubdir}/grubenv unset 
next_entry\n\n"
+           break
+           ;;
+    esac
+done
+
 # Restore saved_entry if it was set by previous version
 prev_saved_entry=`$grub_editenv ${grubdir}/grubenv list | sed -n 
's/^prev_saved_entry=//p'`
 if [ "$prev_saved_entry" ]; then
-- 
2.18.0




reply via email to

[Prev in Thread] Current Thread [Next in Thread]