diff options
| author | Yaniv Levinsky <yaniv.levinsky@compulab.co.il> | 2018-06-24 19:16:56 +0300 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2018-07-19 16:17:58 -0400 |
| commit | 5a04264ecf5f0f853f2d562b4c5efb139d3d4a9d (patch) | |
| tree | 8274c01751e26af15b2c410c135760f2c78757a8 /env | |
| parent | 477f8116d517798d4d6456d920d57ff2f46d4271 (diff) | |
cmd: nvedit: set H_INTERACTIVE in do_env_default
The function set_default_vars() in common.c adds H_INTERACTIVE to the
h_import() flag, but the function has no way of telling if the command
actually was user directed like this flag suggest. The flag should be
set by the calling function do_env_default() in nvedit.c instead, where
the command is certainty user directed.
Move the H_INTERACTIVE flag from set_default_vars() to do_env_default().
Signed-off-by: Yaniv Levinsky <yaniv.levinsky@compulab.co.il>
Acked-by: Igor Grinberg <grinberg@compulab.co.il>
Diffstat (limited to 'env')
| -rw-r--r-- | env/common.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/env/common.c b/env/common.c index 6cf5eddaf6..05183a4af0 100644 --- a/env/common.c +++ b/env/common.c @@ -97,7 +97,7 @@ int set_default_vars(int nvars, char * const vars[], int flags) * Special use-case: import from default environment * (and use \0 as a separator) */ - flags |= H_NOCLEAR | H_INTERACTIVE; + flags |= H_NOCLEAR; return himport_r(&env_htab, (const char *)default_environment, sizeof(default_environment), '\0', flags, 0, nvars, vars); |