Re: [PATCH] add clear screen command

[Carlos Roberto do Nascimento Costa]

Hi Felix, thank you for reviewing this patch.

Felix Zielcke wrote:
> Hello,
> 
> + *  Copyright (C) 2003,2007  Free Software Foundation, Inc.
> + *  Copyright (C) 2003  NIIBE Yutaka <address@hidden>
> 
> This should be just `Copyright (C) 2009  Free Software Foundation, Inc.',
> because it's a new file from this year.

My bad, I'm sorry. Now is fixed, see new [clear2.patch] file attached.

> +  (void)mod;                   /* To stop warning. */
> +  cmd = grub_register_command ("clear", grub_cmd_clear,
> +               0, "Clear the terminal screen.");
> 
> I think there shouldn't be a `0' but `GRUB_COMMAND_FLAG_CMDLINE'.

Yes, you are right. grub_register_command isn't the correct function to register
a command and grub_register_extcmd is the new way. However, there are still
modules need to be fixed, right?! I'll write another e-mail about it soon.

> And please write a ChangeLog entry.
> 

My bad. Now fixed too.

Best Regards,

-- 
Carlos Roberto do Nascimento Costa
E-mails: address@hidden
IBM - Linux Technology Center Brasil

Index: ChangeLog
===================================================================
--- ChangeLog   (revision 2073)
+++ ChangeLog   (working copy)
@@ -1,3 +1,11 @@
+2009-04-08  Carlos Costa  <address@hidden>
+
+       * commands/clear.c: new file.
+       * conf/common.rmk (pkglib_MODULES):     Added clear.mod.
+       (clear_mod_SOURCES): new variable.
+       (clear_mod_CFLAGS): likewise.
+       (clear_mod_LDFLAGS): likewise.
+
 2009-04-08  Felix Zielcke  <address@hidden>
 
        * disk/lvm.c (grub_lvm_scan_device): Add a missing NULL check.
Index: commands/clear.c
===================================================================
--- commands/clear.c    (revision 0)
+++ commands/clear.c    (revision 0)
@@ -0,0 +1,50 @@
+/* clear.c - command to clear the terminal screen. */
+/*
+ *  GRUB  --  GRand Unified Bootloader
+ *  Copyright (C) 2009  Free Software Foundation, Inc.
+ *
+ *  GRUB is free software: you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation, either version 3 of the License, or
+ *  (at your option) any later version.
+ *
+ *  GRUB is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *
+ *  You should have received a copy of the GNU General Public License
+ *  along with GRUB.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <grub/types.h>
+#include <grub/misc.h>
+#include <grub/err.h>
+#include <grub/dl.h>
+#include <grub/term.h>
+#include <grub/command.h>
+
+
+static grub_err_t
+grub_cmd_clear (struct grub_extcmd *cmd __attribute__ ((unused)),
+               int argc __attribute__ ((unused)),
+               char **args __attribute__ ((unused)))
+{
+  grub_cls ();
+  return 0;
+}
+
+static grub_command_t cmd;
+
+GRUB_MOD_INIT(clear)
+{
+  (void)mod;                   /* To stop warning. */
+  cmd = grub_register_extcmd ("clear", grub_cmd_clear,
+            GRUB_COMMAND_FLAG_CMDLINE,"clear",
+            "Clear the terminal screen.",0);
+}
+
+GRUB_MOD_FINI(clear)
+{
+  grub_unregister_extcmd (cmd);
+}
Index: conf/common.rmk
===================================================================
--- conf/common.rmk     (revision 2073)
+++ conf/common.rmk     (working copy)
@@ -332,7 +332,7 @@
 
 # Commands.
 pkglib_MODULES += minicmd.mod extcmd.mod hello.mod handler.mod \
-        ls.mod cmp.mod cat.mod help.mod search.mod             \
+        ls.mod cmp.mod cat.mod help.mod search.mod clear.mod  \
        loopback.mod fs_uuid.mod configfile.mod echo.mod        \
        terminfo.mod test.mod blocklist.mod hexdump.mod         \
        read.mod sleep.mod loadenv.mod crc.mod parttool.mod pcpart.mod
@@ -382,6 +382,11 @@
 cat_mod_CFLAGS = $(COMMON_CFLAGS)
 cat_mod_LDFLAGS = $(COMMON_LDFLAGS)
 
+# For clear.mod
+clear_mod_SOURCES = commands/clear.c
+clear_mod_CFLAGS = $(COMMON_CFLAGS)
+clear_mod_LDFLAGS = $(COMMON_LDFLAGS)
+
 # For echo.mod
 echo_mod_SOURCES = commands/echo.c
 echo_mod_CFLAGS = $(COMMON_CFLAGS)