Skip to content

Fix compile issue in Azure EP unit test #24446

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 16, 2025
Merged

Conversation

adrianlizarraga
Copy link
Contributor

@adrianlizarraga adrianlizarraga commented Apr 16, 2025

Description

Fix compilation issue (undeclared identifier) in Azure EP unit test.

Motivation and Context

A previous PR caused a compilation issue in the Azure EP unit test: #24433

Our PR CI pipelines did not catch it. It was caught by our post-merge packaging pipelines.

D:\a\_work\1\s\onnxruntime\test\providers\azure\azure_basic_test.cc(28,3): error C2065: 'session_options': undeclared identifier [D:\a\_work\1\b\RelWithDebInfo\onnxruntime_test_all.vcxproj]
D:\a\_work\1\s\onnxruntime\test\providers\azure\azure_basic_test.cc(29,3): error C2065: 'session_options': undeclared identifier [D:\a\_work\1\b\RelWithDebInfo\onnxruntime_test_all.vcxproj]
D:\a\_work\1\s\onnxruntime\test\providers\azure\azure_basic_test.cc(30,3): error C2065: 'session_options': undeclared identifier [D:\a\_work\1\b\RelWithDebInfo\onnxruntime_test_all.vcxproj]

@adrianlizarraga adrianlizarraga requested a review from fs-eire April 16, 2025 19:20
@fs-eire fs-eire merged commit 01c8e83 into main Apr 16, 2025
93 of 116 checks passed
@fs-eire fs-eire deleted the adrianl/azure-ep-fix-test branch April 16, 2025 21:30
ashrit-ms pushed a commit that referenced this pull request Apr 24, 2025
### Description
Fix compilation issue (undeclared identifier) in Azure EP unit test.



### Motivation and Context
A previous PR caused a compilation issue in the Azure EP unit test:
#24433

Our PR CI pipelines did not catch it. It was caught by our post-merge
packaging pipelines.

```shell
D:\a\_work\1\s\onnxruntime\test\providers\azure\azure_basic_test.cc(28,3): error C2065: 'session_options': undeclared identifier [D:\a\_work\1\b\RelWithDebInfo\onnxruntime_test_all.vcxproj]
D:\a\_work\1\s\onnxruntime\test\providers\azure\azure_basic_test.cc(29,3): error C2065: 'session_options': undeclared identifier [D:\a\_work\1\b\RelWithDebInfo\onnxruntime_test_all.vcxproj]
D:\a\_work\1\s\onnxruntime\test\providers\azure\azure_basic_test.cc(30,3): error C2065: 'session_options': undeclared identifier [D:\a\_work\1\b\RelWithDebInfo\onnxruntime_test_all.vcxproj]
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants