From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [PATCH] efidebug: Introduce bootmgr command
Date: Wed, 10 Feb 2021 13:26:15 +0100 [thread overview]
Message-ID: <79242.1612959975@gemini.denx.de> (raw)
In-Reply-To: <YCPJU0hIUTayLuhw@apalos.home>
Dear Ilias,
In message <YCPJU0hIUTayLuhw@apalos.home> you wrote:
>
> The efidebug for boot options wasn't introduced that long ago and I don't
> think anyone uses it in production. If someone would want to have it backwards
> compatible, please shout and we'll see what we can do, but I'd strongly prefer
> replacing it overall. If we truly want backwards compatibility though we must keep
> efidebug, changing the name to something like 'efi debug' just for the name
> similarity wouldn't help much as it would break things regardless.
In this case, "debug" would just be a sub-command of the "efi"
command, with more sub-commands under efi (like bootmgr or such)
following, same as we did for example with "env" (where many
commands maintain backward compatibility, but here this was
necessary because these have been in use forever):
env print -> printenv
env save -> saveenv
env set -> setenv
etc.
Maybe a similar approach makes sense for "efi" (with or without
backward compatibility, as you like - after all, this is just a
little name space pollution ;-) :
efi:
efi debug
efi bootmgr
efi ...
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
If you believe that feeling bad or worrying long enough will change a
past or future event, then you are residing on another planet with a
different reality system.
next prev parent reply other threads:[~2021-02-10 12:26 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-10 10:54 [PATCH] efidebug: Introduce bootmgr command Ilias Apalodimas
2021-02-10 11:23 ` Wolfgang Denk
2021-02-10 11:53 ` Ilias Apalodimas
2021-02-10 12:26 ` Wolfgang Denk [this message]
2021-02-10 12:46 ` Ilias Apalodimas
2021-02-10 16:10 ` Wolfgang Denk
2021-02-10 12:50 ` AKASHI Takahiro
2021-02-10 14:59 ` Ilias Apalodimas
2021-02-10 12:53 ` Heinrich Schuchardt
2021-02-10 16:17 ` Wolfgang Denk
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=79242.1612959975@gemini.denx.de \
--to=wd@denx.de \
--cc=u-boot@lists.denx.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox