[BusyBox] New ps patch

Erik Andersen andersen at lineo.com
Fri Mar 16 23:52:44 UTC 2001


On Fri Mar 16, 2001 at 03:23:54PM +0100, Christophe Boyanique wrote:
> 
> Here is the sequel of my previous ps patch:
> 
> Corrects option parsing in ps.c (two ps_main() versions were not
> identical: first one never checked arguments and never called
> show_usage(); second one called show_usage() only if first char of
> arguments different from '-'.
> Now both functions called show_usage() if arguments are passed except if:
> 
> Optional:
>  
> Adds BB_FEATURE_PS to allow option parsing in ps
> 
> Adds BB_FEATURE_PS_ALL to change behaviour of ps: print only user's
> processes like usual ps:
> ps   -> ps ux
> ps a -> ps aux
> 
> I spread this in two parts to allow easy adding of new features.

If we are going to change ps, then having a single option
to add BB_FEATURE_PS_ALL is sufficient.  I don't see much
value in splitting these appart.  Mind reworking it abit
so there is just one BB_FEATURE_PS_ALL option?

 -Erik

--
Erik B. Andersen   email:  andersen at lineo.com
--This message was written using 73% post-consumer electrons--





More information about the busybox mailing list