Skip to content

android: gradle build#13386

Merged
opencv-pushbot merged 3 commits intoopencv:masterfrom
alalek:android_gradle
Dec 21, 2018
Merged

android: gradle build#13386
opencv-pushbot merged 3 commits intoopencv:masterfrom
alalek:android_gradle

Conversation

@alalek
Copy link
Copy Markdown
Member

@alalek alalek commented Dec 7, 2018

resolves #8460

Usage (build Android package):

python2 <opencv_dir>/platforms/android/build_sdk.py --config ndk-17.config.py

Android package for testing is here.

TODO:

  • Update infrastructure
    • repaired test_cmake_build
    • added test_gradle (require test_gradle=1 property)
  • (postponed) Repair Android Java tests
  • Fix samples build within "make all" target
  • (postponed) OpenCV manager => OpenCV loader (debug only) or drop completelly?
    • new API to replace "initDebug()"
  • (postponed) update samples menu implementation ("context" menu is available in android 8 only - min version is 21/16)
test_modules=none
buildworker:Android pack=linux-4
docker_image:Android pack=android-gradle
buildworker:Android armeabi-v7a=linux-4
docker_image:Android armeabi-v7a=android-gradle
test_gradle=1
allow_multiple_commits=1

log.info("=====")

os.chdir(engdest)
builder.build_engine(abi, engdest)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

+1 for removing

@@ -303,7 +236,6 @@ def gather_results(self, engines):
parser.add_argument('--sign_with', help="Certificate to sign the Manager apk")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could you change
parser.add_argument('--config', default='ndk-10.config.py',
to?
parser.add_argument('--config', default='ndk-17.config.py',

@@ -303,7 +236,6 @@ def gather_results(self, engines):
parser.add_argument('--sign_with', help="Certificate to sign the Manager apk")
parser.add_argument('--build_doc', action="store_true", help="Build javadoc")
parser.add_argument('--no_ccache', action="store_true", help="Do not use ccache during library build")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could you disable ccache by default? (maybe change no_ccache to enable_ccache)
ccache in not available by default on Linux
ccache is not available on Windows

@crazyhappygame
Copy link
Copy Markdown

@alalek

  1. When will you merge this PR?
  2. When will you create clang build for android (opencv 3.4.4, 4.0) and add to https://opencv.org/releases.html ?

@alalek
Copy link
Copy Markdown
Member Author

alalek commented Dec 17, 2018

You can try Android package with this PR (see description above for the link)

@alalek
Copy link
Copy Markdown
Member Author

alalek commented Dec 21, 2018

👍

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.

SDK Tools, Revision 25.3.0 and OpenCVDetectAndroidSDK.cmake

3 participants