[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