Problems building for Android version (Patch #1781)

Added by Hee Seo almost 13 years ago. Updated almost 13 years ago.

Status:Done Start date:2012-04-09
Priority:Normal Due date:
Assignee:Andrey Kamaev % Done:

0%
Category:build/install
Target version:2.4.0
Affected version: Operating System:
Difficulty: HW Platform:
Pull request:

Description

Hello, everyone
I've been first time to join the code.OpenCV.org.

I've checked out the HEAD code of subversion of OpenCV.
: svn co http://code.opencv.org/svn/opencv/trunk/opencv

I guess there's a problem that the source code is not compiled rightly.

The cause seems to be the error for the Cmake syntax like this :
http://code.opencv.org/svn/opencv/trunk/opencv/cmake/OpenCVDetectAndroidSDK.cmake

When the Cmake calls a macro function named* 'get_compatible_android_api_level'* to search for compatible API levels,
The $(ANDROID_SDK_TARGETS}'s value must be android-n ( e.g. android-14 or android-15 ).

However a part of get_compatible_android_api_level doesn't handle $(ANDROID_SDK_TARGETS}' correctly like the attached patch file

Please check the attached patch and what I mentioned.
Thanks!

OpenCVDetectAndroidSDK.cmake.patch - This file is a patch file to fix the error I mentioned (512 Bytes) Hee Seo, 2012-04-09 04:37 pm

__-patch-opencv-trunk-opencv-android__수정됨__1__.png - This file is a screen shot for a compile error. (23.4 kB) Hee Seo, 2012-04-09 04:37 pm

Associated revisions

Revision f7853b26
Added by Andrey Kamaev almost 13 years ago

Fixed cmake configuration for Android for case when only one SDK target is available #1781

Revision f461930d
Added by Roman Donchenko over 11 years ago

Merge pull request #1781 from SpecLad:merge-2.4

History

#1
Updated by Andrey Kamaev almost 13 years ago

Thanks for reporting.

But your patch is not correct. There was just an attempt to access list with wrong element index.
Problem is fixed in 2.4 branch. Fix will be ported to trunk after the OpenCV 2.4 release.

  • Status changed from Open to Done
  • Target version changed from 3.0 to 2.4.0
  • Assignee changed from Alexander Shishkov to Andrey Kamaev

#2
Updated by Hee Seo almost 13 years ago

Thank for your feedback. ;-)

Also available in: Atom PDF