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

[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 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?
>>
>> Let's have "clear" only, as it's more clear :-)
>>
>> Please include the ChangeLog entry.
>
>
> 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.

-- 
Robert Millan

  The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
  how) you may access your data; but nobody's threatening your freedom: we
  still allow you to remove your data and not access it at all."