[coreboot] [PATCH] flashrom: Tell users that the command line interface may change

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Tue Apr 28 15:28:45 CEST 2009


Ping?

On 27.04.2009 18:43, Carl-Daniel Hailfinger wrote:
> Since the command line interface for flashrom will change for 1.0
> (all-caps or no-caps for short options, exclude range syntax, etc.) we
> should tell users in the man page and the usage message about this.
>
> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>
> Index: flashrom-usage_hint/flashrom.8
> ===================================================================
> --- flashrom-usage_hint/flashrom.8	(Revision 444)
> +++ flashrom-usage_hint/flashrom.8	(Arbeitskopie)
> @@ -17,6 +17,11 @@
>  .B http://coreboot.org
>  for details on coreboot)
>  .SH OPTIONS
> +Please note that the command line interface for flashrom will change before
> +flashrom 1.0. Do not use flashrom in scripts or other automated tools without
> +checking that your flashrom version won't interpret them in a totally different
> +way.
> +.PP
>  If no file is specified, then all that happens
>  is that flash info is dumped and the flash chip is set to writable.
>  .TP
> Index: flashrom-usage_hint/flashrom.c
> ===================================================================
> --- flashrom-usage_hint/flashrom.c	(Revision 444)
> +++ flashrom-usage_hint/flashrom.c	(Arbeitskopie)
> @@ -306,6 +306,10 @@
>  	printf("usage: %s [-rwvEVfLhR] [-c chipname] [-s exclude_start]\n",
>  	       name);
>  	printf("       [-e exclude_end] [-m [vendor:]part] [-l file.layout] [-i imagename] [file]\n");
> +	printf("Please  note that the command line interface for flashrom will "
> +		"change before flashrom 1.0. Do not use flashrom in scripts or "
> +		"other automated tools without checking that your flashrom "
> +		"version won't interpret them in a totally different way.\n\n");
>  	printf
>  	    ("   -r | --read:                      read flash and save into file\n"
>  	     "   -w | --write:                     write file into flash\n"
>
>
>   


-- 
http://www.hailfinger.org/





More information about the coreboot mailing list