qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue
Date: Tue, 18 Dec 2012 16:35:53 -0600
User-agent: Notmuch/0.13.2+93~ged93d79 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu)

Paolo Bonzini <address@hidden> writes:

> Ok... I had really merged the wrong branch with mst's.  I fixed it up and
> pushed again to the same place; it's now commit 6cdf60e, "make check"
> builds and passes.
>
> Here is the diff, sorry for the mistake.

No problem.  I'm processing other pull requests now but will merged this
one tonight after I push the others.

Michael, I'll still plan on merging your changes through this request so
I won't be processing your pinged pull request since Paolo's contains yours.

Regards,

Anthony Liguori

>
> Paolo
>
>
> On Tue, Dec 18, 2012 at 10:11 PM, Paolo Bonzini <address@hidden> wrote:
>
>> Il 18/12/2012 21:59, Anthony Liguori ha scritto:
>> > But this series breaks the build of make check.  It's not just one test
>> > case but almost every single one.  I think you must have eliminated an
>> > implicit include of qemu-common.h which makes PRId64 et al all
>> > undeclared.
>> >
>> > I started fixing this too but it became too much for a merge commit
>> > since it affects almost all tests.
>> >
>> > Can you fixup make check and send this series out?  I'll make sure to
>> > check tomorrow and merge your pull request if I see it tomorrow.
>> > Otherwise, let me know when you plan on sending it and I'll make sure to
>> > be available to merge it.
>>
>> Hmm, I must have pushed the wrong branch because I remember this failure.
>>
>> Paolo
>>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]