summary refs log tree commit diff
path: root/src/error.c
diff options
context:
space:
mode:
authorJonathan Nieder <jrnieder@gmail.com>2011-07-07 11:55:42 +0800
committerHerbert Xu <herbert@gondor.apana.org.au>2011-07-07 12:56:32 +0800
commit66b614e29038e31745c4a5d296f64f8d64f5c377 (patch)
treebca0ac7ca1f8b2f62aa9017412976042cbccaade /src/error.c
parent[PARSER] Fix clobbering of checkkwd (diff)
downloaddash-66b614e29038e31745c4a5d296f64f8d64f5c377.tar.gz
dash-66b614e29038e31745c4a5d296f64f8d64f5c377.zip
[EVAL] Remove unused EV_BACKCMD flag
The original ash defered forking commands in backquotes so builtins
could be run in the same context as the shell.  This behavior was
controlled using the EV_BACKCMD to evaltree.

Unfortunately, as Matthias Scheler noticed in 1999 (NetBSD PR/7814),
the result was counterintuitive; for example, echo "`cd /`" would
change the cwd.  So ash 0.3.5 left out that optimization.  The
EV_BACKCMD codepath stayed around, unused.

Some time between ash 0.3.5-11 and ash 0.3.8-37, Debian ash omitted
the EV_BACKCMD pathway by guarding it with #ifdef notyet.  In dash
0.5.1 and later, the commented code is no more.  Let's finish the job
and remove the last vestiges.  If someone wants to work on omitting
the fork in backcmd, the remaining hints are not going to be very
helpful, anyway.

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'src/error.c')
0 files changed, 0 insertions, 0 deletions