All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: Stefan Hajnoczi <stefanha@gmail.com>
Cc: qemu-devel@nongnu.org,  stefanha@redhat.com
Subject: Re: [PULL v2 00/28] QAPI patches patches for 2022-10-25
Date: Thu, 03 Nov 2022 15:02:03 +0100	[thread overview]
Message-ID: <87sfj05e78.fsf@pond.sub.org> (raw)
In-Reply-To: <CAJSP0QV0_jwZWU93VKNjycicD8ng2NPczzXmCU4QCNJm+nvB+w@mail.gmail.com> (Stefan Hajnoczi's message of "Sun, 30 Oct 2022 15:04:55 -0400")

Stefan Hajnoczi <stefanha@gmail.com> writes:

> On Wed, 26 Oct 2022 at 14:44, Markus Armbruster <armbru@redhat.com> wrote:
>>
>> The following changes since commit e750a7ace492f0b450653d4ad368a77d6f660fb8:
>>
>>   Merge tag 'pull-9p-20221024' of https://github.com/cschoenebeck/qemu into staging (2022-10-24 14:27:12 -0400)
>>
>> are available in the Git repository at:
>>
>>   https://repo.or.cz/qemu/armbru.git tags/pull-qapi-2022-10-25-v2
>>
>> for you to fetch changes up to c0f24f8f31ca82e34ef037bfe34ef71eeecb401d:
>>
>>   qapi: Drop temporary logic to support conversion step by step (2022-10-26 20:08:52 +0200)
>>
>> ----------------------------------------------------------------
>> QAPI patches patches for 2022-10-25
>>
>> ----------------------------------------------------------------
>> Markus Armbruster (28):
>>       docs/devel/qapi-code-gen: Update example to match current code
>>       qapi: Tidy up whitespace in generated code
>>       docs/devel/qapi-code-gen: Extend example for next commit's change
>>       qapi: Start to elide redundant has_FOO in generated C
>>       qapi tests: Elide redundant has_FOO in generated C
>>       qapi acpi: Elide redundant has_FOO in generated C
>>       qapi audio: Elide redundant has_FOO in generated C
>>       qapi block: Elide redundant has_FOO in generated C
>
> This commit breaks qemu-iotests 056 in CI. I have included
> instructions for reproducing it locally below. I will drop this pull
> request for now. Please note that the QEMU 7.2 soft freeze is on
> Tuesday.

I finally managed to debug this one.  I think a v3 is best.  And then
we'll see whether the series can still go in, or needs to wait for the
next cycle.

Thanks!



      parent reply	other threads:[~2022-11-03 14:02 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-26 18:35 [PULL v2 00/28] QAPI patches patches for 2022-10-25 Markus Armbruster
2022-10-26 18:35 ` [PULL v2 01/28] docs/devel/qapi-code-gen: Update example to match current code Markus Armbruster
2022-10-26 18:35 ` [PULL v2 02/28] qapi: Tidy up whitespace in generated code Markus Armbruster
2022-10-26 18:35 ` [PULL v2 03/28] docs/devel/qapi-code-gen: Extend example for next commit's change Markus Armbruster
2022-10-26 18:35 ` [PULL v2 04/28] qapi: Start to elide redundant has_FOO in generated C Markus Armbruster
2022-10-26 18:35 ` [PULL v2 05/28] qapi tests: Elide " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 06/28] qapi acpi: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 07/28] qapi audio: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 08/28] qapi block: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 09/28] qapi chardev: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 10/28] qapi crypto: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 11/28] qapi dump: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 12/28] qapi job: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 13/28] qapi machine: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 14/28] qapi migration: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 15/28] qapi misc: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 16/28] qapi net: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 17/28] qapi pci: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 18/28] qapi qdev qom: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 19/28] qapi replay: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 20/28] qapi rocker: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 21/28] qapi run-state: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 22/28] qapi stats: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 23/28] qapi tpm: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 24/28] qapi transaction: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 25/28] qapi ui: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 26/28] qapi virtio: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 27/28] qapi qga: " Markus Armbruster
2022-10-26 18:35 ` [PULL v2 28/28] qapi: Drop temporary logic to support conversion step by step Markus Armbruster
2022-10-30 19:04 ` [PULL v2 00/28] QAPI patches patches for 2022-10-25 Stefan Hajnoczi
2022-10-30 19:06   ` Stefan Hajnoczi
2022-11-03 14:02   ` Markus Armbruster [this message]

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=87sfj05e78.fsf@pond.sub.org \
    --to=armbru@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@gmail.com \
    --cc=stefanha@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.