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