[Top][All Lists]
[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 17:36:13 -0600 |
User-agent: |
Notmuch/0.13.2+93~ged93d79 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
"Michael S. Tsirkin" <address@hidden> writes:
> On Tue, Dec 18, 2012 at 04:35:53PM -0600, Anthony Liguori wrote:
>> 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
>
> This is painful - I hoped for fast merge to reduce conflicts :(.
Uh, your pull request didn't build so that pretty much eliminates the
possibility of a fast merge.
It'll get pushed today. It's next inline after the current test run
completes.
Regards,
Anthony Liguori
> I already got some patches submitted on list that don't apply on my tree.
>
>> >
>> > 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
>> >>
- [Qemu-devel] [PATCH 23/32] exec: move include files to include/exec/, (continued)
- [Qemu-devel] [PATCH 23/32] exec: move include files to include/exec/, Paolo Bonzini, 2012/12/17
- [Qemu-devel] [PATCH 32/32] janitor: move remaining public headers to include/, Paolo Bonzini, 2012/12/17
- [Qemu-devel] [PATCH 29/32] softmmu: move remaining include files to include/ subdirectories, Paolo Bonzini, 2012/12/17
- [Qemu-devel] [PATCH 25/32] migration: move include files to include/migration/, Paolo Bonzini, 2012/12/17
- [Qemu-devel] [PATCH 27/32] misc: move include files to include/qemu/, Paolo Bonzini, 2012/12/17
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Anthony Liguori, 2012/12/18
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Paolo Bonzini, 2012/12/18
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Anthony Liguori, 2012/12/18
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Paolo Bonzini, 2012/12/19
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Anthony Liguori, 2012/12/18
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Anthony Liguori, 2012/12/18
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Alexander Graf, 2012/12/18
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, David Gibson, 2012/12/18
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Paolo Bonzini, 2012/12/19
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Blue Swirl, 2012/12/19
- Re: [Qemu-devel] [PULL 00/45] Include reorganization + PCI patch queue, Michael S. Tsirkin, 2012/12/19