[PATCH] remove long options from hdparm + minor bugfixes

Rob Landley rob at landley.net
Wed May 31 22:53:21 UTC 2006


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.  Could you stick in a check for stdin isn't a TTY?

> 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
-- 
Never bet against the cheap plastic solution.



More information about the busybox mailing list