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: [U-Boot] [PATCH 0/2] cmd: auto-complete args starting with a $
Date: Tue, 04 Dec 2018 10:44:19 +0100	[thread overview]
Message-ID: <20181204094419.6907C242267@gemini.denx.de> (raw)
In-Reply-To: <20181203220726.19370-1-boris.brezillon@bootlin.com>

Dear Boris,

In message <20181203220726.19370-1-boris.brezillon@bootlin.com> you wrote:
> 
> It's pretty common to pass arguments that start with a $ and are then
> expanded by the shell, and I'm this kind of lazy guy that hits tab all
> the time and expects the shell to suggest something appropriate. So
> here is a patchset adding support for ${} auto-completion and using the
> new helper from the mtd command.

You mean, this feature depends on MTD support?  this is not goot.

Also, you should make this feature configurable.  Not everybody may
want to use it or may have the memory available.

Technically - should autocompletion not be prevented for escaped
'$' characters, i. e. hitting TAB after a '\$' sequence should NOT
expand?

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 use modules, you pay the price. Sane embedded solutions
running in "tight" environments don't use modules :-)
    -- Benjamin Herrenschmidt in <1258234866.2140.451.camel@pasglop>

  parent reply	other threads:[~2018-12-04  9:44 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-03 22:07 [U-Boot] [PATCH 0/2] cmd: auto-complete args starting with a $ Boris Brezillon
2018-12-03 22:07 ` [U-Boot] [PATCH 1/2] common: command: Provide a dollar_complete() helper Boris Brezillon
2018-12-03 22:07 ` [U-Boot] [PATCH 2/2] cmd: mtd: auto-complete args starting with a $ when appropriate Boris Brezillon
2018-12-04  9:44 ` Wolfgang Denk [this message]
2018-12-04  9:54   ` [U-Boot] [PATCH 0/2] cmd: auto-complete args starting with a $ Boris Brezillon
2018-12-04 10:14     ` Wolfgang Denk
2018-12-04 10:33       ` Boris Brezillon
2018-12-04 13:00         ` Wolfgang Denk
2018-12-04 13:11           ` Boris Brezillon
2018-12-04 13:54         ` Boris Brezillon

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=20181204094419.6907C242267@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