On 14:45 Fri 08 Oct , Sascha Hauer wrote: > On Fri, Oct 08, 2010 at 02:36:27PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote: > > On 14:24 Fri 08 Oct , Sascha Hauer wrote: > > > Positional parameters are not nice, so implement a getopt > > > function. This has to be done as a builtin because otherwise > > > we have no access to the parents argc/argv. > > > > > > getopt works as expected, here is a little example: > > > > > > while getopt "hs:" OPT > > > do > > > if [ $OPT = h ]; then > > > echo "usage" > > > exit 1 > > > else > > > echo "scr: opt: $OPT optarg: $OPTARG" > > > fi > > > done > > > > > > Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > > how about make it optional? > > Probably a good idea. I just had in mind that we need to adjust > the defconfigs then to still have a working defaultenv. > > How about making it optional and combined with your defaultenv-Kconfig > patch we could do a > > select HUSH > select HUSH_GETOPT > > to make sure all commands needed by the defaultenv are in place. that's why I move the defaultenv to Kconfig, I'm still in vacation. I'll send the patch tomorrow Best Regards, J. _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
