diff options
| author | Ron Yorston | 2020-12-11 12:34:21 +0000 |
|---|---|---|
| committer | Denys Vlasenko | 2020-12-16 09:38:25 +0100 |
| commit | 8baa643a3445882ec9c39dfcabb7374081c13aee (patch) | |
| tree | d0542acb9a36d2bcabe82e362f34b8d10fa8dee2 /libbb/lineedit.c | |
| parent | 8506dd673030539b2890dd617f885ec20f1e8a7d (diff) | |
| download | busybox-8baa643a3445882ec9c39dfcabb7374081c13aee.zip busybox-8baa643a3445882ec9c39dfcabb7374081c13aee.tar.gz | |
lineedit: match local directories when searching PATH
When tab-completing a command we search PATH if the partial text
doesn't include a slash. Also match subdirectories of the current
directory, in case the user intends to run a binary from one of
them.
function old new delta
complete_cmd_dir_file 894 917 +23
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 1/0 up/down: 23/0) Total: 23 bytes
Signed-off-by: Ron Yorston <rmy@pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'libbb/lineedit.c')
| -rw-r--r-- | libbb/lineedit.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/libbb/lineedit.c b/libbb/lineedit.c index d64d7d0..5eb701f 100644 --- a/libbb/lineedit.c +++ b/libbb/lineedit.c @@ -753,7 +753,7 @@ static int path_parse(char ***p) return 1; tmp = (char*)pth; - npth = 1; /* path component count */ + npth = 2; /* path component count */ while (1) { tmp = strchr(tmp, ':'); if (!tmp) @@ -776,6 +776,8 @@ static int path_parse(char ***p) break; /* :<empty> */ res[npth++] = tmp; } + /* special case: match subdirectories of the current directory */ + res[npth++] = NULL; return npth; } @@ -843,6 +845,11 @@ static NOINLINE unsigned complete_cmd_dir_file(const char *command, int type) struct stat st; char *found; + if (paths[i] == NULL) { + type = FIND_DIR_ONLY; + paths[i] = (char *)"."; + } + dir = opendir(paths[i]); if (!dir) continue; /* don't print an error */ |