Android Builds of OpenJDK Mobile project now available

[Ali Ebrahimi]

Hi,

On Thu, Feb 18, 2016 at 6:48 PM, Bob Vandette <bob.vandette at oracle.com>
wrote:

>
> On Feb 17, 2016, at 6:31 PM, Ali Ebrahimi <ali.ebrahimi1781 at gmail.com>
> wrote:
>
> Hi,
>
> On Thu, Feb 18, 2016 at 12:45 AM, Bob Vandette <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/
>>
>> 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.



>
> 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