Skip to content

cmake: fix handling of INF_ENGINE_RELEASE#20700

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
alalek:cmake_fix_handling_ie_version
Sep 14, 2021
Merged

cmake: fix handling of INF_ENGINE_RELEASE#20700
opencv-pushbot merged 1 commit intoopencv:3.4from
alalek:cmake_fix_handling_ie_version

Conversation

@alalek
Copy link
Copy Markdown
Member

@alalek alalek commented Sep 14, 2021

  • default value should be handled earlier

Avoid messages like these:

CMake Warning at cmake/OpenCVDetectInferenceEngine.cmake:82 (message):
  OpenMP for IE have not been found.  Set INF_ENGINE_OMP_DIR variable if you
  experience build errors.
Call Stack (most recent call first):
  cmake/OpenCVDetectInferenceEngine.cmake:110 (add_custom_ie_build)
  CMakeLists.txt:796 (include)


CMake Warning at cmake/OpenCVDetectInferenceEngine.cmake:144 (message):
  InferenceEngine version has not been set, 2021.4.1 will be used by default.
  Set INF_ENGINE_RELEASE variable if you experience build errors.
Call Stack (most recent call first):
  CMakeLists.txt:796 (include)

(caused by wrong handling order of INF_ENGINE_RELEASE)

force_builders=Custom Win
build_image:Custom Win=winpack-dldt

build_image:Custom=ubuntu-openvino-2021.2.0:20.04
Xbuild_image:Custom Win=openvino-2021.2.0
build_image:Custom Mac=openvino-2021.2.0

test_modules:Custom=dnn,python2,python3,java
Xtest_modules:Custom Win=dnn,python2,python3,java
test_modules:Custom Mac=dnn,python2,python3,java

buildworker:Custom=linux-1
# disabled due high memory usage: test_opencl:Custom=ON
test_opencl:Custom=OFF
test_bigdata:Custom=1
test_filter:Custom=*

- default value should be handled earlier
@alalek
Copy link
Copy Markdown
Member Author

alalek commented Sep 14, 2021

👍

@opencv-pushbot opencv-pushbot merged commit 6fa63dc into opencv:3.4 Sep 14, 2021
@alalek alalek mentioned this pull request Sep 25, 2021
@alalek alalek mentioned this pull request Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants