win32: additional improvements to handling of executables

Consistent processing of file extensions, as described in the previous
commit, has been applied to the 'which' applet and the functions
find_executable and mingw_spawn_interpreter.

In spawnveq check that the file to be executed exists and is executable,
and ensure that it won't have any extensions added by spawnve.

It's intended that all files passed to spawnve should have their names
fully specified.  If this isn't the case the tests here will cause errors
which will need to be fixed.

1 files changed, 6 insertions, 5 deletions

diff --git a/debianutils/which.c b/debianutils/which.c
index 9060a5b47..0b0a6a645 100644
--- a/debianutils/which.c
+++ b/debianutils/which.c
@@ -66,17 +66,18 @@ int which_main(int argc UNUSED_PARAM, char **argv)
 
                 /* If file contains a slash don't use PATH */
                 if (strchr(*argv, '/') || (ENABLE_PLATFORM_MINGW32 && strchr(*argv, '\\'))) {
-                        if (file_is_executable(*argv)) {
-                                missing = 0;
-                                puts(*argv);
-                        }
 #if ENABLE_PLATFORM_MINGW32
-                        else if ((p=add_win32_extension(*argv)) != NULL) {
+                        if ((p=add_win32_extension(*argv)) != NULL) {
                                 missing = 0;
                                 puts(p);
                                 free(p);
                         }
+                        else
 #endif
+                        if (file_is_executable(*argv)) {
+                                missing = 0;
+                                puts(*argv);
+                        }
                 } else {
                         char *path;