qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Stefan Berger <stefanb@us.ibm.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] More Win32 build fixes
Date: Fri, 05 Dec 2008 13:30:52 -0600	[thread overview]
Message-ID: <4939816C.7040106@codemonkey.ws> (raw)
In-Reply-To: <OF295B866B.B03121B3-ON85257516.00556B35-85257516.005583FD@us.ibm.com>

Stefan Berger wrote:
>
> Anthony Liguori <anthony@codemonkey.ws> wrote on 12/05/2008 09:29:01 AM:
>
> > Anthony Liguori <anthony@codemonkey.ws>
> > 12/05/2008 09:29 AM
> >
> > To
> >
> > Stefan Berger/Watson/IBM@IBMUS, qemu-devel@nongnu.org
> >
> > cc
> >
> > Subject
> >
> > Re: [Qemu-devel] More Win32 build fixes
> >
> > Stefan Berger wrote:
> > > This fixes one more current build problem on Win32.
> > >
> > > Signed-off-by: Stefan Berger <stefanb@us.ibm.com>
> > >  
> >
> > Are you using the latest SVN?  QEMU builds fine on Win32 for me with 
> the
> > latest SVN.
>
> Yes, as checked out a few hours ago. I get this here if the patch is 
> not applied:
>
> gcc -g -Wl,--warn-common  -m32 -o qemu.exe vl.o /work/qemu/osdep.o 
> monitor.o pci.o loader.o isa_mmio.o machine.o fw_cfg.o 
> /work/qemu/block-raw-win32.o lsi53c895a.o esp.o usb-ohci.o 
> eeprom93xx.o eepro100.o ne2000.o pcnet.o rtl8139.o e1000.o ide.o 
> pckbd.o ps2.o vga.o sb16.o es1370.o dma.o fdc.o mc146818rtc.o serial.o 
> i8259.o i8254.o pcspk.o pc.o cirrus_vga.o apic.o parallel.o acpi.o 
> piix_pci.o usb-uhci.o vmmouse.o vmport.o vmware_vga.o tpm_tis.o 
> tpm_emu.o gdbstub.o gdbstub-xml.o ../libqemu_common.a libqemu.a  -lm 
> -lwinmm -lws2_32 -liphlpapi -lz

You've got a merge conflict.  tpm_tis and tpm_emu.o are not in upstream 
QEMU.  You should see virtio.o and virtio-blk.o right after vmware_vga.o.

Regards,

Anthony Liguori

>  -L/usr/local/lib -lmingw32 -lSDLmain -lSDL -mconsole     -lgmp
> pc.o: In function `pc_init1':
> C:/msys/1.0/work/qemu/hw/pc.c:1109: undefined reference to 
> `virtio_blk_init'
> C:/msys/1.0/work/qemu/hw/pc.c:1117: undefined reference to 
> `virtio_balloon_init'
>
>   Stefan
>
> >
> > Regards,
> >
> > Anthony Liguori
> >

  reply	other threads:[~2008-12-05 19:30 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-05 14:11 [Qemu-devel] More Win32 build fixes Stefan Berger
2008-12-05 14:29 ` Anthony Liguori
2008-12-05 15:34   ` Stefan Berger
2008-12-05 19:30     ` Anthony Liguori [this message]
2008-12-05 20:03       ` Stefan Berger

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=4939816C.7040106@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanb@us.ibm.com \
    /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).