Skip to content

cmake: process modules in the same CMake scope#20446

Merged
opencv-pushbot merged 1 commit intoopencv:masterfrom
alalek:cmake_eliminate_modules_scope
Jul 31, 2021
Merged

cmake: process modules in the same CMake scope#20446
opencv-pushbot merged 1 commit intoopencv:masterfrom
alalek:cmake_eliminate_modules_scope

Conversation

@alalek
Copy link
Copy Markdown
Member

@alalek alalek commented Jul 22, 2021

Merge with contrib: opencv/opencv_contrib#3010

Provides better support of module's external dependencies.

Changed:

  • <module>cmake/init.cmake is processed in the scope of the root CMakeLists.txt

Other changes:

  • no modules/cmake_install.cmake script
  • TBD

Follow-up tasks:

  • (separate PR): FIXIT modules/highgui/cmake/init.cmake (move cmake/OpenCVFindLibsGUI.cmake content)
build_contrib:iOS=ON

Copy link
Copy Markdown
Contributor

@mshabunin mshabunin left a comment

Choose a reason for hiding this comment

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

👍

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.

3 participants