Skip to content

Commit

Permalink
[Cherry-pick] Fix Windows Release CI failure due to unresolved extern…
Browse files Browse the repository at this point in the history
…al symbol (#5689)

### Description
Cherry-pick #5678.

### Motivation and Context
To fix Windows Release CI failure due to unresolved external symbol:
cherry-pick #5678.

Signed-off-by: jcwchen <jacky82226@gmail.com>
  • Loading branch information
jcwchen committed Oct 20, 2023
1 parent 6a20ba8 commit ccde5da
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions cmake/summary.cmake
Expand Up @@ -27,6 +27,8 @@ function (onnx_print_configuration_summary)
message(STATUS " ONNX_WERROR : ${ONNX_WERROR}")
message(STATUS " ONNX_BUILD_TESTS : ${ONNX_BUILD_TESTS}")
message(STATUS " ONNX_BUILD_BENCHMARKS : ${ONNX_BUILD_BENCHMARKS}")
message(STATUS " ONNX_BUILD_SHARED_LIBS : ${ONNX_BUILD_SHARED_LIBS}")
message(STATUS " BUILD_SHARED_LIBS : ${BUILD_SHARED_LIBS}")
message(STATUS "")
message(STATUS " Protobuf compiler : ${PROTOBUF_PROTOC_EXECUTABLE}")
message(STATUS " Protobuf includes : ${PROTOBUF_INCLUDE_DIRS}")
Expand Down
2 changes: 1 addition & 1 deletion workflow_scripts/protobuf/build_protobuf_win.ps1
Expand Up @@ -25,7 +25,7 @@ echo "Set paths"
$protoc_path = Join-Path -Path $protobuf_root_dir -ChildPath "protobuf_install\bin"
$protoc_lib_path = Join-Path -Path $protobuf_root_dir -ChildPath "protobuf_install\lib"
$protobuf_include_path = Join-Path -Path $protobuf_root_dir -ChildPath "protobuf_install\include"
$Env:PATH="$ENV:PATH;$protoc_path;$protoc_lib_path;$protobuf_include_path"
$Env:PATH="$protoc_path;$protoc_lib_path;$protobuf_include_path;$ENV:PATH"
$($Env:PATH).Split(';')
protoc
cd ../../

0 comments on commit ccde5da

Please sign in to comment.