I've committed the patch. Notes:

On Dec/21/2009, Vladimir '??-coder/phcoder' Serbinenko wrote:
> === modified file 'commands/blocklist.c'
> --- commands/blocklist.c      2009-06-10 21:04:23 +0000
> +++ commands/blocklist.c      2009-12-21 00:09:02 +0000
> @@ -87,7 +87,7 @@ grub_cmd_blocklist (grub_command_t cmd _
>  
>    if (! file->device->disk)
>      return grub_error (GRUB_ERR_BAD_DEVICE,
> -                    "this command is available only for disk devices.");
> +                    "this command is available only for disk devices"); 
> Capitalisation and full stop

I'll talk about this in a new thread.

>    cmd = grub_register_command ("halt", grub_cmd_halt,
> -                            0, "halts the computer.  This command does not"
> +                            0, "Halts the computer.  This command does not"
>                              " work on all firmware.");
> I would rather say "On *every* firmware."

I followed Watson suggestion


>  static const struct grub_arg_option options[] =
>    {
> -    {"file", 'f', 0, "specify filename", 0, ARG_TYPE_PATHNAME},
> +    {"file", 'f', 0, "Specify filename.", 0, ARG_TYPE_PATHNAME},
> Filename for what?

I'll add a comment when I will gettextizze it.

-- 
Carles Pina i Estany
        http://pinux.info


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to