[Qemu-devel] [PATCH] MAINTAINERS: Add qemu-options* to related field

Zhang Chen posted 1 patch 5 years ago
Test checkpatch passed
Test asan passed
Test docker-clang@ubuntu passed
Test docker-mingw@fedora passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190426093920.5616-1-chen.zhang@intel.com
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
[Qemu-devel] [PATCH] MAINTAINERS: Add qemu-options* to related field
Posted by Zhang Chen 5 years ago
From: Zhang Chen <chen.zhang@intel.com>

No one to maintain qemu-options related file, add it to Markus's
Command line option argument parsing field.

Signed-off-by: Zhang Chen <chen.zhang@intel.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 23db6f8408..acc3b32f88 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1800,6 +1800,7 @@ F: tests/test-keyval.c
 F: tests/test-qemu-opts.c
 F: util/keyval.c
 F: util/qemu-option.c
+F: qemu-options*
 
 Coverity model
 M: Markus Armbruster <armbru@redhat.com>
-- 
2.17.GIT


Re: [Qemu-devel] [PATCH] MAINTAINERS: Add qemu-options* to related field
Posted by Markus Armbruster 5 years ago
Zhang Chen <chen.zhang@intel.com > writes:

> From: Zhang Chen <chen.zhang@intel.com>
>
> No one to maintain qemu-options related file, add it to Markus's
> Command line option argument parsing field.
>
> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 23db6f8408..acc3b32f88 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1800,6 +1800,7 @@ F: tests/test-keyval.c
   Command line option argument parsing
   M: Markus Armbruster <armbru@redhat.com>
   S: Supported
   F: include/qemu/option.h
   F: tests/test-keyval.c
>  F: tests/test-qemu-opts.c
>  F: util/keyval.c
>  F: util/qemu-option.c
> +F: qemu-options*
>  
>  Coverity model
>  M: Markus Armbruster <armbru@redhat.com>

Please don't.

There's command line option parsing infrastructure, and there's the
command line interface.  The latter uses the former.

I'm maintaining the former.  I don't have capacity to take on the latter
right now.

Note that the same split exists between QAPI/QMP infrastructure (stanza
"QAPI", maintained by Michael Roth and me) and the QAPI Schema (stanza
"QAPI Schema", maintained by Eric Blake and me).