From 7a8e491dffe98ebc19fa83c2785c8477522e3270 Mon Sep 17 00:00:00 2001 From: Rob Landley Date: Sat, 1 Jul 2006 17:19:02 +0000 Subject: [PATCH] Patch from Shaun Jackman moving the var=value logic to here it can do some good. --- shell/lash.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/shell/lash.c b/shell/lash.c index 1b34052a1..83baee29d 100644 --- a/shell/lash.c +++ b/shell/lash.c @@ -1171,12 +1171,6 @@ static int pseudo_exec(struct child_prog *child) { struct built_in_command *x; - /* Check if the command sets an environment variable. */ - if( strchr(child->argv[0], '=') != NULL ) { - child->argv[1] = child->argv[0]; - _exit(builtin_export(child)); - } - /* 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 @@ -1300,6 +1294,12 @@ static int run_command(struct job *newjob, int inbg, int outpipe[2]) * is doomed to failure, and doesn't work on bash, either. */ if (newjob->num_progs == 1) { + /* Check if the command sets an environment variable. */ + if (strchr(child->argv[0], '=') != NULL) { + child->argv[1] = child->argv[0]; + return builtin_export(child); + } + for (x = bltins; x->cmd; x++) { if (strcmp(child->argv[0], x->cmd) == 0 ) { int rcode; -- 2.25.1