On Saturday 03 May 2014 16:38:06 you wrote: > On Friday 02 May 2014 21:31, Tito wrote: > > On Friday 02 May 2014 15:42:40 you wrote: > > > Tito wrote: > > > > > > > That's it about PATCH 1. > > > > There is still PATCH 2.... > > > Your patch searches in PATH for dir/file if dir/file is not executable. > > > The normal behavior is to not search PATH if the argument contains a '/'. > > > > Fixed. Thanks! > > > > > If you rewrite "which" anyway, you could also change the following points: > > > - Remove default PATH, saves some bytes > > > > It mimics the behavior of debianutils which that being a script > > sets PATH (from profile?) in its subshell: > > > > debian:~/Desktop/SourceCode/new_bb/busybox$ cp /usr/bin/which . > > debian:~/Desktop/SourceCode/new_bb/busybox$ nano which (add echo $PATH) > > debian:~/Desktop/SourceCode/new_bb/busybox$ unset PATH > > debian:~/Desktop/SourceCode/new_bb/busybox$ echo $PATH > > > > > > > > debian:~/Desktop/SourceCode/new_bb/busybox$ ./which pippo > > > > PATH /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin > > debian:~/Desktop/SourceCode/new_bb/busybox$ echo $PATH > > > > > > > > > - Output message if not found (maybe if DESKTOP) > > > > > - Return number of commands not found as per man page (Which returns the > > > number of failed arguments, or -1 when no `programname´ was given) > > Applied, thanks! >
Thanks! PS: you credited the patch to Timo Teräs <timo.te...@iki.fi> maybe he will not be happy to be linked to my ugly patches ;-) _______________________________________________ busybox mailing list busybox@busybox.net http://lists.busybox.net/mailman/listinfo/busybox