[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-distro-pkg-dev
Subject:    [PATCH REVIEW] IcedTea7 2.0 Backports
From:       ptisnovs () redhat ! com (Pavel Tisnovsky)
Date:       2011-09-30 16:04:32
Message-ID: 4E85E890.4040209 () redhat ! com
[Download RAW message or body]

Hi Andrew,

these patches look ok. Thank you.

Cheers,
Pavel

Dr Andrew John Hughes wrote:
> A couple of issues have come up with building IcedTea7 using itself (full
> bootstrap) and not requiring a new enough version of Glib for --enable-system-gio.
> 
> The fixes are:
> 
> For building with IcedTea7:
> 
> http://icedtea.classpath.org/hg/icedtea7/rev/f75092cac189
> Only build bootstrap classes with source/target 1.6 if the VM doesn't support 1.7 bytecode.
> 
> 2011-09-30 Andrew John Hughes <ahughes at redhat.com>
> 
> * Makefile.am:
> (ICEDTEA_BOOT_PATCHES): Only apply
> use_target_6_for_bootstrap_classes if 1.7
> bytecode is not supported by the VM. Fixes
> bootstrap with IcedTea7.
> * acinclude.m4:
> (IT_BYTECODE7_CHECK): Check if the VM supports
> 1.7 bytecode.
> * configure.ac: Call IT_BYTECODE7_CHECK.
> 
> http://icedtea.classpath.org/hg/icedtea7/rev/15866ac3f41b
> Turn off forced -Werror when building com.sun.java.pack
> 
> 2011-09-30 Andrew John Hughes <ahughes at redhat.com>
> 
> Bump changesets to bring in fix which
> turns off forced -Werror.
> * Makefile.am:
> (CORBA_CHANGESET): Updated.
> (HOTSPOT_CHANGESET): Likewise.
> (JAXP_CHANGESET): Likewise.
> (JAXWS_CHANGESET): Likewise.
> (JDK_CHANGESET): Likewise.
> (LANGTOOLS_CHANGESET): Likewise.
> (OPENJDK_CHANGESET): Likewise.
> (CORBA_SHA256SUM): Likewise.
> (HOTSPOT_SHA256SUM): Likewise.
> (JAXP_SHA256SUM): Likewise.
> (JAXWS_SHA256SUM): Likewise.
> (JDK_SHA256SUM): Likewise.
> (LANGTOOLS_SHA256SUM): Likewise.
> (OPENJDK_SHA256SUM): Likewise.
> 
> This also requires bringing http://icedtea.classpath.org/hg/icedtea7-forest/jdk/rev/d9fca71ba183
> into the 2.0 forest:
> 
> --- a/make/com/sun/java/pack/Makefile	Wed Sep 28 23:13:29 2011 +0100
> +++ b/make/com/sun/java/pack/Makefile	Fri Sep 30 05:30:03 2011 +0100
> @@ -33,7 +33,6 @@
>  PRODUCT = sun
>  PGRM = unpack200
>  JAVAC_MAX_WARNINGS=true
> -JAVAC_WARNINGS_FATAL=true
>  include $(BUILDDIR)/common/Defs.gmk
>   
>  CPLUSPLUSLIBRARY=true
> 
> And finally the issue caught by Pavel and Mark today, requiring GIO/Glib 2.26 for GSettings:
> 
> http://icedtea.classpath.org/hg/icedtea7/rev/fbad82bc6985
> Check for GIO >= 2.26 for GSettings.
> 
> 2011-09-30 Andrew John Hughes <ahughes at redhat.com>
> 
> * acinclude.m4:
> (IT_CHECK_FOR_GIO): Require 2.26 and
> above for GSettings.
> 
> Are these ok for 2.0?
> 
> Patches attached.
> 
> Thanks,
> 


[prev in list] [next in list] [prev in thread] [next in thread]