Re: [PATCH 2/2] Alias cls as clear

2009-11-19 Thread Robert Millan
Committed with some adjustments to ChangeLog. On Tue, Jul 28, 2009 at 10:58:55PM -0700, Joe Auricchio wrote: > > On 25 Jul, 2009, at 09:21 , Robert Millan wrote: >> On Thu, Jul 23, 2009 at 04:05:00PM -0700, Joe Auricchio wrote: >>> >>> >>> How's this? >> >> The ChangeLog entry is incomplete. It

Re: [PATCH 2/2] Alias cls as clear

2009-07-31 Thread Robert Millan
On Tue, Jul 28, 2009 at 10:58:55PM -0700, Joe Auricchio wrote: > > On 25 Jul, 2009, at 09:21 , Robert Millan wrote: >> On Thu, Jul 23, 2009 at 04:05:00PM -0700, Joe Auricchio wrote: >>> >>> >>> How's this? >> >> The ChangeLog entry is incomplete. It should be e.g.: >> >> * commands/minicmd.c

Re: [PATCH 2/2] Alias cls as clear

2009-07-28 Thread Joe Auricchio
On 25 Jul, 2009, at 09:21 , Robert Millan wrote: On Thu, Jul 23, 2009 at 04:05:00PM -0700, Joe Auricchio wrote: How's this? The ChangeLog entry is incomplete. It should be e.g.: * commands/minicmd.c (grub_mini_cmd_clear): New function. etc. See other entries as an example. Upd

Re: [PATCH 2/2] Alias cls as clear

2009-07-25 Thread Robert Millan
On Thu, Jul 23, 2009 at 04:05:00PM -0700, Joe Auricchio wrote: > > On 23 Jul, 2009, at 15:42 , Pavel Roskin wrote: >> On Wed, 2009-07-22 at 22:08 -0700, Joe Auricchio wrote: >>> * commands/minicmd.c: Add clear as alias for cls. >>> >>> My background is in Unix, so 'clear' comes much more naturally

Re: [PATCH 2/2] Alias cls as clear

2009-07-25 Thread Robert Millan
On Thu, Jul 23, 2009 at 06:42:11PM -0400, Pavel Roskin wrote: > On Wed, 2009-07-22 at 22:08 -0700, Joe Auricchio wrote: > > * commands/minicmd.c: Add clear as alias for cls. > > > > My background is in Unix, so 'clear' comes much more naturally to my > > fingers than 'cls'. But it's bad to clutter

Re: [PATCH 2/2] Alias cls as clear

2009-07-23 Thread Joe Auricchio
On 23 Jul, 2009, at 15:42 , Pavel Roskin wrote: On Wed, 2009-07-22 at 22:08 -0700, Joe Auricchio wrote: * commands/minicmd.c: Add clear as alias for cls. My background is in Unix, so 'clear' comes much more naturally to my fingers than 'cls'. But it's bad to clutter grub with too many comman

Re: [PATCH 2/2] Alias cls as clear

2009-07-23 Thread Pavel Roskin
On Wed, 2009-07-22 at 22:08 -0700, Joe Auricchio wrote: > * commands/minicmd.c: Add clear as alias for cls. > > My background is in Unix, so 'clear' comes much more naturally to my > fingers than 'cls'. But it's bad to clutter grub with too many commands. > So I leave this to the maintainers: do w

[PATCH 2/2] Alias cls as clear

2009-07-22 Thread Joe Auricchio
* commands/minicmd.c: Add clear as alias for cls. My background is in Unix, so 'clear' comes much more naturally to my fingers than 'cls'. But it's bad to clutter grub with too many commands. So I leave this to the maintainers: do we add both commands, only one, or neither? -joe --- a/comma