Openembedded Devel Discussions
 help / color / mirror / Atom feed
From: Khem Raj <raj.khem@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH 1/4] openjdk: use qemu now for the awt sizer binary too
Date: Wed, 16 Mar 2011 10:45:27 -0700	[thread overview]
Message-ID: <4D80F737.70906@gmail.com> (raw)
In-Reply-To: <1300272884-24220-2-git-send-email-heinold@inf.fu-berlin.de>

On 3/16/2011 3:54 AM, Henning Heinold wrote:
> From: Henning Heinold<h.heinold@tarent.de>
>
> * bump INC_PR
> ---
>   .../icedtea-crosscompile-fix.patch                 |   24 ++++++++++---------
>   recipes/openjdk/openjdk-6-common.inc               |    8 +-----
>   2 files changed, 14 insertions(+), 18 deletions(-)
>

the inc file has
DEPENDS_append_arm = " qemu-native "
so I guess it will work fine for arm how about other architectures ?
otherwise patch looks ok

> diff --git a/recipes/openjdk/openjdk-6-6b18-1.8/icedtea-crosscompile-fix.patch b/recipes/openjdk/openjdk-6-6b18-1.8/icedtea-crosscompile-fix.patch
> index ec0f259..652f7ac 100644
> --- a/recipes/openjdk/openjdk-6-6b18-1.8/icedtea-crosscompile-fix.patch
> +++ b/recipes/openjdk/openjdk-6-6b18-1.8/icedtea-crosscompile-fix.patch
> @@ -73,17 +73,6 @@ Index: openjdk/jdk/make/sun/xawt/Makefile
>    endif
>
>    ifeq ($(PLATFORM), solaris)
> -@@ -237,7 +238,9 @@
> - 	$(BOOT_JAVA_CMD) -cp $(WRAPPER_GENERATOR_TEMPDIR) WrapperGenerator \
> - 	    $(SIZER_DIR) $(XLIBTYPES) "sizer" $(subst .,,$(suffix $(basename $@)))
> -
> --$(SIZES): $(SIZERS)
> -+#$(SIZES): $(SIZERS)
> -+# Prevent binaries being created. Instead take what is available.
> -+$(SIZES):
> - 	@if [ "$(DOHACK)$@" = "true$(PREDEFINED_SIZES)" ]; then \
> - 	    $(ECHO) COPYING $(PREDEFINED_SIZES_TMPL) into $@; \
> - 	    $(CP) $(PREDEFINED_SIZES_TMPL) $@; \
>   Index: openjdk/hotspot/make/linux/makefiles/vm.make
>   ===================================================================
>   --- openjdk.orig/hotspot/make/linux/makefiles/vm.make	2009-01-24 17:54:57.000000000 +0100
> @@ -212,3 +201,16 @@ Index: openjdk/hotspot/make/linux/makefiles/launcher.make
>      LIBS_LAUNCHER             += -l$(JVM) $(LIBS) -lstdc++
>    endif
>
> +Index: icedtea6-1.8.5/build/openjdk/jdk/make/sun/xawt/Makefile
> +===================================================================
> +--- openjdk/jdk/make/sun/xawt/Makefile	2011-03-14 14:17:03.926207269 +0100
> ++++ openjdk/jdk/make/sun/xawt/Makefile	2011-03-14 14:30:11.956186631 +0100
> +@@ -245,7 +245,7 @@
> + 	    $(CHMOD) +w $@;\
> + 	else	\
> + 	    $(ECHO) GENERATING $@; \
> +-	    $(WRAPPER_GENERATOR_DIR)/sizer$(suffix $@)>  $@; \
> ++	    $(QEMU) $(WRAPPER_GENERATOR_DIR)/sizer$(suffix $@)>  $@; \
> + 	fi
> + 	@if [ "$(DOCOMPARE)$(suffix $@)" = "true.64" ]; then \
> + 	    $(ECHO) COMPARING $@ and $(STORED_SIZES_TMPL_$(PLATFORM)_$(LIBARCH)); \
> diff --git a/recipes/openjdk/openjdk-6-common.inc b/recipes/openjdk/openjdk-6-common.inc
> index f04800b..54c45f8 100644
> --- a/recipes/openjdk/openjdk-6-common.inc
> +++ b/recipes/openjdk/openjdk-6-common.inc
> @@ -10,7 +10,7 @@ ICEDTEA = "NEEDS TO BE SET"
>   S = "${WORKDIR}/${ICEDTEA}"
>   B = "${S}/build"
>
> -INC_PR = "r10"
> +INC_PR = "r11"
>
>   SRC_URI = "\
>     ${ICEDTEA_URI} \
> @@ -288,12 +288,6 @@ addtask compilestepone after do_configure before do_compilesteptwo
>   do_compilesteptwo() {
>   	echo "2/3 Building bootstrap JDK"
>
> -	# Take pre-generated binaries from -native build
> -	install -d generated/sun/awt/X11/generator
> -	ln -sf ${STAGING_LIBDIR_JVM_NATIVE}/icedtea6-native/generated/sizer.32 generated/sun/awt/X11/generator/sizer.32
> -	install -d openjdk/${BUILD_DIR}/gensrc/sun/awt/X11/generator
> -	ln -sf ${STAGING_LIBDIR_JVM_NATIVE}/icedtea6-native/generated/sizer.32 openjdk/${BUILD_DIR}/gensrc/sun/awt/X11/generator/sizer.32
> -
>   	# Now cross-compile bootstrap JDK
>   	rm stamps/icedtea-against-ecj.stamp stamps/icedtea-ecj.stamp || true
>   	oe_runmake icedtea-against-ecj




  reply	other threads:[~2011-03-16 17:47 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-16 10:54 updating java Henning Heinold
2011-03-16 10:54 ` [PATCH 1/4] openjdk: use qemu now for the awt sizer binary too Henning Heinold
2011-03-16 17:45   ` Khem Raj [this message]
2011-03-16 18:06     ` Henning Heinold
2011-03-16 10:54 ` [PATCH 2/4] cacao-hg: update to latest revision Henning Heinold
2011-03-16 17:46   ` Khem Raj
2011-03-16 10:54 ` [PATCH 3/4] icedtea6-native: update to 1.7.10 version Henning Heinold
2011-03-16 11:11   ` Koen Kooi
2011-03-16 10:54 ` [PATCH 4/4] angstrom-jalimo: use cacao-native hg version for javac-native Henning Heinold
2011-03-16 11:12   ` Koen Kooi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4D80F737.70906@gmail.com \
    --to=raj.khem@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox