[PATCH] remove long options from hdparm + minor bugfixes

Tito farmatito at tiscali.it
Wed May 31 23:18:36 UTC 2006


On Thursday 1 June 2006 00:53, Rob Landley wrote:
> On Wednesday 31 May 2006 6:03 pm, Tito wrote:
> > Hi to all,
> > this patch:
> > 1)	removes totally long option support from hdparm,
> > 	but the --Istdin functionality is preserved because now the program
> > 	reads from stdin if no device names are specified:
> >
> > 	cat /proc/ide/hda/identify |./busybox hdparm
> 
> Hmmm...  The hdparm people are used to dumps its --help when you run it 
> without arguments.

They will wonder why the program blocks... ;-P
hit Ctrl-c
do hdparm -h
and discover the truth!!!

> Could you stick in a check for stdin isn't a TTY? 

I could, but our menuconfig system and the verbose usage describe the non standard behaviour,
so maybe it is not worth the trouble....

Ciao,
Tito
> > 2) fixes the handling of -h.
> > 3) fixes usage.h to match the current behaviour.
> > 4) probably reduces size (i forgot to save the reference binary...).
> >
> > The patch is tested and seems to work.
> > Please apply, if you like it.
> 
> Applied.
> 
> Rob



More information about the busybox mailing list