templates: Enable fwsetup on EFI platforms only

Only perform call to fwsetup if one is on EFI platform. On all other
platforms fwsetup command does not exists, and thus returns 0 and
a useless uefi-firmware menu entry gets generated.

Signed-off-by: Dimitri John Ledkov <dimitri.ledkov@canonical.com>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>

diff --git a/util/grub.d/30_uefi-firmware.in b/util/grub.d/30_uefi-firmware.in
index c1731e5bbbb33191f11eadc7773bf944be1ef1ac..1c2365ddbd3517e0bc9b9aa16abac5b361198876 100644 (file)
--- a/util/grub.d/30_uefi-firmware.in
+++ b/util/grub.d/30_uefi-firmware.in
@@ -31,10 +31,12 @@ LABEL="UEFI Firmware Settings"
 gettext_printf "Adding boot menu entry for UEFI Firmware Settings ...\n" >&2
 
 cat << EOF
-fwsetup --is-supported
-if [ "\$grub_platform" = "efi" -a "\$?" = 0 ]; then
-       menuentry '$LABEL' \$menuentry_id_option 'uefi-firmware' {
-               fwsetup
-       }
+if [ "\$grub_platform" = "efi" ]; then
+       fwsetup --is-supported
+       if [ "\$?" = 0 ]; then
+               menuentry '$LABEL' \$menuentry_id_option 'uefi-firmware' {
+                       fwsetup
+               }
+       fi
 fi
 EOF