[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 27/29] qapi: Move qapi-schema.json to qapi/,
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 27/29] qapi: Move qapi-schema.json to qapi/, rename generated files |
Date: |
Mon, 12 Feb 2018 16:36:38 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 02/11/2018 03:36 AM, Markus Armbruster wrote:
Move qapi-schema.json to qapi/, so it's next to its modules, and all
files get generated to qapi/, not just the ones generated for modules.
Consistently name the generated files qapi-MODULE.EXT:
qmp-commands.[ch] become qapi-commands.[ch], qapi-event.[ch] become
qapi-events.[ch], and qmp-introspect.[ch] become qapi-introspect.[ch].
This gets rid of the temporary hacks in scripts/qapi/commands.py and
scripts/qapi/events.py.
Ah, so my parallel series that proposed naming the file
qapi/qmp-schema.qapi gets interesting, with your patch favoring the
qapi- naming everywhere. I'll have to think about how much (or little)
of my series to rebase on top of this (I like my notion of renaming to
the .qapi suffix, though, as we really are using files that aren't JSON,
but only resemble it).
Signed-off-by: Markus Armbruster <address@hidden>
---
+++ b/.gitignore
@@ -29,8 +29,8 @@
/qga/qapi-generated
/qapi-generated
/qapi-gen-timestamp
-/qapi-builtin-types.[ch]
-/qapi-builtin-visit.[ch]
+/qapi/qapi-builtin-types.[ch]
+/qapi/qapi-builtin-visit.[ch]
Might be some interesting churn if you like my idea of using globs for
easier maintenance of this file.
+++ b/tpm.c
@@ -182,7 +182,6 @@ int tpm_config_parse(QemuOptsList *opts_list, const char
*optarg)
/*
* Walk the list of active TPM backends and collect information about them
- * following the schema description in qapi-schema.json.
*/
Should the overall comment keep the trailing '.'?
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- [Qemu-devel] [PATCH v2 06/29] qapi: Reduce use of global variables in generators some, (continued)
- [Qemu-devel] [PATCH v2 06/29] qapi: Reduce use of global variables in generators some, Markus Armbruster, 2018/02/11
- [Qemu-devel] [PATCH v2 07/29] qapi: Turn generators into modules, Markus Armbruster, 2018/02/11
- [Qemu-devel] [PATCH v2 19/29] qapi: Make code-generating visitors use QAPIGen more, Markus Armbruster, 2018/02/11
- [Qemu-devel] [PATCH v2 27/29] qapi: Move qapi-schema.json to qapi/, rename generated files, Markus Armbruster, 2018/02/11
- [Qemu-devel] [PATCH v2 15/29] qapi: Record 'include' directives in parse tree, Markus Armbruster, 2018/02/11
- [Qemu-devel] [PATCH v2 16/29] qapi: Generate in source order, Markus Armbruster, 2018/02/11