Bug 15692 - install.sh in UEFI has useless parameters
Summary: install.sh in UEFI has useless parameters
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: Installer (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Thierry Vignaud
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-13 22:13 CEST by Barry Jackson
Modified: 2015-04-15 06:07 CEST (History)
1 user (show)

See Also:
Source RPM: drakx
CVE:
Status comment:


Attachments
just run grub2-install for UEFI (816 bytes, patch)
2015-04-13 22:34 CEST, Thierry Vignaud
Details | Diff

Description Barry Jackson 2015-04-13 22:13:42 CEST
Description of problem:
As summary.
The grub2-install command in install.sh contains the following in a clean UEFI install:

"grub2-install --grub-setup=/bin/true /dev/sda --directory /usr/lib/grub/x86_64-efi"

AFAIK none of those parameters are required (or valid) in UEFI mode.

grub2-install

is enough.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.


Reproducible: 

Steps to Reproduce:
Comment 1 Thierry Vignaud 2015-04-13 22:34:52 CEST
Created attachment 6263 [details]
just run grub2-install for UEFI

So you would advice this?

CC: (none) => thierry.vignaud

Comment 2 Barry Jackson 2015-04-14 00:50:42 CEST
Yes, to quote Andrei Borzenkov (a grub2 dev) in reference to my query relating to next line:

> grub2-install --grub-setup=/bin/true /dev/sda4

"Both arguments are redundant. Install device is ignored on EFI,
grub-setup is never called there either."
Thierry Vignaud 2015-04-14 06:01:07 CEST

Summary: install.sh in UEFI has invalid parameters => install.sh in UEFI has useless parameters

Comment 3 Mageia Robot 2015-04-14 22:21:08 CEST
commit 8c3bec9a1c13fed19839e8a85a97ddadfaf17315
Author: Thierry Vignaud <thierry.vignaud@...>
Date:   Mon Apr 13 22:32:32 2015 +0200

    do not pass useless parameters (mga#15692)
    
    ... to grub2 under UEFI
---
 Commit Link:
   http://gitweb.mageia.org/software/drakx/commit/?id=8c3bec9a1c13fed19839e8a85a97ddadfaf17315
Comment 4 Thierry Vignaud 2015-04-15 06:07:52 CEST
Fixed

Status: NEW => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.