cmake: rework MAP_IMPORTED_CONFIG to prevent unintended build failures#10299
Conversation
Introduce CMake variable OPENCV_MAP_IMPORTED_CONFIG which controls this behaviour Added defaults for MSVS targets
|
@TobiasLangnerFUB @tfoote @pjreed @rokm @bblanchon |
|
@alalek I'm not able to test right now as I'm working on a different project. However, it looks very complicated, so I think it would be more reasonable to simply revert my commit. |
|
@alalek The patch works for my configuration. |
|
I'm also having success with this patch in my builds. |
|
My use case is building and using ROS packages that depend on OpenCV (my current environment is Ubuntu 16.04 with ROS Kinetic). I have OpenCV installed via ROS apt packages from their buildfarm (specifically This works for me. |
|
Thanks all for testing that! @bblanchon I tested this patch with MSVS CMake generator, Debug+Release OpenCV installation and |
|
👍 |
|
@vpisarev Hello from itSeez3D! |
Introduce CMake variable OPENCV_MAP_IMPORTED_CONFIG which controls this behaviour
Added defaults for MSVS targets
resolves #10105