Small size optimization from Aaron Lehmann
[oweals/busybox.git] / lash.c
diff --git a/lash.c b/lash.c
index bb8a456b740796dc3b1163e7f67dce21bfdae73b..0af669116006455ccd0347773f4397f1f85d86a2 100644 (file)
--- a/lash.c
+++ b/lash.c
@@ -2,7 +2,7 @@
 /*
  * lash -- the BusyBox Lame-Ass SHell
  *
- * Copyright (C) 2000 by Lineo, inc.
+ * Copyright (C) 1999,2000,2001 by Lineo, inc.
  * Written by Erik Andersen <andersen@lineo.com>, <andersee@debian.org>
  *
  * Based in part on ladsh.c by Michael K. Johnson and Erik W. Troan, which is
  *
  */
 
-//
-//This works pretty well now, and is not on by default.
-#define BB_FEATURE_SH_ENVIRONMENT
-//
-//Backtick support has some problems, use at your own risk!
-//#define BB_FEATURE_SH_BACKTICKS
-//
-//If, then, else, etc. support is really, really broken.  Don't even
-//bother to mess with this yet, since you will not be happy with it.
-//#define BB_FEATURE_SH_IF_EXPRESSIONS
-//
+/* This shell's parsing engine is officially at a dead-end.
+ * Future work shell work should be done using hush.c
+ */
+
 //For debugging/development on the shell only...
 //#define DEBUG_SHELL
 
 
-#include "busybox.h"
 #include <stdio.h>
 #include <stdlib.h>
 #include <ctype.h>
 #include <errno.h>
 #include <fcntl.h>
-#include <glob.h>
 #include <signal.h>
 #include <string.h>
 #include <sys/ioctl.h>
 #include <sys/wait.h>
 #include <unistd.h>
 #include <getopt.h>
+#include <termios.h>
+#include "busybox.h"
 #include "cmdedit.h"
 
-#define MAX_LINE       256     /* size of input buffer for `read' builtin */
-#define MAX_READ       128     /* size of input buffer for `read' builtin */
-#define JOB_STATUS_FORMAT "[%d] %-22s %.40s\n"
-extern size_t NUM_APPLETS;
+#ifdef BB_LOCALE_SUPPORT
+#include <locale.h>
+#endif
+
+#include <glob.h>
+#define expand_t       glob_t
 
 
+static const int MAX_READ = 128;       /* size of input buffer for `read' builtin */
+#define JOB_STATUS_FORMAT "[%d] %-22s %.40s\n"
 
 
-enum redirectionType { REDIRECT_INPUT, REDIRECT_OVERWRITE,
+enum redir_type { REDIRECT_INPUT, REDIRECT_OVERWRITE,
        REDIRECT_APPEND
 };
 
-static const unsigned int REGULAR_JOB_CONTEXT=0x1;
+static const unsigned int DEFAULT_CONTEXT=0x1;
 static const unsigned int IF_TRUE_CONTEXT=0x2;
 static const unsigned int IF_FALSE_CONTEXT=0x4;
 static const unsigned int THEN_EXP_CONTEXT=0x8;
 static const unsigned int ELSE_EXP_CONTEXT=0x10;
 
 
-struct jobSet {
+struct jobset {
        struct job *head;                       /* head of list of running jobs */
        struct job *fg;                         /* current foreground job */
 };
 
-struct redirectionSpecifier {
-       enum redirectionType type;      /* type of redirection */
+struct redir_struct {
+       enum redir_type type;   /* type of redirection */
        int fd;                                         /* file descriptor being redirected */
        char *filename;                         /* file to redirect fd to */
 };
 
-struct childProgram {
+struct child_prog {
        pid_t pid;                                      /* 0 if exited */
        char **argv;                            /* program name and arguments */
-       int numRedirections;            /* elements in redirection array */
-       struct redirectionSpecifier *redirections;      /* I/O redirections */
-       glob_t globResult;                      /* result of parameter globbing */
-       int freeGlob;                           /* should we globfree(&globResult)? */
-       int isStopped;                          /* is the program currently running? */
+       int num_redirects;                      /* elements in redirection array */
+       struct redir_struct *redirects; /* I/O redirects */
+       int is_stopped;                         /* is the program currently running? */
+       struct job *family;                     /* pointer back to the child's parent job */
 };
 
 struct job {
-       int jobId;                                      /* job number */
-       int numProgs;                           /* total number of programs in job */
-       int runningProgs;                       /* number of programs running */
+       int jobid;                                      /* job number */
+       int num_progs;                          /* total number of programs in job */
+       int running_progs;                      /* number of programs running */
        char *text;                                     /* name of job */
-       char *cmdBuf;                           /* buffer various argv's point into */
+       char *cmdbuf;                           /* buffer various argv's point into */
        pid_t pgrp;                                     /* process group ID for the job */
-       struct childProgram *progs;     /* array of programs in job */
+       struct child_prog *progs;       /* array of programs in job */
        struct job *next;                       /* to track background commands */
-       int stoppedProgs;                       /* number of programs alive, but stopped */
-       int jobContext;                         /* bitmask defining current context */
+       int stopped_progs;                      /* number of programs alive, but stopped */
+       unsigned int job_context;       /* bitmask defining current context */
+       struct jobset *job_list;
 };
 
-struct builtInCommand {
+struct built_in_command {
        char *cmd;                                      /* name */
        char *descr;                            /* description */
-       int (*function) (struct job *, struct jobSet * jobList);        /* function ptr */
+       int (*function) (struct child_prog *);  /* function ptr */
+};
+
+struct close_me {
+       int fd;
+       struct close_me *next;
 };
 
 /* function prototypes for builtins */
-static int builtin_cd(struct job *cmd, struct jobSet *junk);
-static int builtin_env(struct job *dummy, struct jobSet *junk);
-static int builtin_exec(struct job *cmd, struct jobSet *junk);
-static int builtin_exit(struct job *cmd, struct jobSet *junk);
-static int builtin_fg_bg(struct job *cmd, struct jobSet *jobList);
-static int builtin_help(struct job *cmd, struct jobSet *junk);
-static int builtin_jobs(struct job *dummy, struct jobSet *jobList);
-static int builtin_pwd(struct job *dummy, struct jobSet *junk);
-static int builtin_export(struct job *cmd, struct jobSet *junk);
-static int builtin_source(struct job *cmd, struct jobSet *jobList);
-static int builtin_unset(struct job *cmd, struct jobSet *junk);
-static int builtin_read(struct job *cmd, struct jobSet *junk);
-#ifdef BB_FEATURE_SH_IF_EXPRESSIONS
-static int builtin_if(struct job *cmd, struct jobSet *junk);
-static int builtin_then(struct job *cmd, struct jobSet *junk);
-static int builtin_else(struct job *cmd, struct jobSet *junk);
-static int builtin_fi(struct job *cmd, struct jobSet *junk);
-#endif
+static int builtin_cd(struct child_prog *cmd);
+static int builtin_exec(struct child_prog *cmd);
+static int builtin_exit(struct child_prog *cmd);
+static int builtin_fg_bg(struct child_prog *cmd);
+static int builtin_help(struct child_prog *cmd);
+static int builtin_jobs(struct child_prog *dummy);
+static int builtin_pwd(struct child_prog *dummy);
+static int builtin_export(struct child_prog *cmd);
+static int builtin_source(struct child_prog *cmd);
+static int builtin_unset(struct child_prog *cmd);
+static int builtin_read(struct child_prog *cmd);
 
 
 /* function prototypes for shell stuff */
-static void checkJobs(struct jobSet *jobList);
-static int getCommand(FILE * source, char *command);
-static int parseCommand(char **commandPtr, struct job *job, struct jobSet *jobList, int *inBg);
-static int runCommand(struct job *newJob, struct jobSet *jobList, int inBg, int outPipe[2]);
+static void mark_open(int fd);
+static void mark_closed(int fd);
+static void close_all(void);
+static void checkjobs(struct jobset *job_list);
+static void remove_job(struct jobset *j_list, struct job *job);
+static int get_command(FILE * source, char *command);
+static int parse_command(char **command_ptr, struct job *job, int *inbg);
+static int run_command(struct job *newjob, int inbg, int outpipe[2]);
+static int pseudo_exec(struct child_prog *cmd) __attribute__ ((noreturn));
 static int busy_loop(FILE * input);
 
 
 /* Table of built-in functions (these are non-forking builtins, meaning they
  * can change global variables in the parent shell process but they will not
  * work with pipes and redirects; 'unset foo | whatever' will not work) */
-static struct builtInCommand bltins[] = {
+static struct built_in_command bltins[] = {
        {"bg", "Resume a job in the background", builtin_fg_bg},
        {"cd", "Change working directory", builtin_cd},
        {"exec", "Exec command, replacing this shell with the exec'd process", builtin_exec},
@@ -157,236 +157,264 @@ static struct builtInCommand bltins[] = {
        {"unset", "Unset environment variable", builtin_unset},
        {"read", "Input environment variable", builtin_read},
        {".", "Source-in and run commands in a file", builtin_source},
-#ifdef BB_FEATURE_SH_IF_EXPRESSIONS
-       {"if", NULL, builtin_if},
-       {"then", NULL, builtin_then},
-       {"else", NULL, builtin_else},
-       {"fi", NULL, builtin_fi},
-#endif
+       /* to do: add ulimit */
        {NULL, NULL, NULL}
 };
 
 /* Table of forking built-in functions (things that fork cannot change global
  * variables in the parent process, such as the current working directory) */
-static struct builtInCommand bltins_forking[] = {
-       {"env", "Print all environment variables", builtin_env},
+static struct built_in_command bltins_forking[] = {
        {"pwd", "Print current directory", builtin_pwd},
        {"help", "List shell built-in commands", builtin_help},
        {NULL, NULL, NULL}
 };
 
-static char prompt[3];
-static char *cwd;
+
+static int shell_context;  /* Type prompt trigger (PS1 or PS2) */
+
+
+/* Globals that are static to this file */
+static const char *cwd;
 static char *local_pending_command = NULL;
-static char *promptStr = NULL;
-static struct jobSet jobList = { NULL, NULL };
+static struct jobset job_list = { NULL, NULL };
 static int argc;
 static char **argv;
-#ifdef BB_FEATURE_SH_ENVIRONMENT
-static int lastBgPid=-1;
-static int lastReturnCode=-1;
-static int showXtrace=FALSE;
-#endif
-       
+static struct close_me *close_me_head;
+static int last_return_code;
+static int last_bg_pid;
+static unsigned int last_jobid;
+static int shell_terminal;
+static pid_t shell_pgrp;
+static char *PS1;
+static char *PS2 = "> ";
+
+
 #ifdef DEBUG_SHELL
 static inline void debug_printf(const char *format, ...)
 {
        va_list args;
        va_start(args, format);
-       vfprintf(stderr, s, p);
+       vfprintf(stderr, format, args);
        va_end(args);
 }
 #else
 static inline void debug_printf(const char *format, ...) { }
 #endif
 
-#ifdef BB_FEATURE_SH_COMMAND_EDITING
-static inline void win_changed(int junk)
-{
-       struct winsize win = { 0, 0, 0, 0 };
-       ioctl(0, TIOCGWINSZ, &win);
-       if (win.ws_col > 0) {
-               cmdedit_setwidth( win.ws_col - 1);
-       }
-}
-#else
-static inline void win_changed(int junk) {}
-#endif
-
+/*
+       Most builtins need access to the struct child_prog that has
+       their arguments, previously coded as cmd->progs[0].  That coding
+       can exhibit a bug, if the builtin is not the first command in
+       a pipeline: "echo foo | exec sort" will attempt to exec foo.
+
+builtin   previous use      notes
+------ -----------------  ---------
+cd      cmd->progs[0]
+exec    cmd->progs[0]  squashed bug: didn't look for applets or forking builtins
+exit    cmd->progs[0]
+fg_bg   cmd->progs[0], job_list->head, job_list->fg
+help    0
+jobs    job_list->head
+pwd     0
+export  cmd->progs[0]
+source  cmd->progs[0]
+unset   cmd->progs[0]
+read    cmd->progs[0]
+
+I added "struct job *family;" to struct child_prog,
+and switched API to builtin_foo(struct child_prog *child);
+So   cmd->text        becomes  child->family->text
+     cmd->job_context  becomes  child->family->job_context
+     cmd->progs[0]    becomes  *child
+     job_list          becomes  child->family->job_list
+ */
 
 /* built-in 'cd <path>' handler */
-static int builtin_cd(struct job *cmd, struct jobSet *junk)
+static int builtin_cd(struct child_prog *child)
 {
        char *newdir;
 
-       if (!cmd->progs[0].argv[1] == 1)
+       if (child->argv[1] == NULL)
                newdir = getenv("HOME");
        else
-               newdir = cmd->progs[0].argv[1];
+               newdir = child->argv[1];
        if (chdir(newdir)) {
-               printf("cd: %s: %s\n", newdir, strerror(errno));
+               printf("cd: %s: %m\n", newdir);
                return EXIT_FAILURE;
        }
-       getcwd(cwd, sizeof(char)*MAX_LINE);
-
+       cwd = xgetcwd((char *)cwd);
+       if (!cwd)
+               cwd = unknown;
        return EXIT_SUCCESS;
 }
 
-/* built-in 'env' handler */
-static int builtin_env(struct job *dummy, struct jobSet *junk)
-{
-       char **e;
-
-       for (e = environ; *e; e++) {
-               fprintf(stdout, "%s\n", *e);
-       }
-       return (0);
-}
-
 /* built-in 'exec' handler */
-static int builtin_exec(struct job *cmd, struct jobSet *junk)
+static int builtin_exec(struct child_prog *child)
 {
-       if (cmd->progs[0].argv[1])
-       {
-               cmd->progs[0].argv++;
-               execvp(cmd->progs[0].argv[0], cmd->progs[0].argv);
-               fatalError("Exec to %s failed: %s\n", cmd->progs[0].argv[0],
-                               strerror(errno));
-       }
-       return EXIT_SUCCESS;
+       if (child->argv[1] == NULL)
+               return EXIT_SUCCESS;   /* Really? */
+       child->argv++;
+       close_all();
+       pseudo_exec(child);
+       /* never returns */
 }
 
 /* built-in 'exit' handler */
-static int builtin_exit(struct job *cmd, struct jobSet *junk)
+static int builtin_exit(struct child_prog *child)
 {
-       if (!cmd->progs[0].argv[1] == 1)
+       if (child->argv[1] == NULL)
                exit(EXIT_SUCCESS);
 
-       exit (atoi(cmd->progs[0].argv[1]));
+       exit (atoi(child->argv[1]));
 }
 
 /* built-in 'fg' and 'bg' handler */
-static int builtin_fg_bg(struct job *cmd, struct jobSet *jobList)
+static int builtin_fg_bg(struct child_prog *child)
 {
-       int i, jobNum;
+       int i, jobnum;
        struct job *job=NULL;
 
-       if (!jobList->head) {
-               if (!cmd->progs[0].argv[1] || cmd->progs[0].argv[2]) {
-                       errorMsg("%s: exactly one argument is expected\n",
-                                       cmd->progs[0].argv[0]);
+       /* If they gave us no args, assume they want the last backgrounded task */
+       if (!child->argv[1]) {
+               for (job = child->family->job_list->head; job; job = job->next) {
+                       if (job->jobid == last_jobid) {
+                               break;
+                       }
+               }
+               if (!job) {
+                       error_msg("%s: no current job", child->argv[0]);
                        return EXIT_FAILURE;
                }
-               if (sscanf(cmd->progs[0].argv[1], "%%%d", &jobNum) != 1) {
-                       errorMsg("%s: bad argument '%s'\n",
-                                       cmd->progs[0].argv[0], cmd->progs[0].argv[1]);
+       } else {
+               if (sscanf(child->argv[1], "%%%d", &jobnum) != 1) {
+                       error_msg("%s: bad argument '%s'", child->argv[0], child->argv[1]);
                        return EXIT_FAILURE;
-                       for (job = jobList->head; job; job = job->next) {
-                               if (job->jobId == jobNum) {
-                                       break;
-                               }
+               }
+               for (job = child->family->job_list->head; job; job = job->next) {
+                       if (job->jobid == jobnum) {
+                               break;
                        }
                }
-       } else {
-               job = jobList->head;
+               if (!job) {
+                       error_msg("%s: %d: no such job", child->argv[0], jobnum);
+                       return EXIT_FAILURE;
+               }
        }
 
-       if (!job) {
-               errorMsg("%s: unknown job %d\n",
-                               cmd->progs[0].argv[0], jobNum);
-               return EXIT_FAILURE;
-       }
+       if (*child->argv[0] == 'f') {
+               /* Put the job into the foreground.  */
+               tcsetpgrp(shell_terminal, job->pgrp);
 
-       if (*cmd->progs[0].argv[0] == 'f') {
-               /* Make this job the foreground job */
-               /* suppress messages when run from /linuxrc mag@sysgo.de */
-               if (tcsetpgrp(0, job->pgrp) && errno != ENOTTY)
-                       perror("tcsetpgrp"); 
-               jobList->fg = job;
+               child->family->job_list->fg = job;
        }
 
        /* Restart the processes in the job */
-       for (i = 0; i < job->numProgs; i++)
-               job->progs[i].isStopped = 0;
+       for (i = 0; i < job->num_progs; i++)
+               job->progs[i].is_stopped = 0;
 
-       kill(-job->pgrp, SIGCONT);
+       job->stopped_progs = 0;
 
-       job->stoppedProgs = 0;
+       if ( (i=kill(- job->pgrp, SIGCONT)) < 0) {
+               if (i == ESRCH) {
+                       remove_job(&job_list, job);
+               } else {
+                       perror_msg("kill (SIGCONT)");
+               }
+       }
 
        return EXIT_SUCCESS;
 }
 
 /* built-in 'help' handler */
-static int builtin_help(struct job *dummy, struct jobSet *junk)
+static int builtin_help(struct child_prog *dummy)
 {
-       struct builtInCommand *x;
+       struct built_in_command *x;
 
-       fprintf(stdout, "\nBuilt-in commands:\n");
-       fprintf(stdout, "-------------------\n");
+       printf("\nBuilt-in commands:\n");
+       printf("-------------------\n");
        for (x = bltins; x->cmd; x++) {
                if (x->descr==NULL)
                        continue;
-               fprintf(stdout, "%s\t%s\n", x->cmd, x->descr);
+               printf("%s\t%s\n", x->cmd, x->descr);
        }
        for (x = bltins_forking; x->cmd; x++) {
                if (x->descr==NULL)
                        continue;
-               fprintf(stdout, "%s\t%s\n", x->cmd, x->descr);
+               printf("%s\t%s\n", x->cmd, x->descr);
        }
-       fprintf(stdout, "\n\n");
+       printf("\n\n");
        return EXIT_SUCCESS;
 }
 
 /* built-in 'jobs' handler */
-static int builtin_jobs(struct job *dummy, struct jobSet *jobList)
+static int builtin_jobs(struct child_prog *child)
 {
        struct job *job;
-       char *statusString;
+       char *status_string;
 
-       for (job = jobList->head; job; job = job->next) {
-               if (job->runningProgs == job->stoppedProgs)
-                       statusString = "Stopped";
+       for (job = child->family->job_list->head; job; job = job->next) {
+               if (job->running_progs == job->stopped_progs)
+                       status_string = "Stopped";
                else
-                       statusString = "Running";
+                       status_string = "Running";
 
-               printf(JOB_STATUS_FORMAT, job->jobId, statusString, job->text);
+               printf(JOB_STATUS_FORMAT, job->jobid, status_string, job->text);
        }
        return EXIT_SUCCESS;
 }
 
 
 /* built-in 'pwd' handler */
-static int builtin_pwd(struct job *dummy, struct jobSet *junk)
+static int builtin_pwd(struct child_prog *dummy)
 {
-       getcwd(cwd, sizeof(char)*MAX_LINE);
-       fprintf(stdout, "%s\n", cwd);
+       cwd = xgetcwd((char *)cwd);
+       if (!cwd)
+               cwd = unknown;
+       puts(cwd);
        return EXIT_SUCCESS;
 }
 
 /* built-in 'export VAR=value' handler */
-static int builtin_export(struct job *cmd, struct jobSet *junk)
+static int builtin_export(struct child_prog *child)
 {
        int res;
+       char *v = child->argv[1];
 
-       if (!cmd->progs[0].argv[1] == 1) {
-               return (builtin_env(cmd, junk));
+       if (v == NULL) {
+               char **e;
+               for (e = environ; *e; e++) {
+                       puts(*e);
+               }
+               return 0;
        }
-       res = putenv(cmd->progs[0].argv[1]);
+       res = putenv(v);
        if (res)
-               fprintf(stdout, "export: %s\n", strerror(errno));
+               fprintf(stderr, "export: %m\n");
+#ifdef BB_FEATURE_SH_FANCY_PROMPT
+       if (strncmp(v, "PS1=", 4)==0)
+               PS1 = getenv("PS1");
+#endif
+
+#ifdef BB_LOCALE_SUPPORT
+       if(strncmp(v, "LC_ALL=", 7)==0)
+               setlocale(LC_ALL, getenv("LC_ALL"));
+       if(strncmp(v, "LC_CTYPE=", 9)==0)
+               setlocale(LC_CTYPE, getenv("LC_CTYPE"));
+#endif
+
        return (res);
 }
 
 /* built-in 'read VAR' handler */
-static int builtin_read(struct job *cmd, struct jobSet *junk)
+static int builtin_read(struct child_prog *child)
 {
        int res = 0, len, newlen;
        char *s;
        char string[MAX_READ];
 
-       if (cmd->progs[0].argv[1]) {
+       if (child->argv[1]) {
                /* argument (VAR) given: put "VAR=" into buffer */
-               strcpy(string, cmd->progs[0].argv[1]);
+               strcpy(string, child->argv[1]);
                len = strlen(string);
                string[len++] = '=';
                string[len]   = '\0';
@@ -403,7 +431,7 @@ static int builtin_read(struct job *cmd, struct jobSet *junk)
                if((s = strdup(string)))
                        res = putenv(s);
                if (res)
-                       fprintf(stdout, "read: %s\n", strerror(errno));
+                       fprintf(stderr, "read: %m\n");
        }
        else
                fgets(string, sizeof(string), stdin);
@@ -411,185 +439,132 @@ static int builtin_read(struct job *cmd, struct jobSet *junk)
        return (res);
 }
 
-#ifdef BB_FEATURE_SH_IF_EXPRESSIONS
-/* Built-in handler for 'if' commands */
-static int builtin_if(struct job *cmd, struct jobSet *jobList)
-{
-       int status;
-       char* charptr1=cmd->text+3; /* skip over the leading 'if ' */
-
-       /* Now run the 'if' command */
-       status=strlen(charptr1);
-       local_pending_command = xmalloc(status+1);
-       strncpy(local_pending_command, charptr1, status); 
-       local_pending_command[status]='\0';
-       debug_printf(stderr, "'if' now testing '%s'\n", local_pending_command);
-       status = busy_loop(NULL); /* Frees local_pending_command */
-       debug_printf(stderr, "if test returned ");
-       if (status == 0) {
-               debug_printf(stderr, "TRUE\n");
-               cmd->jobContext |= IF_TRUE_CONTEXT;
-       } else {
-               debug_printf(stderr, "FALSE\n");
-               cmd->jobContext |= IF_FALSE_CONTEXT;
-       }
-
-       return status;
-}
-
-/* Built-in handler for 'then' (part of the 'if' command) */
-static int builtin_then(struct job *cmd, struct jobSet *junk)
-{
-       int status;
-       char* charptr1=cmd->text+5; /* skip over the leading 'then ' */
-
-       if (! (cmd->jobContext & (IF_TRUE_CONTEXT|IF_FALSE_CONTEXT))) {
-               errorMsg("unexpected token `then'\n");
-               return EXIT_FAILURE;
-       }
-       /* If the if result was FALSE, skip the 'then' stuff */
-       if (cmd->jobContext & IF_FALSE_CONTEXT) {
-               return EXIT_SUCCESS;
-       }
-
-       cmd->jobContext |= THEN_EXP_CONTEXT;
-       //printf("Hit an then -- jobContext=%d\n", cmd->jobContext);
-
-       /* Now run the 'then' command */
-       status=strlen(charptr1);
-       local_pending_command = xmalloc(status+1);
-       strncpy(local_pending_command, charptr1, status); 
-       local_pending_command[status]='\0';
-       debug_printf(stderr, "'then' now running '%s'\n", charptr1);
-       return( busy_loop(NULL));
-}
-
-/* Built-in handler for 'else' (part of the 'if' command) */
-static int builtin_else(struct job *cmd, struct jobSet *junk)
-{
-       int status;
-       char* charptr1=cmd->text+5; /* skip over the leading 'else ' */
-
-       if (! (cmd->jobContext & (IF_TRUE_CONTEXT|IF_FALSE_CONTEXT))) {
-               errorMsg("unexpected token `else'\n");
-               return EXIT_FAILURE;
-       }
-       /* If the if result was TRUE, skip the 'else' stuff */
-       if (cmd->jobContext & IF_TRUE_CONTEXT) {
-               return EXIT_SUCCESS;
-       }
-
-       cmd->jobContext |= ELSE_EXP_CONTEXT;
-       //printf("Hit an else -- jobContext=%d\n", cmd->jobContext);
-
-       /* Now run the 'else' command */
-       status=strlen(charptr1);
-       local_pending_command = xmalloc(status+1);
-       strncpy(local_pending_command, charptr1, status); 
-       local_pending_command[status]='\0';
-       debug_printf(stderr, "'else' now running '%s'\n", charptr1);
-       return( busy_loop(NULL));
-}
-
-/* Built-in handler for 'fi' (part of the 'if' command) */
-static int builtin_fi(struct job *cmd, struct jobSet *junk)
-{
-       if (! (cmd->jobContext & (IF_TRUE_CONTEXT|IF_FALSE_CONTEXT))) {
-               errorMsg("unexpected token `fi'\n");
-               return EXIT_FAILURE;
-       }
-       /* Clear out the if and then context bits */
-       cmd->jobContext &= ~(IF_TRUE_CONTEXT|IF_FALSE_CONTEXT|THEN_EXP_CONTEXT|ELSE_EXP_CONTEXT);
-       debug_printf(stderr, "Hit an fi   -- jobContext=%d\n", cmd->jobContext);
-       return EXIT_SUCCESS;
-}
-#endif
-
 /* Built-in '.' handler (read-in and execute commands from file) */
-static int builtin_source(struct job *cmd, struct jobSet *junk)
+static int builtin_source(struct child_prog *child)
 {
        FILE *input;
        int status;
+       int fd;
 
-       if (!cmd->progs[0].argv[1] == 1)
+       if (child->argv[1] == NULL)
                return EXIT_FAILURE;
 
-       input = fopen(cmd->progs[0].argv[1], "r");
+       input = fopen(child->argv[1], "r");
        if (!input) {
-               fprintf(stdout, "Couldn't open file '%s'\n",
-                               cmd->progs[0].argv[1]);
+               printf( "Couldn't open file '%s'\n", child->argv[1]);
                return EXIT_FAILURE;
        }
 
+       fd=fileno(input);
+       mark_open(fd);
        /* Now run the file */
        status = busy_loop(input);
        fclose(input);
+       mark_closed(fd);
        return (status);
 }
 
 /* built-in 'unset VAR' handler */
-static int builtin_unset(struct job *cmd, struct jobSet *junk)
+static int builtin_unset(struct child_prog *child)
 {
-       if (!cmd->progs[0].argv[1] == 1) {
-               fprintf(stdout, "unset: parameter required.\n");
+       if (child->argv[1] == NULL) {
+               printf( "unset: parameter required.\n");
                return EXIT_FAILURE;
        }
-       unsetenv(cmd->progs[0].argv[1]);
+       unsetenv(child->argv[1]);
        return EXIT_SUCCESS;
 }
 
+static void mark_open(int fd)
+{
+       struct close_me *new = xmalloc(sizeof(struct close_me));
+       new->fd = fd;
+       new->next = close_me_head;
+       close_me_head = new;
+}
+
+static void mark_closed(int fd)
+{
+       struct close_me *tmp;
+       if (close_me_head == NULL || close_me_head->fd != fd)
+               error_msg_and_die("corrupt close_me");
+       tmp = close_me_head;
+       close_me_head = close_me_head->next;
+       free(tmp);
+}
+
+static void close_all()
+{
+       struct close_me *c, *tmp;
+       for (c=close_me_head; c; c=tmp) {
+               close(c->fd);
+               tmp=c->next;
+               free(c);
+       }
+       close_me_head = NULL;
+}
+
+
 /* free up all memory from a job */
-static void freeJob(struct job *cmd)
+static void free_job(struct job *cmd)
 {
        int i;
+       struct jobset *keep;
 
-       for (i = 0; i < cmd->numProgs; i++) {
+       for (i = 0; i < cmd->num_progs; i++) {
                free(cmd->progs[i].argv);
-               if (cmd->progs[i].redirections)
-                       free(cmd->progs[i].redirections);
-               if (cmd->progs[i].freeGlob)
-                       globfree(&cmd->progs[i].globResult);
+               if (cmd->progs[i].redirects)
+                       free(cmd->progs[i].redirects);
        }
-       free(cmd->progs);
+       if (cmd->progs)
+               free(cmd->progs);
        if (cmd->text)
                free(cmd->text);
-       free(cmd->cmdBuf);
+       if (cmd->cmdbuf)
+               free(cmd->cmdbuf);
+       keep = cmd->job_list;
        memset(cmd, 0, sizeof(struct job));
+       cmd->job_list = keep;
 }
 
-/* remove a job from the jobList */
-static void removeJob(struct jobSet *jobList, struct job *job)
+/* remove a job from a jobset */
+static void remove_job(struct jobset *j_list, struct job *job)
 {
-       struct job *prevJob;
+       struct job *prevjob;
 
-       freeJob(job);
-       if (job == jobList->head) {
-               jobList->head = job->next;
+       free_job(job);
+       if (job == j_list->head) {
+               j_list->head = job->next;
        } else {
-               prevJob = jobList->head;
-               while (prevJob->next != job)
-                       prevJob = prevJob->next;
-               prevJob->next = job->next;
+               prevjob = j_list->head;
+               while (prevjob->next != job)
+                       prevjob = prevjob->next;
+               prevjob->next = job->next;
        }
 
+       if (j_list->head)
+               last_jobid = j_list->head->jobid;
+       else
+               last_jobid = 0;
+
        free(job);
 }
 
 /* Checks to see if any background processes have exited -- if they 
    have, figure out why and see if a job has completed */
-static void checkJobs(struct jobSet *jobList)
+static void checkjobs(struct jobset *j_list)
 {
        struct job *job;
        pid_t childpid;
        int status;
-       int progNum = 0;
+       int prognum = 0;
 
        while ((childpid = waitpid(-1, &status, WNOHANG | WUNTRACED)) > 0) {
-               for (job = jobList->head; job; job = job->next) {
-                       progNum = 0;
-                       while (progNum < job->numProgs &&
-                                  job->progs[progNum].pid != childpid) progNum++;
-                       if (progNum < job->numProgs)
+               for (job = j_list->head; job; job = job->next) {
+                       prognum = 0;
+                       while (prognum < job->num_progs &&
+                                  job->progs[prognum].pid != childpid) prognum++;
+                       if (prognum < job->num_progs)
                                break;
                }
 
@@ -599,37 +574,45 @@ static void checkJobs(struct jobSet *jobList)
 
                if (WIFEXITED(status) || WIFSIGNALED(status)) {
                        /* child exited */
-                       job->runningProgs--;
-                       job->progs[progNum].pid = 0;
+                       job->running_progs--;
+                       job->progs[prognum].pid = 0;
 
-                       if (!job->runningProgs) {
-                               printf(JOB_STATUS_FORMAT, job->jobId, "Done", job->text);
-                               removeJob(jobList, job);
+                       if (!job->running_progs) {
+                               printf(JOB_STATUS_FORMAT, job->jobid, "Done", job->text);
+                               last_jobid=0;
+                               remove_job(j_list, job);
                        }
                } else {
                        /* child stopped */
-                       job->stoppedProgs++;
-                       job->progs[progNum].isStopped = 1;
-
-                       if (job->stoppedProgs == job->numProgs) {
-                               printf(JOB_STATUS_FORMAT, job->jobId, "Stopped",
+                       job->stopped_progs++;
+                       job->progs[prognum].is_stopped = 1;
+
+#if 0
+                       /* Printing this stuff is a pain, since it tends to
+                        * overwrite the prompt an inconveinient moments.  So
+                        * don't do that.  */
+                       if (job->stopped_progs == job->num_progs) {
+                               printf(JOB_STATUS_FORMAT, job->jobid, "Stopped",
                                           job->text);
                        }
+#endif 
                }
        }
 
        if (childpid == -1 && errno != ECHILD)
-               perror("waitpid");
+               perror_msg("waitpid");
 }
 
-static int setupRedirections(struct childProgram *prog)
+/* squirrel != NULL means we squirrel away copies of stdin, stdout,
+ * and stderr if they are redirected. */
+static int setup_redirects(struct child_prog *prog, int squirrel[])
 {
        int i;
        int openfd;
        int mode = O_RDONLY;
-       struct redirectionSpecifier *redir = prog->redirections;
+       struct redir_struct *redir = prog->redirects;
 
-       for (i = 0; i < prog->numRedirections; i++, redir++) {
+       for (i = 0; i < prog->num_redirects; i++, redir++) {
                switch (redir->type) {
                case REDIRECT_INPUT:
                        mode = O_RDONLY;
@@ -646,12 +629,14 @@ static int setupRedirections(struct childProgram *prog)
                if (openfd < 0) {
                        /* this could get lost if stderr has been redirected, but
                           bash and ash both lose it as well (though zsh doesn't!) */
-                       errorMsg("error opening %s: %s\n", redir->filename,
-                                       strerror(errno));
+                       perror_msg("error opening %s", redir->filename);
                        return 1;
                }
 
                if (openfd != redir->fd) {
+                       if (squirrel && redir->fd < 3) {
+                               squirrel[redir->fd] = dup(redir->fd);
+                       }
                        dup2(openfd, redir->fd);
                        close(openfd);
                }
@@ -660,11 +645,53 @@ static int setupRedirections(struct childProgram *prog)
        return 0;
 }
 
+static void restore_redirects(int squirrel[])
+{
+       int i, fd;
+       for (i=0; i<3; i++) {
+               fd = squirrel[i];
+               if (fd != -1) {
+                       /* No error checking.  I sure wouldn't know what
+                        * to do with an error if I found one! */
+                       dup2(fd, i);
+                       close(fd);
+               }
+       }
+}
 
-static int getCommand(FILE * source, char *command)
+static inline void cmdedit_set_initial_prompt(void)
 {
-       char *user,buf[255],*s;
-       
+#ifndef BB_FEATURE_SH_FANCY_PROMPT
+       PS1 = NULL;
+#else
+       PS1 = getenv("PS1");
+       if(PS1==0)
+               PS1 = "\\w \\$ ";
+#endif 
+}
+
+static inline void setup_prompt_string(char **prompt_str)
+{
+#ifndef BB_FEATURE_SH_FANCY_PROMPT
+       /* Set up the prompt */
+       if (shell_context == 0) {
+               if (PS1)
+                       free(PS1);
+               PS1=xmalloc(strlen(cwd)+4);
+               sprintf(PS1, "%s %s", cwd, ( geteuid() != 0 ) ?  "$ ":"# ");
+               *prompt_str = PS1;
+       } else {
+               *prompt_str = PS2;
+       }
+#else
+       *prompt_str = (shell_context==0)? PS1 : PS2;
+#endif 
+}
+
+static int get_command(FILE * source, char *command)
+{
+       char *prompt_str;
+
        if (source == NULL) {
                if (local_pending_command) {
                        /* a command specified (-c option): return it & mark it done */
@@ -676,69 +703,32 @@ static int getCommand(FILE * source, char *command)
                return 1;
        }
 
-       /* get User Name and setup prompt */
-       strcpy(prompt,( geteuid() != 0 ) ? "$ ":"# ");
-       user=xcalloc(sizeof(int), 9);
-       my_getpwuid(user, geteuid());
-       
-       /* get HostName */
-       gethostname(buf, 255);
-       s = strchr(buf, '.');
-       if (s)
-               *s = 0;
-       
        if (source == stdin) {
-#ifdef BB_FEATURE_SH_COMMAND_EDITING
-               int len;
+               setup_prompt_string(&prompt_str);
 
+#ifdef BB_FEATURE_COMMAND_EDITING
                /*
                ** enable command line editing only while a command line
                ** is actually being read; otherwise, we'll end up bequeathing
                ** atexit() handlers and other unwanted stuff to our
                ** child processes (rob@sysgo.de)
                */
-               cmdedit_init();
-               signal(SIGWINCH, win_changed);
-               len=fprintf(stdout, "[%s@%s %s]%s", user, buf, 
-                               get_last_path_component(cwd), prompt);
-               fflush(stdout);
-               promptStr=(char*)xmalloc(sizeof(char)*(len+1));
-               sprintf(promptStr, "[%s@%s %s]%s", user, buf, 
-                               get_last_path_component(cwd), prompt);
-               cmdedit_read_input(promptStr, command);
-               free( promptStr);
-               cmdedit_terminate();
-               signal(SIGWINCH, SIG_DFL);
+               cmdedit_read_input(prompt_str, command);
                return 0;
 #else
-               i=strlen(cwd);
-               i--;
-               if (i>1){
-                       while ((i>0) && (*(cwd+i)!='/') ) i--;
-                       if (*(cwd+i)=='/') i++;
-               }
-               
-               fprintf(stdout, "[%s@%s %s]%s",user, buf, (cwd+i), prompt);
-               fflush(stdout);
+               fputs(prompt_str, stdout);
 #endif
        }
 
-       /* don't leak memory */
-       free(user);
-       
        if (!fgets(command, BUFSIZ - 2, source)) {
                if (source == stdin)
                        printf("\n");
                return 1;
        }
 
-       /* remove trailing newline */
-       command[strlen(command) - 1] = '\0';
-
        return 0;
 }
 
-#ifdef BB_FEATURE_SH_ENVIRONMENT
 static char* itoa(register int i)
 {
        static char a[7]; /* Max 7 ints */
@@ -758,168 +748,259 @@ static char* itoa(register int i)
                *--b = '-';
        return b;
 }
-#endif
 
-static void globLastArgument(struct childProgram *prog, int *argcPtr,
-                                                        int *argcAllocedPtr)
+char * strsep_space( char *string, int * ix)
 {
-       int argc_l = *argcPtr;
-       int argcAlloced = *argcAllocedPtr;
-       int rc;
-       int flags;
-       int i;
-       char *src, *dst, *var;
+       char *token, *begin;
 
-       if (argc_l > 1) {                               /* cmd->globResult is already initialized */
-               flags = GLOB_APPEND;
-               i = prog->globResult.gl_pathc;
-       } else {
-               prog->freeGlob = 1;
-               flags = 0;
-               i = 0;
+       begin = string;
+
+       /* Short circuit the trivial case */
+       if ( !string || ! string[*ix])
+               return NULL;
+
+       /* Find the end of the token. */
+       while( string && string[*ix] && !isspace(string[*ix]) ) {
+               (*ix)++;
        }
-       /* do shell variable substitution */
-       if(*prog->argv[argc_l - 1] == '$') {
-               if ((var = getenv(prog->argv[argc_l - 1] + 1))) {
-                       prog->argv[argc_l - 1] = var;
-               } 
-#ifdef BB_FEATURE_SH_ENVIRONMENT
-               else {
-                       switch(*(prog->argv[argc_l - 1] + 1)) {
-                               case '?':
-                                       prog->argv[argc_l - 1] = itoa(lastReturnCode);
-                                       break;
-                               case '$':
-                                       prog->argv[argc_l - 1] = itoa(getpid());
-                                       break;
-                               case '#':
-                                       prog->argv[argc_l - 1] = itoa(argc-1);
-                                       break;
-                               case '!':
-                                       if (lastBgPid==-1)
-                                               *(prog->argv[argc_l - 1])='\0';
-                                       else
-                                               prog->argv[argc_l - 1] = itoa(lastBgPid);
-                                       break;
-                               case '0':case '1':case '2':case '3':case '4':
-                               case '5':case '6':case '7':case '8':case '9':
-                                       {
-                                               int index=*(prog->argv[argc_l - 1] + 1)-48;
-                                               if (index >= argc) {
-                                                       *(prog->argv[argc_l - 1])='\0';
-                                               } else {
-                                                       prog->argv[argc_l - 1] = argv[index];
-                                               }
-                                       }
-                                       break;
-                       }
-               }
-#endif
+
+       /* Find the end of any whitespace trailing behind 
+        * the token and let that be part of the token */
+       while( string && string[*ix] && isspace(string[*ix]) ) {
+               (*ix)++;
        }
 
-       if (strpbrk(prog->argv[argc_l - 1],"*[]?")!= NULL){
-               rc = glob(prog->argv[argc_l - 1], flags, NULL, &prog->globResult);
-               if (rc == GLOB_NOSPACE) {
-                       errorMsg("out of space during glob operation\n");
-                       return;
-               } else if (rc == GLOB_NOMATCH ||
-                          (!rc && (prog->globResult.gl_pathc - i) == 1 &&
-                               strcmp(prog->argv[argc_l - 1],
-                                               prog->globResult.gl_pathv[i]) == 0)) {
-                       /* we need to remove whatever \ quoting is still present */
-                       src = dst = prog->argv[argc_l - 1];
-                       while (*src) {
-                               if (*src == '\\') {
-                                       src++; 
-                                       *dst++ = process_escape_sequence(&src);
-                               } else { 
-                                       *dst++ = *src;
-                                       src++;
+       if (! string && *ix==0) {
+               /* Nothing useful was found */
+               return NULL;
+       }
+
+       token = xmalloc(*ix+1);
+       token[*ix] = '\0';
+       strncpy(token, string,  *ix); 
+
+       return token;
+}
+
+static int expand_arguments(char *command)
+{
+       int total_length=0, length, i, retval, ix = 0;
+       expand_t expand_result;
+       char *tmpcmd, *cmd, *cmd_copy;
+       char *src, *dst, *var;
+       const char *out_of_space = "out of space during expansion";
+       int flags = GLOB_NOCHECK
+#ifdef GLOB_BRACE
+               | GLOB_BRACE
+#endif 
+#ifdef GLOB_TILDE
+               | GLOB_TILDE
+#endif 
+               ;
+
+       /* get rid of the terminating \n */
+       chomp(command);
+
+       /* Fix up escape sequences to be the Real Thing(tm) */
+       while( command && command[ix]) {
+               if (command[ix] == '\\') {
+                       const char *tmp = command+ix+1;
+                       command[ix] = process_escape_sequence(  &tmp );
+                       memmove(command+ix + 1, tmp, strlen(tmp)+1);
+               }
+               ix++;
+       }
+       /* Use glob and then fixup environment variables and such */
+
+       /* It turns out that glob is very stupid.  We have to feed it one word at a
+        * time since it can't cope with a full string.  Here we convert command
+        * (char*) into cmd (char**, one word per string) */
+
+       /* We need a clean copy, so strsep can mess up the copy while
+        * we write stuff into the original (in a minute) */
+       cmd = cmd_copy = strdup(command);
+       *command = '\0';
+       for (ix = 0, tmpcmd = cmd; 
+                       (tmpcmd = strsep_space(cmd, &ix)) != NULL; cmd += ix, ix=0) {
+               if (*tmpcmd == '\0')
+                       break;
+               /* we need to trim() the result for glob! */
+               trim(tmpcmd);
+               retval = glob(tmpcmd, flags, NULL, &expand_result);
+               free(tmpcmd); /* Free mem allocated by strsep_space */
+               if (retval == GLOB_NOSPACE) {
+                       /* Mem may have been allocated... */
+                       globfree (&expand_result);
+                       error_msg(out_of_space);
+                       return FALSE;
+               } else if (retval != 0) {
+                       /* Some other error.  GLOB_NOMATCH shouldn't
+                        * happen because of the GLOB_NOCHECK flag in 
+                        * the glob call. */
+                       error_msg("syntax error");
+                       return FALSE;
+               } else {
+                       /* Convert from char** (one word per string) to a simple char*,
+                        * but don't overflow command which is BUFSIZ in length */
+                       for (i=0; i < expand_result.gl_pathc; i++) {
+                               length=strlen(expand_result.gl_pathv[i]);
+                               if (total_length+length+1 >= BUFSIZ) {
+                                       error_msg(out_of_space);
+                                       return FALSE;
                                }
+                               strcat(command+total_length, " ");
+                               total_length+=1;
+                               strcat(command+total_length, expand_result.gl_pathv[i]);
+                               total_length+=length;
                        }
-                       *dst = '\0';
-               } else if (!rc) {
-                       argcAlloced += (prog->globResult.gl_pathc - i);
-                       prog->argv = xrealloc(prog->argv, argcAlloced * sizeof(*prog->argv));
-                       memcpy(prog->argv + (argc_l - 1), prog->globResult.gl_pathv + i,
-                                  sizeof(*(prog->argv)) * (prog->globResult.gl_pathc - i));
-                       argc_l += (prog->globResult.gl_pathc - i - 1);
+                       globfree (&expand_result);
                }
-       }else{
-                       src = dst = prog->argv[argc_l - 1];
-                       while (*src) {
-                               if (*src == '\\') {
-                                       src++; 
-                                       *dst++ = process_escape_sequence(&src);
-                               } else { 
-                                       *dst++ = *src;
-                                       src++;
+       }
+       free(cmd_copy);
+       trim(command);
+
+       /* Now do the shell variable substitutions which 
+        * wordexp can't do for us, namely $? and $! */
+       src = command;
+       while((dst = strchr(src,'$')) != NULL){
+               var = NULL;
+               switch(*(dst+1)) {
+                       case '?':
+                               var = itoa(last_return_code);
+                               break;
+                       case '!':
+                               if (last_bg_pid==-1)
+                                       *(var)='\0';
+                               else
+                                       var = itoa(last_bg_pid);
+                               break;
+                               /* Everything else like $$, $#, $[0-9], etc. should all be
+                                * expanded by wordexp(), so we can in theory skip that stuff
+                                * here, but just to be on the safe side (i.e., since uClibc
+                                * wordexp doesn't do this stuff yet), lets leave it in for
+                                * now. */
+                       case '$':
+                               var = itoa(getpid());
+                               break;
+                       case '#':
+                               var = itoa(argc-1);
+                               break;
+                       case '0':case '1':case '2':case '3':case '4':
+                       case '5':case '6':case '7':case '8':case '9':
+                               {
+                                       int ixx=*(dst + 1)-48;
+                                       if (ixx >= argc) {
+                                               var='\0';
+                                       } else {
+                                               var = argv[ixx];
+                                       }
                                }
+                               break;
+
+               }
+               if (var) {
+                       /* a single character construction was found, and 
+                        * already handled in the case statement */
+                       src=dst+2;
+               } else {
+                       /* Looks like an environment variable */
+                       char delim_hold;
+                       int num_skip_chars=0;
+                       int dstlen = strlen(dst);
+                       /* Is this a ${foo} type variable? */
+                       if (dstlen >=2 && *(dst+1) == '{') {
+                               src=strchr(dst+1, '}');
+                               num_skip_chars=1;
+                       } else {
+                               src=dst+1;
+                               while(isalnum(*src) || *src=='_') src++;
+                       }
+                       if (src == NULL) {
+                               src = dst+dstlen;
                        }
-                       *dst = '\0';
-                       
-                       prog->globResult.gl_pathc=0;
-                       if (flags==0)
-                               prog->globResult.gl_pathv=NULL;
+                       delim_hold=*src;
+                       *src='\0';  /* temporary */
+                       var = getenv(dst + 1 + num_skip_chars);
+                       *src=delim_hold;
+                       src += num_skip_chars;
+               }
+               if (var == NULL) {
+                       /* Seems we got an un-expandable variable.  So delete it. */
+                       var = "";
+               }
+               {
+                       int subst_len = strlen(var);
+                       int trail_len = strlen(src);
+                       if (dst+subst_len+trail_len >= command+BUFSIZ) {
+                               error_msg(out_of_space);
+                               return FALSE;
+                       }
+                       /* Move stuff to the end of the string to accommodate
+                        * filling the created gap with the new stuff */
+                       memmove(dst+subst_len, src, trail_len+1);
+                       /* Now copy in the new stuff */
+                       memcpy(dst, var, subst_len);
+                       src = dst+subst_len;
+               }
        }
-       *argcAllocedPtr = argcAlloced;
-       *argcPtr = argc_l;
+
+       return TRUE;
 }
 
-/* Return cmd->numProgs as 0 if no command is present (e.g. an empty
-   line). If a valid command is found, commandPtr is set to point to
+/* Return cmd->num_progs as 0 if no command is present (e.g. an empty
+   line). If a valid command is found, command_ptr is set to point to
    the beginning of the next command (if the original command had more 
    then one job associated with it) or NULL if no more commands are 
    present. */
-static int parseCommand(char **commandPtr, struct job *job, struct jobSet *jobList, int *inBg)
+static int parse_command(char **command_ptr, struct job *job, int *inbg)
 {
        char *command;
-       char *returnCommand = NULL;
+       char *return_command = NULL;
        char *src, *buf, *chptr;
        int argc_l = 0;
        int done = 0;
-       int argvAlloced;
-       int i;
+       int argv_alloced;
+       int i, saw_quote = 0;
        char quote = '\0';
        int count;
-       struct childProgram *prog;
+       struct child_prog *prog;
 
        /* skip leading white space */
-       while (**commandPtr && isspace(**commandPtr))
-               (*commandPtr)++;
+       while (**command_ptr && isspace(**command_ptr))
+               (*command_ptr)++;
 
        /* this handles empty lines or leading '#' characters */
-       if (!**commandPtr || (**commandPtr == '#')) {
-               job->numProgs=0;
+       if (!**command_ptr || (**command_ptr == '#')) {
+               job->num_progs=0;
                return 0;
        }
 
-       *inBg = 0;
-       job->numProgs = 1;
+       *inbg = 0;
+       job->num_progs = 1;
        job->progs = xmalloc(sizeof(*job->progs));
 
        /* We set the argv elements to point inside of this string. The 
-          memory is freed by freeJob(). Allocate twice the original
+          memory is freed by free_job(). Allocate twice the original
           length in case we need to quote every single character.
 
           Getting clean memory relieves us of the task of NULL 
           terminating things and makes the rest of this look a bit 
           cleaner (though it is, admittedly, a tad less efficient) */
-       job->cmdBuf = command = xcalloc(2*strlen(*commandPtr) + 1, sizeof(char));
+       job->cmdbuf = command = xcalloc(2*strlen(*command_ptr) + 1, sizeof(char));
        job->text = NULL;
 
        prog = job->progs;
-       prog->numRedirections = 0;
-       prog->redirections = NULL;
-       prog->freeGlob = 0;
-       prog->isStopped = 0;
+       prog->num_redirects = 0;
+       prog->redirects = NULL;
+       prog->is_stopped = 0;
+       prog->family = job;
 
-       argvAlloced = 5;
-       prog->argv = xmalloc(sizeof(*prog->argv) * argvAlloced);
-       prog->argv[0] = job->cmdBuf;
+       argv_alloced = 5;
+       prog->argv = xmalloc(sizeof(*prog->argv) * argv_alloced);
+       prog->argv[0] = job->cmdbuf;
 
        buf = command;
-       src = *commandPtr;
+       src = *command_ptr;
        while (*src && !done) {
                if (quote == *src) {
                        quote = '\0';
@@ -927,35 +1008,38 @@ static int parseCommand(char **commandPtr, struct job *job, struct jobSet *jobLi
                        if (*src == '\\') {
                                src++;
                                if (!*src) {
-                                       errorMsg("character expected after \\\n");
-                                       freeJob(job);
+                                       error_msg("character expected after \\");
+                                       free_job(job);
                                        return 1;
                                }
 
                                /* in shell, "\'" should yield \' */
-                               if (*src != quote)
+                               if (*src != quote) {
                                        *buf++ = '\\';
+                                       *buf++ = '\\';
+                               }
                        } else if (*src == '*' || *src == '?' || *src == '[' ||
                                           *src == ']') *buf++ = '\\';
                        *buf++ = *src;
                } else if (isspace(*src)) {
-                       if (*prog->argv[argc_l]) {
+                       if (*prog->argv[argc_l] || saw_quote) {
                                buf++, argc_l++;
                                /* +1 here leaves room for the NULL which ends argv */
-                               if ((argc_l + 1) == argvAlloced) {
-                                       argvAlloced += 5;
+                               if ((argc_l + 1) == argv_alloced) {
+                                       argv_alloced += 5;
                                        prog->argv = xrealloc(prog->argv,
                                                                                  sizeof(*prog->argv) *
-                                                                                 argvAlloced);
+                                                                                 argv_alloced);
                                }
-                               globLastArgument(prog, &argc_l, &argvAlloced);
                                prog->argv[argc_l] = buf;
+                               saw_quote = 0;
                        }
                } else
                        switch (*src) {
                        case '"':
                        case '\'':
                                quote = *src;
+                               saw_quote = 1;
                                break;
 
                        case '#':                       /* comment */
@@ -965,42 +1049,41 @@ static int parseCommand(char **commandPtr, struct job *job, struct jobSet *jobLi
                                        done = 1;
                                break;
 
-                       case '>':                       /* redirections */
+                       case '>':                       /* redirects */
                        case '<':
-                               i = prog->numRedirections++;
-                               prog->redirections = xrealloc(prog->redirections,
-                                                                                         sizeof(*prog->redirections) *
+                               i = prog->num_redirects++;
+                               prog->redirects = xrealloc(prog->redirects,
+                                                                                         sizeof(*prog->redirects) *
                                                                                          (i + 1));
 
-                               prog->redirections[i].fd = -1;
+                               prog->redirects[i].fd = -1;
                                if (buf != prog->argv[argc_l]) {
                                        /* the stuff before this character may be the file number 
                                           being redirected */
-                                       prog->redirections[i].fd =
+                                       prog->redirects[i].fd =
                                                strtol(prog->argv[argc_l], &chptr, 10);
 
                                        if (*chptr && *prog->argv[argc_l]) {
                                                buf++, argc_l++;
-                                               globLastArgument(prog, &argc_l, &argvAlloced);
                                                prog->argv[argc_l] = buf;
                                        }
                                }
 
-                               if (prog->redirections[i].fd == -1) {
+                               if (prog->redirects[i].fd == -1) {
                                        if (*src == '>')
-                                               prog->redirections[i].fd = 1;
+                                               prog->redirects[i].fd = 1;
                                        else
-                                               prog->redirections[i].fd = 0;
+                                               prog->redirects[i].fd = 0;
                                }
 
                                if (*src++ == '>') {
                                        if (*src == '>')
-                                               prog->redirections[i].type =
+                                               prog->redirects[i].type =
                                                        REDIRECT_APPEND, src++;
                                        else
-                                               prog->redirections[i].type = REDIRECT_OVERWRITE;
+                                               prog->redirects[i].type = REDIRECT_OVERWRITE;
                                } else {
-                                       prog->redirections[i].type = REDIRECT_INPUT;
+                                       prog->redirects[i].type = REDIRECT_INPUT;
                                }
 
                                /* This isn't POSIX sh compliant. Oh well. */
@@ -1009,13 +1092,13 @@ static int parseCommand(char **commandPtr, struct job *job, struct jobSet *jobLi
                                        chptr++;
 
                                if (!*chptr) {
-                                       errorMsg("file name expected after %c\n", *src);
-                                       freeJob(job);
-                                       job->numProgs=0;
+                                       error_msg("file name expected after %c", *(src-1));
+                                       free_job(job);
+                                       job->num_progs=0;
                                        return 1;
                                }
 
-                               prog->redirections[i].filename = buf;
+                               prog->redirects[i].filename = buf;
                                while (*chptr && !isspace(*chptr))
                                        *buf++ = *chptr++;
 
@@ -1025,29 +1108,29 @@ static int parseCommand(char **commandPtr, struct job *job, struct jobSet *jobLi
 
                        case '|':                       /* pipe */
                                /* finish this command */
-                               if (*prog->argv[argc_l])
+                               if (*prog->argv[argc_l] || saw_quote)
                                        argc_l++;
                                if (!argc_l) {
-                                       errorMsg("empty command in pipe\n");
-                                       freeJob(job);
-                                       job->numProgs=0;
+                                       error_msg("empty command in pipe");
+                                       free_job(job);
+                                       job->num_progs=0;
                                        return 1;
                                }
                                prog->argv[argc_l] = NULL;
 
                                /* and start the next */
-                               job->numProgs++;
+                               job->num_progs++;
                                job->progs = xrealloc(job->progs,
-                                                                         sizeof(*job->progs) * job->numProgs);
-                               prog = job->progs + (job->numProgs - 1);
-                               prog->numRedirections = 0;
-                               prog->redirections = NULL;
-                               prog->freeGlob = 0;
-                               prog->isStopped = 0;
+                                                                         sizeof(*job->progs) * job->num_progs);
+                               prog = job->progs + (job->num_progs - 1);
+                               prog->num_redirects = 0;
+                               prog->redirects = NULL;
+                               prog->is_stopped = 0;
+                               prog->family = job;
                                argc_l = 0;
 
-                               argvAlloced = 5;
-                               prog->argv = xmalloc(sizeof(*prog->argv) * argvAlloced);
+                               argv_alloced = 5;
+                               prog->argv = xmalloc(sizeof(*prog->argv) * argv_alloced);
                                prog->argv[0] = ++buf;
 
                                src++;
@@ -1055,9 +1138,9 @@ static int parseCommand(char **commandPtr, struct job *job, struct jobSet *jobLi
                                        src++;
 
                                if (!*src) {
-                                       errorMsg("empty command in pipe\n");
-                                       freeJob(job);
-                                       job->numProgs=0;
+                                       error_msg("empty command in pipe");
+                                       free_job(job);
+                                       job->num_progs=0;
                                        return 1;
                                }
                                src--;                  /* we'll ++ it at the end of the loop */
@@ -1065,88 +1148,17 @@ static int parseCommand(char **commandPtr, struct job *job, struct jobSet *jobLi
                                break;
 
                        case '&':                       /* background */
-                               *inBg = 1;
+                               *inbg = 1;
                        case ';':                       /* multiple commands */
                                done = 1;
-                               returnCommand = *commandPtr + (src - *commandPtr) + 1;
-                               break;
-
-#ifdef BB_FEATURE_SH_BACKTICKS
-                       case '`':
-                               /* Exec a backtick-ed command */
-                               {
-                                       char* charptr1=NULL, *charptr2;
-                                       char* ptr=NULL;
-                                       struct job *newJob;
-                                       struct jobSet njobList = { NULL, NULL };
-                                       int pipefd[2];
-                                       int size;
-
-                                       ptr=strchr(++src, '`');
-                                       if (ptr==NULL) {
-                                               fprintf(stderr, "Unmatched '`' in command\n");
-                                               freeJob(job);
-                                               return 1;
-                                       }
-
-                                       /* Make some space to hold just the backticked command */
-                                       charptr1 = charptr2 = xmalloc(1+ptr-src);
-                                       memcpy(charptr1, src, ptr-src);
-                                       charptr1[ptr-src] = '\0';
-                                       newJob = xmalloc(sizeof(struct job));
-                                       /* Now parse and run the backticked command */
-                                       if (!parseCommand(&charptr1, newJob, &njobList, inBg) 
-                                                       && newJob->numProgs) {
-                                               pipe(pipefd);
-                                               runCommand(newJob, &njobList, 0, pipefd);
-                                       }
-                                       checkJobs(jobList);
-                                       freeJob(newJob);
-                                       free(charptr2);
-                                       
-                                       /* Make a copy of any stuff left over in the command 
-                                        * line after the second backtick */
-                                       charptr2 = xmalloc(strlen(ptr)+1);
-                                       memcpy(charptr2, ptr+1, strlen(ptr));
-
-
-                                       /* Copy the output from the backtick-ed command into the
-                                        * command line, making extra room as needed  */
-                                       --src;
-                                       charptr1 = xmalloc(BUFSIZ);
-                                       while ( (size=fullRead(pipefd[0], charptr1, BUFSIZ-1)) >0) {
-                                               int newSize=src - *commandPtr + size + 1 + strlen(charptr2);
-                                               if (newSize > BUFSIZ) {
-                                                       *commandPtr=xrealloc(*commandPtr, src - *commandPtr + 
-                                                                       size + 1 + strlen(charptr2));
-                                               }
-                                               memcpy(src, charptr1, size); 
-                                               src+=size;
-                                       }
-                                       free(charptr1);
-                                       close(pipefd[0]);
-                                       if (*(src-1)=='\n')
-                                               --src;
-
-                                       /* Now paste into the *commandPtr all the stuff 
-                                        * leftover after the second backtick */
-                                       memcpy(src, charptr2, strlen(charptr2)+1);
-                                       free(charptr2);
-
-                                       /* Now recursively call parseCommand to deal with the new
-                                        * and improved version of the command line with the backtick
-                                        * results expanded in place... */
-                                       freeJob(job);
-                                       return(parseCommand(commandPtr, job, jobList, inBg));
-                               }
+                               return_command = *command_ptr + (src - *command_ptr) + 1;
                                break;
-#endif // BB_FEATURE_SH_BACKTICKS
 
                        case '\\':
                                src++;
                                if (!*src) {
-                                       errorMsg("character expected after \\\n");
-                                       freeJob(job);
+                                       error_msg("character expected after \\");
+                                       free_job(job);
                                        return 1;
                                }
                                if (*src == '*' || *src == '[' || *src == ']'
@@ -1159,78 +1171,193 @@ static int parseCommand(char **commandPtr, struct job *job, struct jobSet *jobLi
                src++;
        }
 
-       if (*prog->argv[argc_l]) {
+       if (*prog->argv[argc_l] || saw_quote) {
                argc_l++;
-               globLastArgument(prog, &argc_l, &argvAlloced);
        }
        if (!argc_l) {
-               freeJob(job);
+               free_job(job);
                return 0;
        }
        prog->argv[argc_l] = NULL;
 
-       if (!returnCommand) {
-               job->text = xmalloc(strlen(*commandPtr) + 1);
-               strcpy(job->text, *commandPtr);
+       if (!return_command) {
+               job->text = xmalloc(strlen(*command_ptr) + 1);
+               strcpy(job->text, *command_ptr);
        } else {
                /* This leaves any trailing spaces, which is a bit sloppy */
-               count = returnCommand - *commandPtr;
+               count = return_command - *command_ptr;
                job->text = xmalloc(count + 1);
-               strncpy(job->text, *commandPtr, count);
+               strncpy(job->text, *command_ptr, count);
                job->text[count] = '\0';
        }
 
-       *commandPtr = returnCommand;
+       *command_ptr = return_command;
        
        return 0;
 }
 
-static int runCommand(struct job *newJob, struct jobSet *jobList, int inBg, int outPipe[2])
+/* Run the child_prog, no matter what kind of command it uses.
+ */
+static int pseudo_exec(struct child_prog *child)
 {
-       struct job *theJob;
+       struct built_in_command *x;
+#ifdef BB_FEATURE_SH_STANDALONE_SHELL
+       char *name;
+#endif
+
+       /* Check if the command matches any of the non-forking builtins.
+        * Depending on context, this might be redundant.  But it's
+        * easier to waste a few CPU cycles than it is to figure out
+        * if this is one of those cases.
+        */
+       for (x = bltins; x->cmd; x++) {
+               if (strcmp(child->argv[0], x->cmd) == 0 ) {
+                       exit(x->function(child));
+               }
+       }
+
+       /* Check if the command matches any of the forking builtins. */
+       for (x = bltins_forking; x->cmd; x++) {
+               if (strcmp(child->argv[0], x->cmd) == 0) {
+                       applet_name=x->cmd;
+                       exit (x->function(child));
+               }
+       }
+#ifdef BB_FEATURE_SH_STANDALONE_SHELL
+       /* Check if the command matches any busybox internal
+        * commands ("applets") here.  Following discussions from
+        * November 2000 on busybox@opensource.lineo.com, don't use
+        * get_last_path_component().  This way explicit (with
+        * slashes) filenames will never be interpreted as an
+        * applet, just like with builtins.  This way the user can
+        * override an applet with an explicit filename reference.
+        * The only downside to this change is that an explicit
+        * /bin/foo invocation will fork and exec /bin/foo, even if
+        * /bin/foo is a symlink to busybox.
+        */
+       name = child->argv[0];
+
+#ifdef BB_FEATURE_SH_APPLETS_ALWAYS_WIN
+       /* If you enable BB_FEATURE_SH_APPLETS_ALWAYS_WIN, then
+        * if you run /bin/cat, it will use BusyBox cat even if 
+        * /bin/cat exists on the filesystem and is _not_ busybox.
+        * Some systems want this, others do not.  Choose wisely.  :-)
+        */
+       name = get_last_path_component(name);
+#endif
+
+       {
+           char** argv_l=child->argv;
+           int argc_l;
+           for(argc_l=0;*argv_l!=NULL; argv_l++, argc_l++);
+           optind = 1;
+           run_applet_by_name(name, argc_l, child->argv);
+       }
+#endif
+
+       execvp(child->argv[0], child->argv);
+       perror_msg_and_die("%s", child->argv[0]);
+}
+
+static void insert_job(struct job *newjob, int inbg)
+{
+       struct job *thejob;
+       struct jobset *j_list=newjob->job_list;
+
+       /* find the ID for thejob to use */
+       newjob->jobid = 1;
+       for (thejob = j_list->head; thejob; thejob = thejob->next)
+               if (thejob->jobid >= newjob->jobid)
+                       newjob->jobid = thejob->jobid + 1;
+
+       /* add thejob to the list of running jobs */
+       if (!j_list->head) {
+               thejob = j_list->head = xmalloc(sizeof(*thejob));
+       } else {
+               for (thejob = j_list->head; thejob->next; thejob = thejob->next) /* nothing */;
+               thejob->next = xmalloc(sizeof(*thejob));
+               thejob = thejob->next;
+       }
+
+       *thejob = *newjob;   /* physically copy the struct job */
+       thejob->next = NULL;
+       thejob->running_progs = thejob->num_progs;
+       thejob->stopped_progs = 0;
+
+       if (inbg) {
+               /* we don't wait for background thejobs to return -- append it 
+                  to the list of backgrounded thejobs and leave it alone */
+               printf("[%d] %d\n", thejob->jobid,
+                          newjob->progs[newjob->num_progs - 1].pid);
+               last_jobid = newjob->jobid;
+               last_bg_pid=newjob->progs[newjob->num_progs - 1].pid;
+       } else {
+               newjob->job_list->fg = thejob;
+
+               /* move the new process group into the foreground */
+               /* suppress messages when run from /linuxrc mag@sysgo.de */
+               if (tcsetpgrp(shell_terminal, newjob->pgrp) && errno != ENOTTY)
+                       perror_msg("tcsetpgrp");
+       }
+}
+
+static int run_command(struct job *newjob, int inbg, int outpipe[2])
+{
+       /* struct job *thejob; */
        int i;
        int nextin, nextout;
        int pipefds[2];                         /* pipefd[0] is for reading */
-       struct builtInCommand *x;
-#ifdef BB_FEATURE_SH_STANDALONE_SHELL
-       struct BB_applet search_applet, *applet;
-#endif
+       struct built_in_command *x;
+       struct child_prog *child;
 
        nextin = 0, nextout = 1;
-       for (i = 0; i < newJob->numProgs; i++) {
-               if ((i + 1) < newJob->numProgs) {
-                       pipe(pipefds);
+       for (i = 0; i < newjob->num_progs; i++) {
+               child = & (newjob->progs[i]);
+
+               if ((i + 1) < newjob->num_progs) {
+                       if (pipe(pipefds)<0) perror_msg_and_die("pipe");
                        nextout = pipefds[1];
                } else {
-                       if (outPipe[1]!=-1) {
-                               nextout = outPipe[1];
+                       if (outpipe[1]!=-1) {
+                               nextout = outpipe[1];
                        } else {
                                nextout = 1;
                        }
                }
 
-#ifdef BB_FEATURE_SH_ENVIRONMENT
-               if (showXtrace==TRUE) {
-                       int j;
-                       fprintf(stderr, "+ ");
-                       for (j = 0; newJob->progs[i].argv[j]; j++)
-                               fprintf(stderr, "%s ", newJob->progs[i].argv[j]);
-                       fprintf(stderr, "\n");
-               }
-#endif
 
-               /* Check if the command matches any non-forking builtins */
-               for (x = bltins; x->cmd; x++) {
-                       if (strcmp(newJob->progs[i].argv[0], x->cmd) == 0 ) {
-                               return(x->function(newJob, jobList));
+               /* Check if the command matches any non-forking builtins,
+                * but only if this is a simple command.
+                * Non-forking builtins within pipes have to fork anyway,
+                * and are handled in pseudo_exec.  "echo foo | read bar"
+                * is doomed to failure, and doesn't work on bash, either.
+                */
+               if (newjob->num_progs == 1) {
+                       for (x = bltins; x->cmd; x++) {
+                               if (strcmp(child->argv[0], x->cmd) == 0 ) {
+                                       int squirrel[] = {-1, -1, -1};
+                                       int rcode;
+                                       setup_redirects(child, squirrel);
+                                       rcode = x->function(child);
+                                       restore_redirects(squirrel);
+                                       return rcode;
+                               }
                        }
                }
 
-               if (!(newJob->progs[i].pid = fork())) {
+               if (!(child->pid = fork())) {
+                       /* Set the handling for job control signals back to the default.  */
+                       signal(SIGINT, SIG_DFL);
+                       signal(SIGQUIT, SIG_DFL);
+                       signal(SIGTSTP, SIG_DFL);
+                       signal(SIGTTIN, SIG_DFL);
                        signal(SIGTTOU, SIG_DFL);
+                       signal(SIGCHLD, SIG_DFL);
+
+                       close_all();
 
-                       if (outPipe[1]!=-1) {
-                               close(outPipe[0]);
+                       if (outpipe[1]!=-1) {
+                               close(outpipe[0]);
                        }
                        if (nextin != 0) {
                                dup2(nextin, 0);
@@ -1239,68 +1366,23 @@ static int runCommand(struct job *newJob, struct jobSet *jobList, int inBg, int
 
                        if (nextout != 1) {
                                dup2(nextout, 1);
-                               dup2(nextout, 2);
+                               dup2(nextout, 2);  /* Really? */
                                close(nextout);
                                close(pipefds[0]);
                        }
 
-                       /* explicit redirections override pipes */
-                       setupRedirections(newJob->progs + i);
-
-                       /* Check if the command matches any of the other builtins */
-                       for (x = bltins_forking; x->cmd; x++) {
-                               if (strcmp(newJob->progs[i].argv[0], x->cmd) == 0) {
-                                       applet_name=x->cmd;
-                                       exit (x->function(newJob, jobList));
-                               }
-                       }
-#ifdef BB_FEATURE_SH_STANDALONE_SHELL
-                       /* Check if the command matches any busybox internal
-                        * commands ("applets") here.  Following discussions from
-                        * November 2000 on busybox@opensource.lineo.com, don't use
-                        * get_last_path_component().  This way explicit (with
-                        * slashes) filenames will never be interpreted as an
-                        * applet, just like with builtins.  This way the user can
-                        * override an applet with an explicit filename reference.
-                        * The only downside to this change is that an explicit
-                        * /bin/foo invocation fill fork and exec /bin/foo, even if
-                        * /bin/foo is a symlink to busybox.
-                        */
-                       search_applet.name = newJob->progs[i].argv[0];
-
-#ifdef BB_FEATURE_SH_APPLETS_ALWAYS_WIN
-                       /* If you enable BB_FEATURE_SH_APPLETS_ALWAYS_WIN, then
-                        * if you run /bin/cat, it will use BusyBox cat even if 
-                        * /bin/cat exists on the filesystem and is _not_ busybox.
-                        * Some systems want this, others do not.  Choose wisely.  :-)
-                        */
-                       search_applet.name = get_last_path_component(search_applet.name);
-#endif
-
-                       /* Do a binary search to find the applet entry given the name. */
-                       applet = bsearch(&search_applet, applets, NUM_APPLETS,
-                                       sizeof(struct BB_applet), applet_name_compare);
-                       if (applet != NULL) {
-                               int argc_l;
-                               char** argv=newJob->progs[i].argv;
-                               for(argc_l=0;*argv!=NULL; argv++, argc_l++);
-                               applet_name=applet->name;
-                               optind = 1;
-                               exit((*(applet->main)) (argc_l, newJob->progs[i].argv));
-                       }
-#endif
+                       /* explicit redirects override pipes */
+                       setup_redirects(child,NULL);
 
-                       execvp(newJob->progs[i].argv[0], newJob->progs[i].argv);
-                       fatalError("%s: %s\n", newJob->progs[i].argv[0],
-                                       strerror(errno));
+                       pseudo_exec(child);
                }
-               if (outPipe[1]!=-1) {
-                       close(outPipe[1]);
+               if (outpipe[1]!=-1) {
+                       close(outpipe[1]);
                }
 
                /* put our child in the process group whose leader is the
                   first process in this pipe */
-               setpgid(newJob->progs[i].pid, newJob->progs[0].pid);
+               setpgid(child->pid, newjob->progs[0].pid);
                if (nextin != 0)
                        close(nextin);
                if (nextout != 1)
@@ -1311,44 +1393,9 @@ static int runCommand(struct job *newJob, struct jobSet *jobList, int inBg, int
                nextin = pipefds[0];
        }
 
-       newJob->pgrp = newJob->progs[0].pid;
-
-       /* find the ID for the theJob to use */
-       newJob->jobId = 1;
-       for (theJob = jobList->head; theJob; theJob = theJob->next)
-               if (theJob->jobId >= newJob->jobId)
-                       newJob->jobId = theJob->jobId + 1;
+       newjob->pgrp = newjob->progs[0].pid;
 
-       /* add the theJob to the list of running jobs */
-       if (!jobList->head) {
-               theJob = jobList->head = xmalloc(sizeof(*theJob));
-       } else {
-               for (theJob = jobList->head; theJob->next; theJob = theJob->next);
-               theJob->next = xmalloc(sizeof(*theJob));
-               theJob = theJob->next;
-       }
-
-       *theJob = *newJob;
-       theJob->next = NULL;
-       theJob->runningProgs = theJob->numProgs;
-       theJob->stoppedProgs = 0;
-
-       if (inBg) {
-               /* we don't wait for background theJobs to return -- append it 
-                  to the list of backgrounded theJobs and leave it alone */
-               printf("[%d] %d\n", theJob->jobId,
-                          newJob->progs[newJob->numProgs - 1].pid);
-#ifdef BB_FEATURE_SH_ENVIRONMENT
-               lastBgPid=newJob->progs[newJob->numProgs - 1].pid;
-#endif
-       } else {
-               jobList->fg = theJob;
-
-               /* move the new process group into the foreground */
-               /* suppress messages when run from /linuxrc mag@sysgo.de */
-               if (tcsetpgrp(0, newJob->pgrp) && errno != ENOTTY)
-                       perror("tcsetpgrp");
-       }
+       insert_job(newjob, inbg);
 
        return 0;
 }
@@ -1356,94 +1403,98 @@ static int runCommand(struct job *newJob, struct jobSet *jobList, int inBg, int
 static int busy_loop(FILE * input)
 {
        char *command;
-       char *nextCommand = NULL;
-       struct job newJob;
+       char *next_command = NULL;
+       struct job newjob;
        pid_t  parent_pgrp;
        int i;
-       int inBg;
+       int inbg;
        int status;
-       newJob.jobContext = REGULAR_JOB_CONTEXT;
+       newjob.job_list = &job_list;
+       newjob.job_context = DEFAULT_CONTEXT;
 
        /* save current owner of TTY so we can restore it on exit */
-       parent_pgrp = tcgetpgrp(0);
+       parent_pgrp = tcgetpgrp(shell_terminal);
 
        command = (char *) xcalloc(BUFSIZ, sizeof(char));
 
-       /* don't pay any attention to this signal; it just confuses 
-          things and isn't really meant for shells anyway */
-       signal(SIGTTOU, SIG_IGN);
-
        while (1) {
-               if (!jobList.fg) {
+               if (!job_list.fg) {
                        /* no job is in the foreground */
 
                        /* see if any background processes have exited */
-                       checkJobs(&jobList);
+                       checkjobs(&job_list);
 
-                       if (!nextCommand) {
-                               if (getCommand(input, command))
+                       if (!next_command) {
+                               if (get_command(input, command))
                                        break;
-                               nextCommand = command;
+                               next_command = command;
+                       }
+
+                       if (expand_arguments(next_command) == FALSE) {
+                               free(command);
+                               command = (char *) xcalloc(BUFSIZ, sizeof(char));
+                               next_command = NULL;
+                               continue;
                        }
 
-                       if (!parseCommand(&nextCommand, &newJob, &jobList, &inBg) &&
-                               newJob.numProgs) {
+                       if (!parse_command(&next_command, &newjob, &inbg) &&
+                               newjob.num_progs) {
                                int pipefds[2] = {-1,-1};
-                               runCommand(&newJob, &jobList, inBg, pipefds);
+                               debug_printf( "job=%p fed to run_command by busy_loop()'\n", 
+                                               &newjob);
+                               run_command(&newjob, inbg, pipefds);
                        }
                        else {
                                free(command);
                                command = (char *) xcalloc(BUFSIZ, sizeof(char));
-                               nextCommand = NULL;
+                               next_command = NULL;
                        }
                } else {
                        /* a job is running in the foreground; wait for it */
                        i = 0;
-                       while (!jobList.fg->progs[i].pid ||
-                                  jobList.fg->progs[i].isStopped == 1) i++;
+                       while (!job_list.fg->progs[i].pid ||
+                                  job_list.fg->progs[i].is_stopped == 1) i++;
 
-                       waitpid(jobList.fg->progs[i].pid, &status, WUNTRACED);
+                       if (waitpid(job_list.fg->progs[i].pid, &status, WUNTRACED)<0)
+                               perror_msg_and_die("waitpid(%d)",job_list.fg->progs[i].pid);
 
                        if (WIFEXITED(status) || WIFSIGNALED(status)) {
                                /* the child exited */
-                               jobList.fg->runningProgs--;
-                               jobList.fg->progs[i].pid = 0;
+                               job_list.fg->running_progs--;
+                               job_list.fg->progs[i].pid = 0;
 
-#ifdef BB_FEATURE_SH_ENVIRONMENT
-                               lastReturnCode=WEXITSTATUS(status);
-#endif
-                               debug_printf("'%s' exited -- return code %d\n", jobList.fg->text, lastReturnCode);
-                               if (!jobList.fg->runningProgs) {
-                                       /* child exited */
+                               last_return_code=WEXITSTATUS(status);
 
-                                       removeJob(&jobList, jobList.fg);
-                                       jobList.fg = NULL;
+                               if (!job_list.fg->running_progs) {
+                                       /* child exited */
+                                       remove_job(&job_list, job_list.fg);
+                                       job_list.fg = NULL;
                                }
                        } else {
                                /* the child was stopped */
-                               jobList.fg->stoppedProgs++;
-                               jobList.fg->progs[i].isStopped = 1;
+                               job_list.fg->stopped_progs++;
+                               job_list.fg->progs[i].is_stopped = 1;
 
-                               if (jobList.fg->stoppedProgs == jobList.fg->runningProgs) {
-                                       printf("\n" JOB_STATUS_FORMAT, jobList.fg->jobId,
-                                                  "Stopped", jobList.fg->text);
-                                       jobList.fg = NULL;
+                               if (job_list.fg->stopped_progs == job_list.fg->running_progs) {
+                                       printf("\n" JOB_STATUS_FORMAT, job_list.fg->jobid,
+                                                  "Stopped", job_list.fg->text);
+                                       job_list.fg = NULL;
                                }
                        }
 
-                       if (!jobList.fg) {
+                       if (!job_list.fg) {
                                /* move the shell to the foreground */
                                /* suppress messages when run from /linuxrc mag@sysgo.de */
-                               if (tcsetpgrp(0, getpid()) && errno != ENOTTY)
-                                       perror("tcsetpgrp"); 
+                               if (tcsetpgrp(shell_terminal, getpgrp()) && errno != ENOTTY)
+                                       perror_msg("tcsetpgrp"); 
                        }
                }
        }
        free(command);
 
        /* return controlling TTY back to parent process group before exiting */
-       if (tcsetpgrp(0, parent_pgrp))
-               perror("tcsetpgrp");
+       if (tcsetpgrp(shell_terminal, parent_pgrp))
+               perror_msg("tcsetpgrp");
 
        /* return exit status if called with "-c" */
        if (input == NULL && WIFEXITED(status))
@@ -1456,60 +1507,87 @@ static int busy_loop(FILE * input)
 #ifdef BB_FEATURE_CLEAN_UP
 void free_memory(void)
 {
-       if (promptStr)
-               free(promptStr);
-       if (cwd)
+       if (cwd) {
                free(cwd);
+       }
        if (local_pending_command)
                free(local_pending_command);
 
-       if (jobList.fg && !jobList.fg->runningProgs) {
-               removeJob(&jobList, jobList.fg);
+       if (job_list.fg && !job_list.fg->running_progs) {
+               remove_job(&job_list, job_list.fg);
        }
 }
 #endif
 
+/* Make sure we have a controlling tty.  If we get started under a job
+ * aware app (like bash for example), make sure we are now in charge so
+ * we don't fight over who gets the foreground */
+static void setup_job_control()
+{
+       /* Loop until we are in the foreground.  */
+       while (tcgetpgrp (shell_terminal) != (shell_pgrp = getpgrp ()))
+               kill (- shell_pgrp, SIGTTIN);
+
+       /* Ignore interactive and job-control signals.  */
+       signal(SIGINT, SIG_IGN);
+       signal(SIGQUIT, SIG_IGN);
+       signal(SIGTSTP, SIG_IGN);
+       signal(SIGTTIN, SIG_IGN);
+       signal(SIGTTOU, SIG_IGN);
+       signal(SIGCHLD, SIG_IGN);
 
-int shell_main(int argc_l, char **argv_l)
+       /* Put ourselves in our own process group.  */
+       setsid();
+       shell_pgrp = getpid ();
+       setpgid (shell_pgrp, shell_pgrp);
+
+       /* Grab control of the terminal.  */
+       tcsetpgrp(shell_terminal, shell_pgrp);
+}
+
+int lash_main(int argc_l, char **argv_l)
 {
        int opt, interactive=FALSE;
        FILE *input = stdin;
        argc = argc_l;
        argv = argv_l;
 
+       /* These variables need re-initializing when recursing */
+       last_jobid = 0;
+       local_pending_command = NULL;
+       close_me_head = NULL;
+       job_list.head = NULL;
+       job_list.fg = NULL;
+       last_return_code=1;
 
        if (argv[0] && argv[0][0] == '-') {
-                 FILE *input;
-                 input = fopen("/etc/profile", "r");
-                 if (!input) {
-                         fprintf(stdout, "Couldn't open file '/etc/profile'\n");
-                 } else {
-                         /* Now run the file */
-                         busy_loop(input);
-                         fclose(input);
-                 }
+               FILE *prof_input;
+               prof_input = fopen("/etc/profile", "r");
+               if (prof_input) {
+                       int tmp_fd = fileno(prof_input);
+                       mark_open(tmp_fd);      
+                       /* Now run the file */
+                       busy_loop(prof_input);
+                       fclose(prof_input);
+                       mark_closed(tmp_fd);
+               }
        }
 
-       while ((opt = getopt(argc_l, argv_l, "cx")) > 0) {
+       while ((opt = getopt(argc_l, argv_l, "cxi")) > 0) {
                switch (opt) {
                        case 'c':
                                input = NULL;
                                if (local_pending_command != 0)
-                                       fatalError("multiple -c arguments\n");
+                                       error_msg_and_die("multiple -c arguments");
                                local_pending_command = xstrdup(argv[optind]);
                                optind++;
                                argv = argv+optind;
                                break;
-#ifdef BB_FEATURE_SH_ENVIRONMENT
-                       case 'x':
-                               showXtrace = TRUE;
-                               break;
-#endif
                        case 'i':
                                interactive = TRUE;
                                break;
                        default:
-                               usage(shell_usage);
+                               show_usage();
                }
        }
        /* A shell is interactive if the `-i' flag was given, or if all of
@@ -1519,24 +1597,36 @@ int shell_main(int argc_l, char **argv_l)
         *    standard input is a terminal
         *    standard output is a terminal
         *    Refer to Posix.2, the description of the `sh' utility. */
-       if (interactive==TRUE || ( argv[optind]==NULL && input==stdin && isatty(fileno(stdin)) && isatty(fileno(stdout)))) {
-               //fprintf(stdout, "optind=%d  argv[optind]='%s'\n", optind, argv[optind]);
+       if (argv[optind]==NULL && input==stdin &&
+                       isatty(fileno(stdin)) && isatty(fileno(stdout))) {
+               interactive=TRUE;
+       }
+       setup_job_control();
+       if (interactive==TRUE) {
+               //printf( "optind=%d  argv[optind]='%s'\n", optind, argv[optind]);
                /* Looks like they want an interactive shell */
-               fprintf(stdout, "\n\nBusyBox v%s (%s) Built-in shell\n", BB_VER, BB_BT);
-               fprintf(stdout, "Enter 'help' for a list of built-in commands.\n\n");
+               printf( "\n\n" BB_BANNER " Built-in shell (lash)\n");
+               printf( "Enter 'help' for a list of built-in commands.\n\n");
        } else if (local_pending_command==NULL) {
-               //fprintf(stdout, "optind=%d  argv[optind]='%s'\n", optind, argv[optind]);
+               //printf( "optind=%d  argv[optind]='%s'\n", optind, argv[optind]);
                input = xfopen(argv[optind], "r");
+               mark_open(fileno(input));  /* be lazy, never mark this closed */
        }
 
        /* initialize the cwd -- this is never freed...*/
-       cwd=(char*)xmalloc(sizeof(char)*MAX_LINE+1);
-       getcwd(cwd, sizeof(char)*MAX_LINE);
+       cwd = xgetcwd(0);
+       if (!cwd)
+               cwd = unknown;
 
 #ifdef BB_FEATURE_CLEAN_UP
        atexit(free_memory);
 #endif
 
-       win_changed(0);
+#ifdef BB_FEATURE_COMMAND_EDITING
+       cmdedit_set_initial_prompt();
+#else
+       PS1 = NULL;
+#endif
+       
        return (busy_loop(input));
 }