Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

klatism
Copy link
Contributor

@klatism klatism commented Jul 10, 2022

This PR enables building dynamically linked opencv libs using python build scripts. Shared object for Android and framework for iOS.
On Android building SO was already possible before this PR, now it is exposed in build_sdk.py as a command line parameter.
On iOS the option was already available as a command line argument, but was doing nothing for iOS simulator or device build. This PR enables that build providing it with the required dependencies.

Pull Request Readiness Checklist

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

@klatism
Copy link
Contributor Author

klatism commented Jul 18, 2022

Hi @alalek, thanks for reviewing this. Looks like the tests passed. Can this PR be merged or is there anything missing?

@asmorkalov asmorkalov self-requested a review August 2, 2022 06:29
@asmorkalov asmorkalov self-assigned this Aug 2, 2022
Copy link
Contributor

@asmorkalov asmorkalov left a comment

Choose a reason for hiding this comment

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

👍 LGTM

@opencv-pushbot opencv-pushbot merged commit 50707a2 into opencv:4.x Aug 2, 2022
@klatism klatism deleted the mobile-dynamically-linked-libs branch August 2, 2022 18:33
@alalek alalek mentioned this pull request Aug 21, 2022
@asmorkalov asmorkalov added this to the 4.7.0 milestone Jan 23, 2023
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.

4 participants