Android Builds of OpenJDK Mobile project now available

[Bob Vandette]

> On Feb 18, 2016, at 11:03 AM, Ali Ebrahimi <ali.ebrahimi1781 at gmail.com> wrote:
> 
> Hi,
> 
> On Thu, Feb 18, 2016 at 6:48 PM, Bob Vandette <bob.vandette at oracle.com <mailto:bob.vandette at oracle.com>> wrote:
> 
>> On Feb 17, 2016, at 6:31 PM, Ali Ebrahimi <ali.ebrahimi1781 at gmail.com <mailto:ali.ebrahimi1781 at gmail.com>> wrote:
>> 
>> Hi,
>> 
>> On Thu, Feb 18, 2016 at 12:45 AM, Bob Vandette <bob.vandette at oracle.com <mailto:bob.vandette at oracle.com>> wrote:
>> It appears that I was correct that the NDK has improved in the latest version wrt.
>> source level compatibility with the standard Linux distros and toolchains.
>> 
>> Here’s a webrev and patch that corrects compilation errors and allows both ARM and x86 Android
>> builds using the latest NDK r10e.  
>> 
>> http://cr.openjdk.java.net/~bobv/8150074/ <http://cr.openjdk.java.net/~bobv/8150074/>
>> 
>> We should inspect all #ifdef/#ifndef __ANDROID__ uses throughout the sources
>> to see if we can remove more unnecessary logic.
>> 
>> I can confirm your patch does not break any thing so far for me.
> Did you re-apply this patch on top of a clean set of sources cloned from mobile/dev?
> Yes,
> Of course, in following file
> +++ new/hotspot/src/share/vm/utilities/globalDefinitions_gcc.hpp	2016-02-17 16:04:26.312126636 -0500
> @@ -74,25 +74,10 @@
>  #endif // __STDC_LIMIT_MACROS
>  #include <inttypes.h>
>  #include <signal.h>
> -#ifdef __ANDROID__
> -#include <asm/sigcontext.h>
> ...
>  
>  #ifndef __OpenBSD__
> -#ifndef __ANDROID__
>  #include <ucontext.h>
> -#endif
> +//#endif  <================== we can remove #endif
> 
>  #endif
> 
> Here, we can remove #endif (instead of commenting ) to improve code cleanness.

Yes, I found this and had already fixed it.

Thanks,
Bob.

> 
>  
> 
>> So we should update build requirements for NDK r10e.
> Once we integrate this patch, I will update the build requirements and instructions.
> 
> 
> -- 
> 
> Best Regards,
> Ali Ebrahimi