All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrey Borzenkov <arvidjaar@gmail.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Cc: psusi@ubuntu.com
Subject: Re: grub-install hidden options weirdness
Date: Wed, 25 Jun 2014 21:33:35 +0400	[thread overview]
Message-ID: <20140625213335.3e683422@opensuse.site> (raw)
In-Reply-To: <53AAD927.1000806@ubuntu.com>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

В Wed, 25 Jun 2014 10:13:59 -0400
Phillip Susi <psusi@ubuntu.com> пишет:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> A user complained that grub-install has some undocumented options such
> as --root-directory.  I started looking into it and noticed a lot of
> weirdness here.
> 
> 1)  It seems that --root-directory used to just be used to find the
> boot directory by appending "boot" to rootdir.  --boot-directory was
> added as a more appropriate way to directly specify that.  Shouldn't
> the man page mention that it still accepts --root-directory for
> backwards compatibility but it's use is depreciated?
> 

This could be considered as endorsement to continue to use it ...  

> 2)  It seems that --root-directory has been overloaded to specify the
> efi system directory.  This seems like it was the wrong thing to do
> and again, is totally undocumented.
> 

We already have --efi-directory. Why would you want to endorse this
archaic usage?  

> 3)  It appears that --grub-setup used to allow you to specify an
> alternate program to run instead of grub-setup.  Now it seems to check
> if its argument is "setup" and if so, has the same affect as
> - --no-bootsector, and otherwise has no effect.  This seems completely
> wrong.
> 

This covers two common use cases known to me a) using it from within
build directory with --grub=setup=./grub-setup and b) using it to
disable actual installation with --grub-setup=/bin/true. Current code
is compatible with them.

I know that some people did creative things with grub-setup
replacement ... but it normally can be done using grub-mkimage +
whatever replaces grub-setup. The only thing I miss in this case is
autodetection of necessary modules and prefix. May be grub-probe can
be extended.

> 4)  The similar arguments --grub-mkrelpath, --grub-probe,
> - --grub-editenv, and --font are accepted and completely ignored.  If
> the option no longer works, it should be removed rather than silently
> ignored.  At least that way you don't have people using it and
> wondering why it isn't working.
> 

Again - common use case is using just built grub without installing it.
Do you have example of other non-trivial use cases?
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)

iEYEARECAAYFAlOrB+8ACgkQR6LMutpd94wHhACeL06y+EWV8QX1Ifyye4WknL3m
+H8AoK7KXkhuAotkSD/Pjz4J7Ls3pHSn
=YCwP
-----END PGP SIGNATURE-----

  reply	other threads:[~2014-06-25 17:33 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-25 14:13 grub-install hidden options weirdness Phillip Susi
2014-06-25 17:33 ` Andrey Borzenkov [this message]
2014-06-25 17:44   ` Phillip Susi
2014-06-25 18:48     ` Vladimir 'φ-coder/phcoder' Serbinenko

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=20140625213335.3e683422@opensuse.site \
    --to=arvidjaar@gmail.com \
    --cc=grub-devel@gnu.org \
    --cc=psusi@ubuntu.com \
    /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.