ash: Remove unused EV_BACKCMD flag
authorDenys Vlasenko <vda.linux@googlemail.com>
Wed, 28 Sep 2016 17:44:16 +0000 (19:44 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Wed, 28 Sep 2016 17:44:16 +0000 (19:44 +0200)
    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.

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/ash.c

index ec006af7d2242586675e226257cd3718ea8d7675..f395a16a964137a75cf49fea8a295fd359e1f63e 100644 (file)
@@ -8065,7 +8065,6 @@ static char *funcstring;        /* block to allocate strings from */
 /* flags in argument to evaltree */
 #define EV_EXIT    01           /* exit after evaluating tree */
 #define EV_TESTED  02           /* exit status is checked; ignore -e flag */
-#define EV_BACKCMD 04           /* command executing within back quotes */
 
 static const uint8_t nodesize[N_NUMBER] ALIGN1 = {
        [NCMD     ] = SHELL_ALIGN(sizeof(struct ncmd)),