From: Sascha Hauer <s.hauer@pengutronix.de>
To: Holger Schurig <holgerschurig@gmail.com>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 00/19] commands rework patch series
Date: Tue, 13 May 2014 15:46:39 +0200 [thread overview]
Message-ID: <20140513134639.GR5858@pengutronix.de> (raw)
In-Reply-To: <1399969739-10355-1-git-send-email-holgerschurig@gmail.com>
Hi Holger,
On Tue, May 13, 2014 at 10:28:40AM +0200, Holger Schurig wrote:
> Hi,
>
> this is my current patch series that should improve barebox'
> documentation about it's commands.
>
> * when I started barebox and typed "help", the list of commands
> was scrolling up quickly. Almost always did I had to use the
> mouse and scroll my x-terminal backwards to see what I
> actually wanted to see. So I reduced the output of the "help"
> command to just spit out the command names, however grouped
> by functions. This is now much smaller and fits in one
> screen. The old list is available with "help -l".
> * Usage (synopsis) documention (e.g. "ls [-lCR]") was only in
> the CONFIG_LONGHELP, but it's often usable if one forget the
> exact command option letter. It's now part of the help even
> without CONFIG_LONGHELP.
> * I reworked the long help texts to make them better understandable
> (hopefully), and also tried to recuce their size if possible.
> * when I was new to barebox, I was wandering if I need some
> of the commands in "make xconfig". I didn't have further
> information, because almost no command had a help text.
> This has now changed, the help texts in Kconfig are almost the
> same as the help texts of "help FOOBAR" (with CONFIG_LONGHELP) on.
> * now that "help" has the commands grouped nicely, it was time
> to group them in "make xconfig", too
> * many Kconfig entries turned on commands, but they were located
> at various places (a.g. common, net, arch-depended). Moved them
> all into commands/Kconfig
> * some CONFIG_CMD_foobar things didn't turn on commands. I renamed
> them (and their occurences in the _defconfig)s.
This series looks really great! I haven't looked at the details, but it
has some very nice aspects, like the command grouping and the same
grouping into Kconfig submenus.
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2014-05-13 13:47 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-13 8:28 Holger Schurig
2014-05-13 8:28 ` [PATCH 01/19] commands: group 'help' output Holger Schurig
2014-05-13 13:48 ` Sascha Hauer
2014-05-13 14:07 ` Holger Schurig
2014-05-13 8:28 ` [PATCH 02/19] commands: harmonize in-barebox documentation Holger Schurig
2014-05-13 8:28 ` [PATCH 03/19] commands: harmonize in-barebox docs with Kconfig docs Holger Schurig
2014-05-13 8:28 ` [PATCH 04/19] commands: CMD_MEMORY -> COMPILE_MEMORY Holger Schurig
2014-05-13 8:28 ` [PATCH 05/19] commands: CMD_DIGEST -> COMPILE_DIGEST Holger Schurig
2014-05-13 8:28 ` [PATCH 06/19] commands: move CMD_ARM_CPUINFO to commands/Kconfig Holger Schurig
2014-05-13 8:28 ` [PATCH 07/19] commands: move CMD_ARM_MMUINFO " Holger Schurig
2014-05-13 8:28 ` [PATCH 08/19] commands: move CMD_MIPS_CPUINFO " Holger Schurig
2014-05-13 11:26 ` Antony Pavlov
2014-05-13 11:28 ` Holger Schurig
2014-05-13 12:23 ` Antony Pavlov
2014-05-13 12:43 ` Holger Schurig
2014-05-13 13:09 ` Juergen Borleis
2014-05-13 8:28 ` [PATCH 09/19] commands: move CMD_BOOT_ORDER " Holger Schurig
2014-05-13 8:28 ` [PATCH 10/19] commands: move CMD_AT91_BOOT_TEST " Holger Schurig
2014-05-13 8:28 ` [PATCH 11/19] commands: move CMD_AT91MUX " Holger Schurig
2014-05-13 8:28 ` [PATCH 12/19] commands: move CMD_AT91MIX " Holger Schurig
2014-05-13 8:28 ` [PATCH 13/19] commands: HUSH_GETOPT -> CMD_GETOPT Holger Schurig
2014-05-13 8:28 ` [PATCH 14/19] commands: let all network commands depend on NET Holger Schurig
2014-05-13 8:28 ` [PATCH 15/19] commands: introduce CMD_HOST Holger Schurig
2014-05-13 8:28 ` [PATCH 16/19] commands: NET_PING -> CMD_PING Holger Schurig
2014-05-13 8:28 ` [PATCH 17/19] commands: NET_DHCP -> CMD_DHCP Holger Schurig
2014-05-13 8:28 ` [PATCH 18/19] commands: move CMD_IFUP to commands/Kconfig Holger Schurig
2014-05-13 8:28 ` [PATCH 19/19] commands: move CONFIG_LONGHELP " Holger Schurig
2014-05-13 8:30 ` [PATCH 00/19] commands rework patch series Holger Schurig
2014-05-13 8:41 ` Sascha Hauer
2014-05-13 9:00 ` Holger Schurig
2014-05-13 13:46 ` Sascha Hauer [this message]
2014-05-14 7:02 ` Sascha Hauer
2014-05-14 13:55 ` Antony Pavlov
2014-05-15 8:09 ` Holger Schurig
2014-05-15 11:52 ` Sascha Hauer
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=20140513134639.GR5858@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=barebox@lists.infradead.org \
--cc=holgerschurig@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox