A BUG: can not find "cxxabi.h"

[Oliver001]

There are maybe a mistake in file ./toolchains/android.toolchain.cmake at line 1039.
set( ANDROID_ABI_INCLUDE_DIRS "${ANDROID_CXX_ROOT}/llvm-libc++abi/include" )

and I find the correct dir of ANDROID_ABI_INCLUDE_DIRS is "${ANDROID_CXX_ROOT}/llvm-libc++abi/libcxxabi/include"
so the line 1039 must be set( ANDROID_ABI_INCLUDE_DIRS "${ANDROID_CXX_ROOT}/llvm-libc++abi/libcxxabi/include" )

[matteblair]

Good eye. Did you encounter an error related to this while building the library in a particular configuration?

[Oliver001]

I don't think it happens in a particular configuration. In fact, the file cxxabi.h in the path ${ANDROID_CXX_ROOT}/llvm-libc++abi/libcxxabi/include correctly.

[matteblair]

So it sounds like this isn't actually causing a problem with the build process. Is that correct?

The android.toolchain.cmake file was written by another open-source project, not by us, so I don't want to modify it unless we are very confident that it is causing a problem.

[Oliver001]

It's up to you.
I find there are two guys tried add support for libc++, in this page taka-no-me/android-cmake#36, you can find the line 1039 in the second 'abergmeier added a note on Mar 4, 2015'.
And the other try in this page, taka-no-me/android-cmake@2959905 line 1043.

Can you tell me the android.toolchain.cmake file was written by which open-source project?

[karimnaaji]

Hi @Oliver001 , I remember adding the support for c++ shared a while back 67f0411 (and the cmake file was originally taken from the project you mentioned).

Unfortunately, this part might not work for all the android NDKs since it's highly adapted for our needs, we will be more specific in our README and specify that we currently develop under android-r10e NDK (https://github.com/tangrams/tangram-es#android).

We are also waiting for the new android studio which has CMake support before investing time in adapting this toolchain for all configurations (Refer #807).

[matteblair]

We got the toolchain file from https://github.com/taka-no-me/android-cmake but then had to make some additions and corrections, and actually that line you pointed out was one of the things we added! After realizing that and testing the include path, I went ahead and made the change you suggested: b979ac7

[Oliver001]

Hi @karimnaaji, the new stable android studio which has CMake support maybe release after two month or later.
Thanks.