qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 4/5] configure: move common libraries to config-host.mak
Date: Tue, 2 Apr 2013 19:17:39 +0100	[thread overview]
Message-ID: <CAFEAcA-Us3wP5fmNC8L2LyQ-TYx9__qyiVd8s=aYtZi81=+drA@mail.gmail.com> (raw)
In-Reply-To: <515B14BE.2060705@redhat.com>

On 2 April 2013 18:26, Paolo Bonzini <pbonzini@redhat.com> wrote:
> Il 02/04/2013 19:01, Peter Maydell ha scritto:
>> I think this may perturb the order of the link line slightly,
>> but hopefully not in any way that matters. (Did you test a
>> static build?)
>
> I don't have static versions of half of the prerequisites of system
> emulators...

Really? Ubuntu provides them, maybe you should complain to your
distro vendor :-)

Anyway, I did a compile test of everything with --static and it
does turn out to be busted:

ccache gcc -Werror -m64 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64
-D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall
-Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing
-fstack-protector-all -Wendif-labels -Wmissing-include-dirs
-Wempty-body -Wnested-externs -Wformat-security -Wformat-y2k
-Winit-self -Wignored-qualifiers -Wold-style-declaration
-Wold-style-definition -Wtype-limits -I/usr/include/libpng12
-I/usr/include/pixman-1   -DHAS_AUDIO -DHAS_AUDIO_CHOICE  -I. -I.
-I../linux-headers -I..
-I/home/petmay01/linaro/qemu-from-laptop/qemu/target-i386 -DNEED_CPU_H
-I/home/petmay01/linaro/qemu-from-laptop/qemu/include -pthread
-I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include
-g  -Wl,--warn-common -m64 -static -g   -o qemu-system-i386
../aio-posix.o [.o list elided] translate-all.o xen-stub.o
../libqemuutil.a ../libqemustub.a  -lrt -pthread -lgthread-2.0
-lglib-2.0 -lrt -lpcre     -lm -lutil -lbluetooth   -lncurses -ltinfo
  -lvdeplug -luuid -lpng12 -lz -lm    -lz -laio -lfdt -lpixman-1
[usual complaints about glibc functions needing the libraries
at runtime for NSS stuff elided]
/usr/lib/gcc/x86_64-linux-gnu/4.6/../../../x86_64-linux-gnu/libpixman-1.a(pixman-radial-gradient.o):
In function `radial_compute_color':
(.text+0x16f): undefined reference to `sqrt'
/usr/lib/gcc/x86_64-linux-gnu/4.6/../../../x86_64-linux-gnu/libpixman-1.a(pixman-combine32.o):
In function `combine_soft_light_ca':
(.text+0xa1e7): undefined reference to `sqrt'
/usr/lib/gcc/x86_64-linux-gnu/4.6/../../../x86_64-linux-gnu/libpixman-1.a(pixman-combine32.o):
In function `combine_soft_light_ca':
(.text+0xa25e): undefined reference to `sqrt'
/usr/lib/gcc/x86_64-linux-gnu/4.6/../../../x86_64-linux-gnu/libpixman-1.a(pixman-combine32.o):
In function `combine_soft_light_ca':
(.text+0xa2cc): undefined reference to `sqrt'
/usr/lib/gcc/x86_64-linux-gnu/4.6/../../../x86_64-linux-gnu/libpixman-1.a(pixman-combine32.o):
In function `combine_soft_light_u':
(.text+0xab46): undefined reference to `sqrt'
/usr/lib/gcc/x86_64-linux-gnu/4.6/../../../x86_64-linux-gnu/libpixman-1.a(pixman-combine32.o):(.text+0xabfc):
more undefined references to `sqrt' follow
collect2: ld returned 1 exit status
make[1]: *** [qemu-system-i386] Error 1
make[1]: Leaving directory
`/home/petmay01/linaro/qemu-from-laptop/qemu/i386-softmmu'
make: *** [subdir-i386-softmmu] Error 2

This is because we seem to have lost the trailing "-lm -lz"; here's
the tail end of the link command from a build on master for
comparison:

translate-all.o xen-stub.o ../libqemuutil.a ../libqemustub.a  -lrt
-pthread -lgthread-2.0 -lglib-2.0 -lrt -lpcre    -lutil -lbluetooth
-lncurses -ltinfo    -lvdeplug -luuid -lpng12 -lz -lm    -laio -lfdt
-lpixman-1    -lm -lz

thanks
-- PMM

  reply	other threads:[~2013-04-02 18:18 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-02 14:44 [Qemu-devel] [PATCH 0/5] trim down config-target.mak Paolo Bonzini
2013-04-02 14:44 ` [Qemu-devel] [PATCH 1/5] configure: move TARGET_*_ALIGNMENT to target-*/cpu.h Paolo Bonzini
2013-04-02 16:43   ` Peter Maydell
2013-04-02 16:56     ` Paolo Bonzini
2013-04-02 17:17       ` Peter Maydell
2013-04-02 17:26         ` Paolo Bonzini
2013-04-02 17:47           ` Peter Maydell
2013-04-03  8:55             ` Paolo Bonzini
2013-04-02 17:57     ` Aurelien Jarno
2013-04-02 14:44 ` [Qemu-devel] [PATCH 2/5] cpu: default TARGET_LONG_ALIGNMENT to TARGET_LONG_SIZE Paolo Bonzini
2013-04-02 15:16   ` Peter Maydell
2013-04-02 14:44 ` [Qemu-devel] [PATCH 3/5] configure: move CONFIG_QEMU_LDST_OPTIMIZATION to config-host.mak Paolo Bonzini
2013-04-02 16:45   ` Peter Maydell
2013-04-02 14:44 ` [Qemu-devel] [PATCH 4/5] configure: move common libraries " Paolo Bonzini
2013-04-02 17:01   ` Peter Maydell
2013-04-02 17:26     ` Paolo Bonzini
2013-04-02 18:17       ` Peter Maydell [this message]
2013-04-02 18:51         ` Paolo Bonzini
2013-04-02 14:44 ` [Qemu-devel] [PATCH 5/5] configure: CONFIG_NO_XEN is duplicated Paolo Bonzini
2013-04-02 16:54   ` Stefano Stabellini
2013-04-02 17:24   ` Peter Maydell

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='CAFEAcA-Us3wP5fmNC8L2LyQ-TYx9__qyiVd8s=aYtZi81=+drA@mail.gmail.com' \
    --to=peter.maydell@linaro.org \
    --cc=pbonzini@redhat.com \
    --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).