Merge topic 'execute_process-no-extension'

f6d2efa752 Tests: Add case to cover execute_process support for no extension on Windows
da9df7425a libuv: win/spawn: run executables with no file extension

Acked-by: Kitware Robot <[email protected]>
Acked-by: scivision <[email protected]>
Merge-request: !9017

Tests/RunCMake/CMakeLists.txt

@@ -476,6 +476,7 @@ set(execute_process_ARGS
   -DEXIT_CODE_EXE=$<TARGET_FILE:exit_code>
   -DPRINT_STDIN_EXE=$<TARGET_FILE:print_stdin>
   -DPython_EXECUTABLE=${Python_EXECUTABLE}
+  -DCYGWIN=${CYGWIN}
   )
 if(NOT CMake_TEST_EXTERNAL_CMAKE)
   list(APPEND execute_process_ARGS -DTEST_ENCODING_EXE=$<TARGET_FILE:testEncoding>)

Tests/RunCMake/execute_process/RunCMakeTest.cmake

@@ -46,3 +46,10 @@ if(UNIX AND Python_EXECUTABLE)
   run_cmake_command(LastCommandAbnormalExit-1 ${CMAKE_COMMAND} -DPython_EXECUTABLE=${Python_EXECUTABLE} -P ${RunCMake_SOURCE_DIR}/LastCommandAbnormalExit-1.cmake)
   run_cmake_command(LastCommandAbnormalExit-2 ${CMAKE_COMMAND} -DPython_EXECUTABLE=${Python_EXECUTABLE} -P ${RunCMake_SOURCE_DIR}/LastCommandAbnormalExit-2.cmake)
 endif()
+
+if(WIN32 OR CYGWIN)
+  set(RunCMake_TEST_BINARY_DIR ${RunCMake_BINARY_DIR}/WindowsNoExtension-build)
+  run_cmake(WindowsNoExtension)
+  set(RunCMake_TEST_NO_CLEAN 1)
+  run_cmake_command(WindowsNoExtension-build ${CMAKE_COMMAND} --build . --config Debug --target RunScript)
+endif()

Tests/RunCMake/execute_process/WindowsNoExtension-build-stdout.txt

@@ -0,0 +1,6 @@
+[^
+]*This executable does not have an extension
+[^
+]*This executable does not have an extension
+[^
+]*This executable has an extension

Tests/RunCMake/execute_process/WindowsNoExtension.cmake

@@ -0,0 +1,12 @@
+enable_language(C)
+
+add_executable(exe_extension exe_extension.c)
+add_executable(exe_no_extension exe_no_extension.c)
+
+add_custom_target(RunScript
+  ${CMAKE_COMMAND}
+    -Dexe_extension=$<TARGET_FILE:exe_extension>
+    -Dexe_no_extension=$<TARGET_FILE:exe_no_extension>
+    -P ${CMAKE_CURRENT_SOURCE_DIR}/WindowsNoExtensionRunScript.cmake
+  DEPENDS exe_extension exe_no_extension
+  )

Tests/RunCMake/execute_process/WindowsNoExtensionRunScript.cmake

@@ -0,0 +1,17 @@
+file(COPY_FILE "${exe_no_extension}" "${CMAKE_CURRENT_BINARY_DIR}/exe" INPUT_MAY_BE_RECENT)
+execute_process(
+  COMMAND "${CMAKE_CURRENT_BINARY_DIR}/exe"
+  COMMAND_ERROR_IS_FATAL ANY
+  )
+
+file(COPY_FILE "${exe_extension}" "${CMAKE_CURRENT_BINARY_DIR}/exe.exe" INPUT_MAY_BE_RECENT)
+execute_process(
+  COMMAND "${CMAKE_CURRENT_BINARY_DIR}/exe"
+  COMMAND_ERROR_IS_FATAL ANY
+  )
+
+file(REMOVE "${CMAKE_CURRENT_BINARY_DIR}/exe")
+execute_process(
+  COMMAND "${CMAKE_CURRENT_BINARY_DIR}/exe"
+  COMMAND_ERROR_IS_FATAL ANY
+  )

Tests/RunCMake/execute_process/exe_extension.c

@@ -0,0 +1,7 @@
+#include <stdio.h>
+
+int main(void)
+{
+  printf("This executable has an extension\n");
+  return 0;
+}

Tests/RunCMake/execute_process/exe_no_extension.c

@@ -0,0 +1,7 @@
+#include <stdio.h>
+
+int main(void)
+{
+  printf("This executable does not have an extension\n");
+  return 0;
+}

Utilities/cmlibuv/src/win/process.c

@@ -274,19 +274,16 @@ static WCHAR* path_search_walk_ext(const WCHAR *dir,
                                    const WCHAR *name,
                                    size_t name_len,
                                    WCHAR *cwd,
-                                   size_t cwd_len,
-                                   int name_has_ext) {
+                                   size_t cwd_len) {
   WCHAR* result;
 
-  /* If the name itself has a nonempty extension, try this extension first */
-  if (name_has_ext) {
-    result = search_path_join_test(dir, dir_len,
-                                   name, name_len,
-                                   L"", 0,
-                                   cwd, cwd_len);
-    if (result != NULL) {
-      return result;
-    }
+  /* Try the name itself first */
+  result = search_path_join_test(dir, dir_len,
+                                 name, name_len,
+                                 L"", 0,
+                                 cwd, cwd_len);
+  if (result != NULL) {
+    return result;
   }
 
   /* Try .com extension */
@@ -329,8 +326,7 @@ static WCHAR* path_search_walk_ext(const WCHAR *dir,
  * - If there's really only a filename, check the current directory for file,
  *   then search all path directories.
  *
- * - If filename specified has *any* extension, search for the file with the
- *   specified extension first.
+ * - Search for the file exactly as specified first.
  *
  * - If the literal filename is not found in a directory, try *appending*
  *   (not replacing) .com first and then .exe.
@@ -360,10 +356,8 @@ static WCHAR* search_path(const WCHAR *file,
   int file_has_dir;
   WCHAR* result = NULL;
   WCHAR *file_name_start;
-  WCHAR *dot;
   const WCHAR *dir_start, *dir_end, *dir_path;
   size_t dir_len;
-  int name_has_ext;
 
   size_t file_len = wcslen(file);
   size_t cwd_len = wcslen(cwd);
@@ -387,17 +381,12 @@ static WCHAR* search_path(const WCHAR *file,
 
   file_has_dir = file_name_start != file;
 
-  /* Check if the filename includes an extension */
-  dot = wcschr(file_name_start, L'.');
-  name_has_ext = (dot != NULL && dot[1] != L'\0');
-
   if (file_has_dir) {
     /* The file has a path inside, don't use path */
     result = path_search_walk_ext(
         file, file_name_start - file,
         file_name_start, file_len - (file_name_start - file),
-        cwd, cwd_len,
-        name_has_ext);
+        cwd, cwd_len);
 
   } else {
     dir_end = path;
@@ -405,8 +394,7 @@ static WCHAR* search_path(const WCHAR *file,
     /* The file is really only a name; look in cwd first, then scan path */
     result = path_search_walk_ext(L"", 0,
                                   file, file_len,
-                                  cwd, cwd_len,
-                                  name_has_ext);
+                                  cwd, cwd_len);
 
     while (result == NULL) {
       if (dir_end == NULL || *dir_end == L'\0') {
@@ -454,8 +442,7 @@ static WCHAR* search_path(const WCHAR *file,
 
       result = path_search_walk_ext(dir_path, dir_len,
                                     file, file_len,
-                                    cwd, cwd_len,
-                                    name_has_ext);
+                                    cwd, cwd_len);
     }
   }