[PATCH 10/18] commands: add readlink support

Roberto Nibali rnibali at gmail.com
Fri Aug 24 05:31:59 EDT 2012


Hi Jean-Christophe

Btw, congrats on such a good and lean work on the symlink implementation!

> ---
>  commands/Kconfig    |    6 ++++
>  commands/Makefile   |    1 +
>  commands/readlink.c |   78 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 85 insertions(+)
>  create mode 100644 commands/readlink.c
>
> diff --git a/commands/Kconfig b/commands/Kconfig
> index 92a8152..876f2f8 100644
> --- a/commands/Kconfig
> +++ b/commands/Kconfig
> @@ -220,6 +220,12 @@ config CMD_DIRNAME
>           Strip last component of file name and store the result in a
>           environment variable
>
> +config CMD_READLINK
> +       tristate
> +       prompt "readlink"
> +       help
> +         read value of a symbolic link
> +

Just a general two cents comment: how about generally rephrasing the
"read value of a symbolic link" to "display file status of a symbolic
link"?

Cheers
Roberto



More information about the barebox mailing list