Changes between Initial Version and Version 2 of Ticket #1968


Ignore:
Timestamp:
Jul 5, 2017 4:36:08 AM (7 years ago)
Author:
riza
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #1968

    • Property Status changed from new to closed
    • Property Resolution changed from to fixed

  • Ticket #1968 – Description

    initial v2  
    1 Starting r13, Android NDK has set NDK_TOOLCHAIN_VERSION default value to Clang. 
     1Starting `NDK r13`, Android NDK has set NDK_TOOLCHAIN_VERSION default value to Clang. 
    22This will result the `configure-android` to fail with this error: 
    33{{{ 
    … …  
    66}}} 
    77 
    8 Furthermore, r13 release notes states that: 
     8Furthermore, `r13` release notes states that: 
    99{{{ 
    1010GCC is no longer supported. It will not be removed from the NDK just yet,  
    … …  
    1414It will likely be removed after that point. 
    1515}}} 
    16 So, modification to `configure-android` which expects gcc would be needed. 
     16So, the same issue should be expected on `r14` and later. 
    1717 
    18 At the moment (on r13), you can force the ndk to use gcc by specifying the NDK_TOOLCHAIN_VERSION to 4.9. 
     18At the moment (on `r13`), you can force the ndk to use gcc by specifying the NDK_TOOLCHAIN_VERSION to 4.9. 
    1919{{{ 
    2020e.g: 
    … …  
    2222NDK_TOOLCHAIN_VERSION=4.9 TARGET_ABI=armeabi-v7a ./configure-android --use-ndk-cflags 
    2323}}} 
     24 
     25 
     26Update: 
     27 
     28Since r5620. `configure-android` have been modified to support `clang`, hence user wouldn't need to specify `NDK_TOOLCHAIN_VERSION`. 
     29 
     30Known Issue when building with `clang`: 
     31 
     32When building for older android device, it is needed to specify `APP_PLATFORM`. 
     33{{{ 
     34e.g: APP_PLATFORM=android-17 
     35}}} 
     36 
     37The build might fail with these error: 
     38{{{ 
     39In file included from ../../yuv/source/compare.cc:14: 
     40/app/android-ndk-r13b//sources/cxx-stl/llvm-libc++/include/math.h:661:91: error: use of undeclared identifier 'acosl' 
     41inline _LIBCPP_INLINE_VISIBILITY long double acos(long double __lcpp_x) _NOEXCEPT {return acosl(__lcpp_x);} 
     42                                                                                          ^ 
     43/app/android-ndk-r13b//sources/cxx-stl/llvm-libc++/include/math.h:673:91: error: use of undeclared identifier 'asinl' 
     44inline _LIBCPP_INLINE_VISIBILITY long double asin(long double __lcpp_x) _NOEXCEPT {return asinl(__lcpp_x);} 
     45}}} 
     46 
     47The error is described in detail in [https://social.msdn.microsoft.com/Forums/en-US/4105609b-8236-49c6-9139-27a1ae7c72ab/visual-studio-2015-android-ndk-with-libc-and-cmath-issues?forum=vcgeneral here]. 
     48 
     49To handle it user need to include `$(NDKRoot)/sources/android/support/include` in the `CFLAGS` (user.mak).