Changeset e2ea8d7e in mainline for uspace/app/bdsh/cmds/modules

Timestamp:

2008-08-27T05:36:12Z (17 years ago)

Author:

Tim Post <echo@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

1b4b7b6

Parents:

b510d52

Message:

Housekeeping list, complete lingering things before they get forgotten:

• cli_*() now sets a global cli_errno, error functions cleaned up
• Finish internal cli_*() functions in util.c
• Don't expose cli_init() or cli_finit()
• Get rid of unused globals
• Don't set globals in commands themselves
• Update README files
• Fix stale comments

Location:

uspace/app/bdsh/cmds/modules

Files:

• README (modified) (1 diff)
• help/help.c (modified) (3 diffs)
• quit/quit.c (modified) (1 diff)

uspace/app/bdsh/cmds/modules/README

@@ -1 @@
-Modules are commands or full programs (anything can be made into a module that can return
-int type) should go here. Note, modules do not update the structures containing user info
-such as the working directory, euid, etc.
+Modules are commands or full programs (anything can be made into a module
+that can return int type) should go here. Note, modules do not (can not)
+update or read cliuser_t.
 
-Stuff that needs to write to the user structures contained in scli.h should be made as
-built-in commands, not modules.
+Stuff that needs to write to the user structures contained in scli.h should 
+be made as built-in commands, not modules, but there are very few times when
+you would want to do that.
+
+See the README file in the bdsh root directory for a quick overview of how to
+write a new command, or convert an existig stand-alone program into a module
+for BDSH.
 
 
+
+

uspace/app/bdsh/cmds/modules/help/help.c

@@ -41 +41 @@
 static char *cmdname = "help";
 extern const char *progname;
-extern unsigned int cli_interactive;
 
 #define HELP_IS_MODULE   1
@@ -127 +126 @@
         }
 
-        printf("%sAvailable commands are:\n", cli_interactive ? "\n  " : "");
-        if (cli_interactive)
-                printf(
-                        "  ------------------------------------------------------------\n");
+        printf("\n  Available commands are:\n");
+        printf("  ------------------------------------------------------------\n");
 
         /* First, show a list of built in commands that are available in this mode */
@@ -156 +153 @@
         }
 
-        /* Provide  a little more information and inform them of history / line
-         * editing features if they are present */
-        if (cli_interactive)
-                printf("\n  Try %s %s for more information on how `%s' works.\n\n",
-                        cmdname, cmdname, cmdname);
+        printf("\n  Try %s %s for more information on how `%s' works.\n\n",
+                cmdname, cmdname, cmdname);
 
         return CMD_SUCCESS;

uspace/app/bdsh/cmds/modules/quit/quit.c

@@ -36 +36 @@
 
 static char *cmdname = "quit";
-unsigned int cli_quit = 0;
 
-extern volatile int cli_lasterror;
+extern volatile unsigned int cli_quit;
 extern const char *progname;