[Build] Find openvino based on env variable

[gedoensmax]

I think the change is pretty self explanatory. The openvino installer sets the used Environment variable and therefore finding openvino is easier.

[jywu-msft]

are you encountering a case where it can't find openvino?
currently,

onnxruntime/cmake/CMakeLists.txt

Line 1301 in 0f7a31b

if (NOT DEFINED ENV{INTEL_OPENVINO_DIR})

is already checking that openvino is installed and initialized so I had assumed the existing cmake/scripts should all be working?

[gedoensmax]

Yes I was encountering the issue without this line. It also tells me I should use find_package(OpenVINO REQUIRED rather than the used finders. I see that this find call is only used for some OV versions, I just pushed a fix that removes some complexity in build scrips. I believe that drops support for older OV's but not sure if that would still be needed.

[jywu-msft]

Yes I was encountering the issue without this line. It also tells me I should use find_package(OpenVINO REQUIRED rather than the used finders. I see that this find call is only used for some OV versions, I just pushed a fix that removes some complexity in build scrips. I believe that drops support for older OV's but not sure if that would still be needed.

ok thanks. let me loop in OV EP maintainers to review.

[jywu-msft]

@sfatimar , @preetha-intel , can you take a look at this? seems like @gedoensmax was having issues with the build without making this change. (I believe he's testing on Windows platform)

[jywu-msft]

/azp run Linux OpenVINO CI Pipeline

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[preetha-intel]

@gedoensmax The changes are already part of internal branch that contains changes for OV 2024.0. It will be merged into master around March 15th. Can it happen by then?

[gedoensmax]

@preetha-intel Yes sure, thanks for taking care !

[ilya-lavrenov]

Fixed by #19966