qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Lucas Meneghel Rodrigues <lmr@redhat.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>,
	qemu-devel@nongnu.org, "Michael S. Tsirkin" <mst@redhat.com>
Subject: [Qemu-devel] Re: Build failure for upstream qemu 04-03-2010
Date: Thu, 04 Mar 2010 13:15:48 -0600	[thread overview]
Message-ID: <4B9006E4.7090507@codemonkey.ws> (raw)
In-Reply-To: <1267726847.5322.40.camel@localhost.localdomain>

It was broken by:

commit da51e79b7ff2126cc2448749d657a4f6e3b1270f
Author: Stefan Weil <weil@mail.berlios.de>
Date:   Tue Mar 2 22:37:44 2010 +0100

     eepro100: Support gpxe boot for all eepro100 devices

     Only two boot ROM files are needed for all devices.

     * Add these GPXE ROM files using new naming convention
       (as discussed on qemu-devel). Both files were created
       with http://rom-o-matic.net/, PCI vendor / device ids
       as in ROM filenames and option BANNER_TIMEOUT = 0.

     * Remove old PXE ROM file for i82559er.
       It was replaced by gpxe-eepro100-80861209.rom.

     * Update pc-bios/README (and sort entries).

     Full support still needs additional eepro100 fixes.

     Signed-off-by: Stefan Weil <weil@mail.berlios.de>
     Signed-off-by: Michael S. Tsirkin <mst@redhat.com>


Stefan/Michael, can you please submit a fix?  The Makefile wasn't 
adjusted for the new PXE naming.

Really, the roms should not have been renamed like this in this sort of 
patch.  If we switch to gpxe as a prefix, we should rename all of the 
roms at once as opposed to having some roms with a pxe name and others 
with gpxe.

Regards,

Anthony Liguori

On 03/04/2010 12:20 PM, Lucas Meneghel Rodrigues wrote:
> Hi folks, I found this problem trying to build qemu from the qemu.git
> repo:
>
> 03/04 13:07:34 ERROR|       kvm:0061| Test failed: Command<make install>  failed, rc=2, Command returned non-zero exit status
> * Command:
>      make install
> Exit status: 2
> Duration: 0
>
> stdout:
> install -d -m0755 -p "/usr/local/autotest/tests/kvm/build/etc/qemu"
> install -m0644 -p /usr/local/autotest/tests/kvm/src/kvm_userspace/sysconfigs/target/target-x86_64.conf "/usr/local/autotest/tests/kvm/build/etc/qemu"
> install -d -m0755 -p "/usr/local/autotest/tests/kvm/build/bin"
> install -m0755 -p  qemu-nbd qemu-img qemu-io  "/usr/local/autotest/tests/kvm/build/bin"
> install -d -m0755 -p "/usr/local/autotest/tests/kvm/build/share/qemu"
> set -e; for x in bios.bin vgabios.bin vgabios-cirrus.bin ppc_rom.bin video.x openbios-sparc32 openbios-sparc64 openbios-ppc pxe-e1000.bin pxe-i82559er.bin pxe-ne2k_pci.bin pxe-pcnet.bin pxe-rtl8139.bin pxe-virtio.bin bamboo.dtb petalogix-s3adsp1800.dtb multiboot.bin linuxboot.bin; do \
> 		install -m0644 -p /usr/local/autotest/tests/kvm/src/kvm_userspace/pc-bios/$x "/usr/local/autotest/tests/kvm/build/share/qemu"; \
> 	done
> stderr:
> install: cannot stat `/usr/local/autotest/tests/kvm/src/kvm_userspace/pc-bios/pxe-i82559er.bin': No such file or directory
> make: *** [install] Error 1
>
> The exact commit where the problem was found:
>
> 03/04 12:59:44 INFO | kvm_utils:0367| Commit hash for git://git.savannah.nongnu.org/qemu.git is deb4203dc9ebba6992ac1d583773c143a014e15a (no tag found)
>
> Please let me know if you need more information about the problem
>
>
>    

  reply	other threads:[~2010-03-04 19:16 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-04 18:20 [Qemu-devel] Build failure for upstream qemu 04-03-2010 Lucas Meneghel Rodrigues
2010-03-04 19:15 ` Anthony Liguori [this message]
2010-03-04 19:56   ` [Qemu-devel] " Stefan Weil
2010-03-04 20:31     ` [Qemu-devel] [PATCH] Makefile: Fix names of GPXE ROM files Stefan Weil

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=4B9006E4.7090507@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=lmr@redhat.com \
    --cc=mst@redhat.com \
    --cc=mtosatti@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).