cmake: Fix PATH variable quoting for WSL environments with spaces in directories #6443
Conversation
Add quotes around PATH variable expansion to properly handle paths with spaces in the grass_env_command for non-Windows systems. This ensures consistent path handling across different Unix-like environments.
…ries Add quotes around PATH variable expansion in grass_env_command for Unix systems to properly handle Windows paths with spaces when running GRASS in WSL. This prevents path parsing issues that can occur when WSL mounts Windows directories containing spaces (e.g., 'Program Files') in the PATH.
|
Is CMake 3.25 finally available to us? If so, we could use the env modification feature, https://cmake.org/cmake/help/latest/manual/cmake.1.html#cmdoption-cmake-E-arg-env, https://cmake.org/cmake/help/latest/prop_test/ENVIRONMENT_MODIFICATION.html#prop_test:ENVIRONMENT_MODIFICATION and use the portable path_list_append/path_list_prepend. Also, maybe using some path handling CMake utilities like https://cmake.org/cmake/help/latest/command/cmake_path.html#cmake-path-to-native-path-list. I thought I had already read something that mentioned the space handling, but I couldn’t refind it yet |
|
|
list(APPEND environment_modifications "${dl_paths_variable_name}=path_list_prepend:${native_path}") |
3 participants
Add quotes around PATH variable expansion in grass_env_command for Unix systems to properly handle Windows paths with spaces when running GRASS in WSL. This prevents path parsing issues that can occur when WSL mounts Windows directories containing spaces (e.g., 'Program Files') in the PATH.
@HuidaeCho