Re: [cmake-developers] Bug with CMAKE_ASM_COMPILER and Android NDK

2017-06-26 Thread Brad King
On 06/26/2017 03:06 PM, Robert Dailey wrote: > Are you saying I should modify my local installation? Or are these > your personal notes so that these changes get adopted upstream? These are notes for anyone that wants to fix this in CMake by doing development. I don't know if I'll find time

Re: [cmake-developers] Bug with CMAKE_ASM_COMPILER and Android NDK

2017-06-26 Thread Robert Dailey
On Mon, Jun 26, 2017 at 10:25 AM, Brad King wrote: > On 06/23/2017 03:18 PM, Robert Dailey wrote: >> enable_language(ASM) >> >> CMake appears to find clang.exe in the wrong place > > Add a `Modules/Platform/Android-Determine-ASM.cmake` module > based on

Re: [cmake-developers] Bug with CMAKE_ASM_COMPILER and Android NDK

2017-06-26 Thread Brad King
On 06/23/2017 03:18 PM, Robert Dailey wrote: > enable_language(ASM) > > CMake appears to find clang.exe in the wrong place Add a `Modules/Platform/Android-Determine-ASM.cmake` module based on `Modules/Platform/Android-Determine-CXX.cmake`. Also the modules

Re: [cmake-developers] Bug with CMAKE_ASM_COMPILER and Android NDK

2017-06-23 Thread Robert Dailey
Oh I'm using CMake 3.9.0 RC3 On Fri, Jun 23, 2017 at 2:18 PM, Robert Dailey wrote: > So I have two environment variables: > > ANDROID_NDK = E:\android\ndk > ANDROID_NDK_72 = E:\android\ndk_72 > > In my toolchain file, I do this: > > > > set( CMAKE_SYSTEM_NAME Android )

[cmake-developers] Bug with CMAKE_ASM_COMPILER and Android NDK

2017-06-23 Thread Robert Dailey
So I have two environment variables: ANDROID_NDK = E:\android\ndk ANDROID_NDK_72 = E:\android\ndk_72 In my toolchain file, I do this: set( CMAKE_SYSTEM_NAME Android ) set( CMAKE_SYSTEM_VERSION 15 ) # API level set( CMAKE_ANDROID_ARCH_ABI armeabi-v7a ) set( CMAKE_ANDROID_STL_TYPE c++_shared )