All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Vladimir 'phcoder' Serbinenko" <phcoder@gmail.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: [PATCH v2] grub-install: Mention *efibootmgr* in `no-nvram` description
Date: Thu, 6 Sep 2018 20:37:50 +0300	[thread overview]
Message-ID: <CAEaD8JP3c1G8G14dLxeYMCHXxvwm4b8qiRuqHuv19tN2TZT0rA@mail.gmail.com> (raw)
In-Reply-To: <e4480529-da63-cf66-93e8-99bbd0395450@molgen.mpg.de>

[-- Attachment #1: Type: text/plain, Size: 1578 bytes --]

On Thu, 6 Sep 2018, 17:03 Paul Menzel, <pmenzel@molgen.mpg.de> wrote:

> Date: Thu, 6 Sep 2018 15:15:45 +0200
>
> If executing `efibootmgr` fails, for example, because the system is
> booted in legacy mode, but with a mounted EFI System Partition, the
> error message mentions *efibootmgr*. To make it easier for users to work
> around that problem by skipping running `efibootmgr`, add the word to
> the description of the switch *no-nvram*, which skips the efibootmgr
> execution.
>
> Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
> ---
> v2: Add Signed-off-by line
>
>  util/grub-install.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/util/grub-install.c b/util/grub-install.c
> index a44858198..335662a58 100644
> --- a/util/grub-install.c
> +++ b/util/grub-install.c
> @@ -276,7 +276,7 @@ static struct argp_option options[] = {
>     N_("disk module to use (biosdisk or native). "
>        "This option is only available on BIOS target."), 2},
>    {"no-nvram", OPTION_NO_NVRAM, 0, 0,
> -   N_("don't update the `boot-device'/`Boot*' NVRAM variables. "
> +   N_("don't run efibootmgr to update the `boot-device'/`Boot*' NVRAM
> variables. "
>        "This option is only available on EFI and IEEE1275 targets."), 2},
>
In case of ieee1275 this has nothing to do with efibootmgr

>    {"skip-fs-probe",'s',0,      0,
>     N_("do not probe for filesystems in DEVICE"), 0},
> --
> 2.17.1
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel
>

[-- Attachment #2: Type: text/html, Size: 2424 bytes --]

  reply	other threads:[~2018-09-06 17:38 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-06 13:54 [PATCH] grub-install: Mention *efibootmgr* in no-nvram description Paul Menzel
2018-09-06 14:02 ` [PATCH v2] grub-install: Mention *efibootmgr* in `no-nvram` description Paul Menzel
2018-09-06 17:37   ` Vladimir 'phcoder' Serbinenko [this message]
2018-09-19 14:08     ` Daniel Kiper
2018-09-19 14:18       ` Paul Menzel
2018-09-21 17:58         ` Daniel Kiper

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAEaD8JP3c1G8G14dLxeYMCHXxvwm4b8qiRuqHuv19tN2TZT0rA@mail.gmail.com \
    --to=phcoder@gmail.com \
    --cc=grub-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.