qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Stefan Weil <sw@weilnetz.de>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 2/6] pixman: pass tools to configure
Date: Tue, 13 Nov 2012 18:50:14 +0100	[thread overview]
Message-ID: <50A28856.6040502@weilnetz.de> (raw)
In-Reply-To: <1352796133-956-3-git-send-email-kraxel@redhat.com>

Am 13.11.2012 09:42, schrieb Gerd Hoffmann:
> Complete the set of compiler tool names in config-host.mak,
> pass them to pixman configure to make sure cross builds work.
> Based on a patch from Blue Swirl.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>   Makefile  |    2 +-
>   configure |    4 ++++
>   2 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index ca14a21..7b267c9 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -122,7 +122,7 @@ subdir-pixman: pixman/Makefile
>   	$(call quiet-command,$(MAKE) $(SUBDIR_MAKEFLAGS) -C pixman V="$(V)" all,)
>   
>   pixman/Makefile: $(SRC_PATH)/pixman/configure
> -	(cd pixman; $(SRC_PATH)/pixman/configure --disable-shared --enable-static)
> +	(cd pixman; CC="$(CC)" LD="$(LD)" AR="$(AR)" NM="$(NM)" RANLIB="$(RANLIB)" $(SRC_PATH)/pixman/configure --disable-shared --enable-static)

An alternate solution (which works in my build environment) is passing
--host=... (value derived from QEMU cross_prefix) to the pixman configure.


>   
>   $(SRC_PATH)/pixman/configure:
>   	(cd $(SRC_PATH)/pixman; autoreconf -v --install)
> diff --git a/configure b/configure
> index 18faded..e7ca78b 100755
> --- a/configure
> +++ b/configure
> @@ -252,8 +252,10 @@ done
>   
>   cc="${CC-${cross_prefix}gcc}"
>   ar="${AR-${cross_prefix}ar}"
> +nm="${AR-${cross_prefix}nm}"

NM instead of AR?

>   objcopy="${OBJCOPY-${cross_prefix}objcopy}"
>   ld="${LD-${cross_prefix}ld}"
> +ranlib="${LD-${cross_prefix}ranlib}"

RANLIB instead of LD?

>   libtool="${LIBTOOL-${cross_prefix}libtool}"
>   strip="${STRIP-${cross_prefix}strip}"
>   windres="${WINDRES-${cross_prefix}windres}"
> @@ -3635,8 +3637,10 @@ echo "CC_I386=$cc_i386" >> $config_host_mak
>   echo "HOST_CC=$host_cc" >> $config_host_mak
>   echo "OBJCC=$objcc" >> $config_host_mak
>   echo "AR=$ar" >> $config_host_mak
> +echo "NM=$nm" >> $config_host_mak
>   echo "OBJCOPY=$objcopy" >> $config_host_mak
>   echo "LD=$ld" >> $config_host_mak
> +echo "RANLIB=$ranlib" >> $config_host_mak
>   echo "WINDRES=$windres" >> $config_host_mak
>   echo "LIBTOOL=$libtool" >> $config_host_mak
>   echo "CFLAGS=$CFLAGS" >> $config_host_mak

  reply	other threads:[~2012-11-13 17:50 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-13  8:42 [Qemu-devel] [PULL 0/6] pixman build fixes Gerd Hoffmann
2012-11-13  8:42 ` [Qemu-devel] [PATCH 1/6] pixman: add output dir to include path Gerd Hoffmann
2012-11-13 17:57   ` Stefan Weil
2012-11-14  8:40     ` Gerd Hoffmann
2012-11-14 12:33       ` Gerd Hoffmann
2012-11-13  8:42 ` [Qemu-devel] [PATCH 2/6] pixman: pass tools to configure Gerd Hoffmann
2012-11-13 17:50   ` Stefan Weil [this message]
2012-11-14  8:42     ` Gerd Hoffmann
2012-11-13  8:42 ` [Qemu-devel] [PATCH 3/6] pixman: disable gtk Gerd Hoffmann
2012-11-13 18:27   ` Peter Maydell
2012-11-13  8:42 ` [Qemu-devel] [PATCH 4/6] pixman: add pass cflags, add -fPIC Gerd Hoffmann
2012-11-13 18:28   ` Peter Maydell
2012-11-13  8:42 ` [Qemu-devel] [PATCH 5/6] pixman: build internal version early Gerd Hoffmann
2012-11-13 18:29   ` Peter Maydell
2012-11-13 18:41     ` Johnson, Eric
2012-11-14 12:41       ` Gerd Hoffmann
2012-11-14 18:42         ` Johnson, Eric
2012-11-13  8:42 ` [Qemu-devel] [PATCH 6/6] pixman: add licensing info Gerd Hoffmann
2012-11-13 17:31 ` [Qemu-devel] [PULL 0/6] pixman build fixes Peter Maydell
2012-11-13 18:07   ` Stefan Weil
2012-11-14 16:27 ` Anthony Liguori

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=50A28856.6040502@weilnetz.de \
    --to=sw@weilnetz.de \
    --cc=kraxel@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.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;
as well as URLs for NNTP newsgroup(s).