Skip to content

Find only linkable IE libraries, not plugins#19199

Merged
opencv-pushbot merged 1 commit intoopencv:masterfrom
ilya-lavrenov:find-shared-libraries-only
Dec 23, 2020
Merged

Find only linkable IE libraries, not plugins#19199
opencv-pushbot merged 1 commit intoopencv:masterfrom
ilya-lavrenov:find-shared-libraries-only

Conversation

@ilya-lavrenov
Copy link
Copy Markdown
Contributor

In this change some libraries like inference_engine_ir_reader become plugins and MUST NOT be found by OpenCV cmake since OpenCV tries to link them to opencv_dnn, while plugins (.so on OSX) cannot be linked to other targets.

Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change doesn't look right.

Can we finally use find_package(InferenceEngine) + find_package(ngraph) instead of these hacks? (and remove these hacks completely)
What is still missing to support that?

@ilya-lavrenov ilya-lavrenov force-pushed the find-shared-libraries-only branch from d2f4a29 to 3ed376d Compare December 23, 2020 06:48
@ilya-lavrenov
Copy link
Copy Markdown
Contributor Author

What is still missing to support that?

I suppose need to add a version to InferenceEngineConfig.cmake, but it's not a blocker.
Everything works with find_package on my machine. But CI still passed variables INF_ENGINE_LIB_DIRS which enables old school way to find IE inside OpenCV, while we need to pass path to InferenceEngine_DIR

CC @asuhov can we update CI variables to use find package?

@ilya-lavrenov ilya-lavrenov requested a review from alalek December 23, 2020 06:54
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you 👍

@opencv-pushbot opencv-pushbot merged commit ed00088 into opencv:master Dec 23, 2020
@ilya-lavrenov ilya-lavrenov deleted the find-shared-libraries-only branch December 23, 2020 08:57
@ilya-lavrenov ilya-lavrenov changed the title Find only shared IE libraries, not plugins Find only linkable IE libraries, not plugins Dec 23, 2020
@alalek alalek mentioned this pull request Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants