Skip to content

Conversation

@alalek
Copy link
Member

@alalek alalek commented Apr 18, 2019

#13917 from sturkmen72:removed_c_api
#14308 from m000:master
#14313 from alalek:cmake_pch_fix_defines_from_interface_targets
#14315 from dkurt:tf_squeeze_and_slim_softmax_v2
#14318 from dkurt:fix_ie_github
#14328 from lainy:3.4
#14332 from LaurentBerger:loadknearest
#14337 from alalek:ts_flush_stdout
#14338 from alalek:fix_13849
#14341 from alalek:cmake_fix_pkgconfig_libs
#14343 from alalek:dnn_static_scans_namespace
#14345 from dkurt:fix_dnn_ie_mac
#14347 from dkurt:dnn_samples
#14354 from alalek:workaround_apple_dyld_bug
#14366 from alalek:win32_fix_pdb

Previous "Merge 3.4": #14320

buildworker:Win64 OpenCL=windows-2
buildworker:Custom=linux-1,linux-2,linux-4
docker_image:Docs=docs-js
docker_image:Custom=javascript
#docker_image:Custom=powerpc64le
#docker_image:Custom=ubuntu-openvino:16.04
#buildworker:Custom=linux-2
#docker_image:Custom=ubuntu-vulkan:16.04
#buildworker:Custom=linux-4
#docker_image:Custom=fedora:28
#docker_image:Custom=ubuntu-cuda:16.04
buildworker:Mac OpenCL=macosx-1
build_image:Mac OpenCL=openvino-2019r1
test_modules:Mac OpenCL=dnn,java,python3
**WIP**

sturkmen72 and others added 29 commits March 3, 2019 21:43
process defines from INTERFACE targets
Adds IMPORTED for the TBB::tbb alias target as otherwise cmake produces
the following error:

CMake Error: install(EXPORT "OpenCVModules" ...) includes target "opencv_core" which requires target "tbb" that is not in the export set.

This problem occurs with the defaults used by homebrew on macOS, which
compiles both static and shared versions of TBB and OpenCV.
- drop legacy check_module() macro
- drop 'ALIASOF_'
@alalek
Copy link
Member Author

alalek commented Apr 19, 2019

👍

@alalek alalek merged commit c024edb into opencv:master Apr 19, 2019
@alalek alalek mentioned this pull request Apr 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants