mirror of
https://github.com/saymrwulf/onnxruntime.git
synced 2026-05-16 21:00:14 +00:00
80a60d9a65
Avoid using command line flags to pass in CMAKE_PREFIX_PATH. Use environment variables instead. Because, otherwise the value of CMAKE_PREFIX_PATH could get encoded twice. For example, if the prefix is `C:\a\root`, then in tools/ci_build/github/windows/helpers.ps1 we set it in Env:CMAKE_ARGS which will be consumed by ONNX. Then when ONNX get it and decoded it, ONNX will get `C:aroot` instead. Then because the path doesn't exist, the CMAKE_PREFIX_PATH couldn't take effect when the script installs ONNX. This PR fixes the issue. The issue got discovered when I tried to upgrade cmake to a newer version. Now our Windows CPU CI build pipeline uses cmake 3.27. In the main branch even the CMAKE_PREFIX_PATH setting does not work, cmake still can find protoc.exe from the directories. However, starting from 3.28 cmake changed it. With the newer cmake versions the find_library(), find_path(), and find_file() cmake commands no longer search in installation prefixes derived from the PATH environment variable. |
||
|---|---|---|
| .. | ||
| eager | ||
| post_to_dashboard | ||
| bundle_dlls_gpu.bat | ||
| bundle_nuget_with_native_headers.bat | ||
| extract_nuget_files.ps1 | ||
| extract_nuget_files_gpu.ps1 | ||
| extract_zip_files_gpu.ps1 | ||
| helpers.ps1 | ||
| install_third_party_deps.ps1 | ||
| jar_gpu_packaging.ps1 | ||
| jar_packaging.ps1 | ||
| post_binary_sizes_to_dashboard.py | ||
| post_code_coverage_to_dashboard.py | ||
| set_telemetry_var.ps1 | ||
| setup_env.bat | ||
| setup_env_cuda.bat | ||
| setup_env_gpu.bat | ||
| setup_env_trt.bat | ||
| setup_env_x86.bat | ||