Skip to content

Automatically set the correct OpenCV version in build.gradle#20795

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
JBamberger:gradle-dynamic-cv-version
Oct 2, 2021
Merged

Automatically set the correct OpenCV version in build.gradle#20795
opencv-pushbot merged 1 commit intoopencv:3.4from
JBamberger:gradle-dynamic-cv-version

Conversation

@JBamberger
Copy link
Copy Markdown
Contributor

The example in the build.gradle file of the Android OpenCV SDK about the CMAKE setup uses a hard-coded OpenCV version. Therefore, the example does not match the actual SDK version. This is fixed by using CMAKE to automatically insert the actual version number.

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • 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 other license that is incompatible with OpenCV
  • The PR is proposed to proper branch
  • There is reference to 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

Automatically sets the correct OpenCV version in the CMAKE example contained in the build.gradle
file of the Android SDK.
@JBamberger JBamberger force-pushed the gradle-dynamic-cv-version branch from 8bcbec0 to ef53a92 Compare October 2, 2021 14:08
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 for contribution 👍

@opencv-pushbot opencv-pushbot merged commit 1de2d5c into opencv:3.4 Oct 2, 2021
@alalek alalek mentioned this pull request Oct 2, 2021
@alalek alalek mentioned this pull request Oct 15, 2021
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