Patchwork [meta-java] Building openjdk-7-jre for PPC

login
register
mail settings
Submitter Rudolf Streif
Date Nov. 15, 2013, 6:39 p.m.
Message ID <CA+gsK-reC5jKTGB=4nPqWR5i0a_+MRQRBj9gbBXz6E8gc6OpdQ@mail.gmail.com>
Download mbox | patch
Permalink /patch/61825/
State New, archived
Headers show

Comments

Rudolf Streif - Nov. 15, 2013, 6:39 p.m.
While building openjdk-7-jre 25b30 for PPC architecture with dylan I ran
into the following issues:

The patch icedtea-jdk-nio-use-host-cc.patch is conditionally applied to PPC
only. It also requires this patch of the patch [1]. This patch has not yet
been included with meta-java although a pull request was sent a while ago.

Furthermore, the original  icedtea-jdk-nio-use-host-cc.patch includes this
change of the Sun toplevel makefile:

and other Java classes not to be build and also not to be included into
rt.jar. Hence, applications using awt UI classes will break with a "class
not found" error.

I reverted the change and now it builds correctly, but I would like to
understand why this change was made in the first place.


Cheers,
Rudi

[1]
https://github.com/kraj/meta-java/commit/22983b1ab80b3393ecc27e5c3d3a9b52e3b8e692
Khem Raj - Nov. 15, 2013, 7:40 p.m.
On Nov 15, 2013, at 10:39 AM, Rudolf Streif <rudolf.streif@linux.com> wrote:

> While building openjdk-7-jre 25b30 for PPC architecture with dylan I ran
> into the following issues:
> 
> The patch icedtea-jdk-nio-use-host-cc.patch is conditionally applied to PPC
> only. It also requires this patch of the patch [1]. This patch has not yet
> been included with meta-java although a pull request was sent a while ago.
> 
> Furthermore, the original  icedtea-jdk-nio-use-host-cc.patch includes this
> change of the Sun toplevel makefile:
> 
> Index: openjdk/jdk/make/sun/Makefile
> ===================================================================
> --- openjdk/jdk/make/sun/Makefile       2013-07-25 09:10:09.000000000 -0700
> +++ openjdk/jdk/make/sun/Makefile       2013-10-01 21:32:01.625839149 -0700
> @@ -55,7 +55,7 @@
>     endif
>   endif
>   HEADLESS_SUBDIR = headless
> -  XAWT_SUBDIR     = xawt
> +  XAWT_SUBDIR     =
> endif
> 
> This removes the xawt subdir from the build process resulting in XToolkit
> and other Java classes not to be build and also not to be included into
> rt.jar. Hence, applications using awt UI classes will break with a "class
> not found" error.
> 
> I reverted the change and now it builds correctly, but I would like to
> understand why this change was made in the first place.

xawt was not buildable under ppc a while back, we have updated the revs since 
then, this was best case available

> 
> 
> Cheers,
> Rudi
> 
> [1]
> https://github.com/kraj/meta-java/commit/22983b1ab80b3393ecc27e5c3d3a9b52e3b8e692
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Patch

Index: openjdk/jdk/make/sun/Makefile
===================================================================
--- openjdk/jdk/make/sun/Makefile       2013-07-25 09:10:09.000000000 -0700
+++ openjdk/jdk/make/sun/Makefile       2013-10-01 21:32:01.625839149 -0700
@@ -55,7 +55,7 @@ 
     endif
   endif
   HEADLESS_SUBDIR = headless
-  XAWT_SUBDIR     = xawt
+  XAWT_SUBDIR     =
 endif

This removes the xawt subdir from the build process resulting in XToolkit