diff options
| author | Justin M. Keyes <justinkz@gmail.com> | 2019-07-21 16:37:07 +0200 |
|---|---|---|
| committer | Justin M. Keyes <justinkz@gmail.com> | 2019-07-21 20:51:37 +0200 |
| commit | b08dc3ec195feb4ab69c898ab816c104d2aa1aa1 (patch) | |
| tree | 3e614d685f1b0335719df538377b93a491eae3ed /src/nvim/os | |
| parent | 9d0f8224c97b36116f3c0523bcf6941382892a5b (diff) | |
| download | rneovim-b08dc3ec195feb4ab69c898ab816c104d2aa1aa1.tar.gz rneovim-b08dc3ec195feb4ab69c898ab816c104d2aa1aa1.tar.bz2 rneovim-b08dc3ec195feb4ab69c898ab816c104d2aa1aa1.zip | |
win: jobstart(), system(): $PATHEXT-resolve exe
Windows: In order for jobstart(['foo']), system(['foo']) to find
"foo.cmd", we must replace "foo" with "foo.cmd" before sending `argv` to
process_spawn().
Rationale: jobstart([…]), system([…]) "executable" semantics should be
consistent with the VimL executable() function.
fix #9569
related: #10554
Diffstat (limited to 'src/nvim/os')
| -rw-r--r-- | src/nvim/os/fs.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/nvim/os/fs.c b/src/nvim/os/fs.c index 4a10b5199c..f34592d107 100644 --- a/src/nvim/os/fs.c +++ b/src/nvim/os/fs.c @@ -230,7 +230,7 @@ int os_exepath(char *buffer, size_t *size) /// Checks if the file `name` is executable. /// /// @param[in] name Filename to check. -/// @param[out] abspath Returns resolved executable path, if not NULL. +/// @param[out,allocated] abspath Returns resolved exe path, if not NULL. /// @param[in] use_path Also search $PATH. /// /// @return true if `name` is executable and @@ -271,6 +271,9 @@ bool os_can_exe(const char_u *name, char_u **abspath, bool use_path) } /// Returns true if `name` is an executable file. +/// +/// @param[in] name Filename to check. +/// @param[out,allocated] abspath Returns full exe path, if not NULL. static bool is_executable(const char *name, char_u **abspath) FUNC_ATTR_NONNULL_ARG(1) { |