[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