public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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.

  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