]> git.dujemihanovic.xyz Git - u-boot.git/commit
lib: Add getopt
authorSean Anderson <seanga2@gmail.com>
Tue, 27 Oct 2020 23:55:36 +0000 (19:55 -0400)
committerTom Rini <trini@konsulko.com>
Fri, 30 Oct 2020 14:56:11 +0000 (10:56 -0400)
commit72eda5074b92d54ff67979f7a45ced9a119c4a8c
tree16758651c9b32cf51a8f1927b45be2e6af861600
parent018aad8750181590f84afc42512216946f6caeab
lib: Add getopt

Some commands can get very unweildy if they have too many positional
arguments. Adding options makes them easier to read, remember, and
understand.

This implementation of getopt has been taken from barebox, which has had
option support for quite a while. I have made a few modifications to their
version, such as the removal of opterr in favor of a separate getopt_silent
function. In addition, I have moved all global variables into struct
getopt_context.

The getopt from barebox also re-orders the arguments passed to it so that
non-options are placed last. This allows users to specify options anywhere.
For example, `ls -l foo/ -R` would be re-ordered to `ls -l -R foo/` as
getopt parsed the options. However, this feature conflicts with the const
argv in cmd_tbl->cmd. This was originally added in 54841ab50c ("Make sure
that argv[] argument pointers are not modified."). The reason stated in
that commit is that hush requires argv to stay unmodified. Has this
situation changed? Barebox also uses hush, and does not have this problem.
Perhaps we could use their fix?

I have assigned maintenance of getopt to Simon Glass, as it is currently
only used by the log command. I would also be fine maintaining it.

Signed-off-by: Sean Anderson <seanga2@gmail.com>
MAINTAINERS
doc/api/getopt.rst [new file with mode: 0644]
doc/api/index.rst
include/getopt.h [new file with mode: 0644]
lib/Kconfig
lib/Makefile
lib/getopt.c [new file with mode: 0644]