Message ID | 1364209249-30630-1-git-send-email-stefanha@redhat.com |
---|---|
State | New |
Headers | show |
On Mon, Mar 25, 2013 at 12:00:47PM +0100, Stefan Hajnoczi wrote: > This tiny pull request contains build fixes for a regression introduced in the > last pull request. > > Kevin and I investigated to find out why these build failures were missed. > Both my buildslave and Kevin's workstation did not have the rbd libraries > installed so block/rbd.c was not built. > > The buildslave now has the missing libraries and will warn us if block/rbd.c > breaks again. Kevin has also installed the package so his local builds > exercise the code. > > The mingw build is already failing in buildbot due to an unrelated issue. The > bdrv_file_open() prototype warning was not spotted amongst all the other > warnings. > > The following changes since commit fe6344a05fba26e16863fefcb823242e579b0991: > > Remove device_tree.o from hw/moxie/Makefile.objs. (2013-03-24 11:30:04 +0100) > > are available in the git repository at: > > git://github.com/stefanha/qemu.git block > > for you to fetch changes up to ea804cadf867a0521b9069193db339d041a40689: > > block: Add options QDict to bdrv_file_open() prototypes (fix MinGW build) (2013-03-25 09:53:04 +0100) Tested, and this fixes the build failure of rbd for me. Rich.
On 03/25/2013 05:00 AM, Stefan Hajnoczi wrote: > This tiny pull request contains build fixes for a regression introduced in the > last pull request. > > Kevin and I investigated to find out why these build failures were missed. > Both my buildslave and Kevin's workstation did not have the rbd libraries > installed so block/rbd.c was not built. > > The buildslave now has the missing libraries and will warn us if block/rbd.c > breaks again. Kevin has also installed the package so his local builds > exercise the code. > > The mingw build is already failing in buildbot due to an unrelated issue. The > bdrv_file_open() prototype warning was not spotted amongst all the other > warnings. Series: Reviewed-by: Eric Blake <eblake@redhat.com>