From 8dabf5ac147a0d224a03ad9534915e387866c3ff Mon Sep 17 00:00:00 2001 From: Jonathan Pryor Date: Wed, 15 Jun 2016 08:59:21 -0400 Subject: [PATCH] [mono-runtimes, monodroid] Fix Mono runtime building, dependencies Commit 6bb1016 "broke" `build-tools/mono-runtimes`, in that it didn't build any Android target JIT ABIs, because it used the wrong MSBuild property for the condition checks -- it used `$(AndroidSupportedTargetJitAbis)`, not `$(AndroidSupportedTargetJitAbisForConditionalChecks)`. Doh! Fix that. Additionally, `src/monodroid` needs to depend upon `build-tools/mono-runtimes` because it depends upon eglib-config.h, which is generated as part of the mono build. `src/monodroid` doens't depend upon the generated `libmonosgen-2.0.so` file, but it *does* depend upon the build tree. --- build-tools/mono-runtimes/mono-runtimes.projitems | 10 +++++----- src/monodroid/monodroid.mdproj | 5 +++++ 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/build-tools/mono-runtimes/mono-runtimes.projitems b/build-tools/mono-runtimes/mono-runtimes.projitems index ae791652f72..89484f85f20 100644 --- a/build-tools/mono-runtimes/mono-runtimes.projitems +++ b/build-tools/mono-runtimes/mono-runtimes.projitems @@ -1,7 +1,7 @@ - <_MonoRuntime Include="armeabi" Condition="$(AndroidSupportedTargetJitAbis.Contains (':armeabi:'))"> + <_MonoRuntime Include="armeabi" Condition="$(AndroidSupportedTargetJitAbisForConditionalChecks.Contains (':armeabi:'))"> $(_ArmAr) $(_ArmAs) $(_ArmCc) @@ -21,7 +21,7 @@ libmono-profiler-log libMonoPosixHelper - <_MonoRuntime Include="armeabi-v7a" Condition="$(AndroidSupportedTargetJitAbis.Contains (':armeabi-v7a:'))"> + <_MonoRuntime Include="armeabi-v7a" Condition="$(AndroidSupportedTargetJitAbisForConditionalChecks.Contains (':armeabi-v7a:'))"> $(_ArmAr) $(_ArmAs) $(_ArmCc) @@ -41,7 +41,7 @@ libmono-profiler-log libMonoPosixHelper - <_MonoRuntime Include="arm64-v8a" Condition="$(AndroidSupportedTargetJitAbis.Contains (':arm64-v8a:'))"> + <_MonoRuntime Include="arm64-v8a" Condition="$(AndroidSupportedTargetJitAbisForConditionalChecks.Contains (':arm64-v8a:'))"> $(_Arm64Ar) $(_Arm64As) $(_Arm64Cc) @@ -61,7 +61,7 @@ libmono-profiler-log libMonoPosixHelper - <_MonoRuntime Include="x86" Condition="$(AndroidSupportedTargetJitAbis.Contains (':x86:'))"> + <_MonoRuntime Include="x86" Condition="$(AndroidSupportedTargetJitAbisForConditionalChecks.Contains (':x86:'))"> $(_X86Ar) $(_X86As) $(_X86Cc) @@ -81,7 +81,7 @@ libmono-profiler-log libMonoPosixHelper - <_MonoRuntime Include="x86_64" Condition="$(AndroidSupportedTargetJitAbis.Contains (':x86_64:'))"> + <_MonoRuntime Include="x86_64" Condition="$(AndroidSupportedTargetJitAbisForConditionalChecks.Contains (':x86_64:'))"> $(_X86_64Ar) $(_X86_64As) $(_X86_64Cc) diff --git a/src/monodroid/monodroid.mdproj b/src/monodroid/monodroid.mdproj index 928ab2fa380..3f7cdbdb956 100644 --- a/src/monodroid/monodroid.mdproj +++ b/src/monodroid/monodroid.mdproj @@ -27,5 +27,10 @@ android-toolchain False + + {C03E6CF1-7460-4CDC-A4AB-292BBC0F61F2} + mono-runtimes + False +