[PATCH 10/18] commands: add readlink support

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Fri Aug 24 14:46:06 EDT 2012


On 11:31 Fri 24 Aug     , Roberto Nibali wrote:
> 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"?
no barebox read the value and not display it

Best Regards,
J.



More information about the barebox mailing list