[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 11/15] qemu-common: move scripts/qapi
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 11/15] qemu-common: move scripts/qapi |
Date: |
Thu, 11 Aug 2022 08:52:26 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Marc-André Lureau <marcandre.lureau@gmail.com> writes:
> Hi
>
> On Fri, Aug 5, 2022 at 12:12 PM Markus Armbruster <armbru@redhat.com> wrote:
>
>> marcandre.lureau@redhat.com writes:
>>
>> > From: Marc-André Lureau <marcandre.lureau@redhat.com>
>> >
>> > This is just moving qapi-gen.py and related subdir to qemu-common, to
>> > ease review and proceed step by step. The following patches will move
>> > related necessary code, tests etc.
>> >
>> > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
>>
>> As moved files tend to become low-level annoyances for a long time, I'd
>> like to understand why you want to move them. The commit message says
>> "to ease review", which I suspect isn't the real reason. Perhaps you
>> explained all that elsewhere already, but I missed it.
>>
>>
>>
> The end goal is to split some projects, such as qemu-ga, to standalone
> meson projects/subprojects. We will be able to build them independently
> from the rest of QEMU, and later on perhaps handle them outside of QEMU
> main repository. To achieve this, I first introduce a qemu-common
> subproject, where qapi and common units are provided. You can check
> https://gitlab.com/marcandre.lureau/qemu/-/commits/qga for a sneak peek at
> current end result.
I worry this move of the QAPI generator code into
subjprojects/common/scripts/qapi/ will be followed by a move into its
own subproject.
Ignorant question: could we turn the QAPI generator into a subproject in
place?
> I said "to ease review and proceed step by step" simply because there are
> no other changes: I don't move the rest of the qapi code & tests all
> together, it's in the subsequent series.
I'd recommend to provide a bit more context in the commit message, even
if you copy it to several messages in a row. Our future selves will
likely be grateful.
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Markus Armbruster, 2022/08/05
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Marc-André Lureau, 2022/08/05
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi,
Markus Armbruster <=
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Marc-André Lureau, 2022/08/11
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Markus Armbruster, 2022/08/11
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Marc-André Lureau, 2022/08/11
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Peter Maydell, 2022/08/11
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Marc-André Lureau, 2022/08/11
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Daniel P . Berrangé, 2022/08/11
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Markus Armbruster, 2022/08/11
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Marc-André Lureau, 2022/08/22
- Re: [PATCH v2 11/15] qemu-common: move scripts/qapi, Markus Armbruster, 2022/08/11