Message ID | 1538650419-7995-1-git-send-email-thuth@redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [v2] MAINTAINERS: Add an entry for qemu-options* files in main directory | expand |
On 04/10/2018 12:53, Thomas Huth wrote: > The file "qemu-options.h", "qemu-options.hx" and "qemu-options-wrapper.h" > in the main directory are currently without maintainer according to our > get_maintainers.pl script. Considering that the command line options are > a public interface and thus quite important, this is quite a bad state. > Add an entry for these files which is maintained by Paolo, since the > option handling is tightly coupled with the code in vl.c (that Paolo > already handles via the "Main loop" entry). > > And since I'm interested in the command line interface of QEMU, add > myself (and also Markus) as reviewer here. Sometimes, a file is really only touched as part of changes to other files. Command line options can be changed by block device patches, vl.c patches, or something else. I think changes to qemu-options* alone are quite rare, and I'd rather not have more stuff added to my misc tree (and my inbox)... Paolo > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > v2: Create a separate entry > > MAINTAINERS | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 3275cc6..66c13f2 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1522,6 +1522,14 @@ F: tests/test-qemu-opts.c > F: util/keyval.c > F: util/qemu-option.c > > +Generic command line options > +M: Paolo Bonzini <pbonzini@redhat.com> > +R: Markus Armbruster <armbru@redhat.com> > +R: Thomas Huth <thuth@redhat.com> > +S: Maintained > +F: qemu-options* > +F: vl.c > + > Coverity model > M: Markus Armbruster <armbru@redhat.com> > S: Supported >
Paolo Bonzini <pbonzini@redhat.com> writes: > On 04/10/2018 12:53, Thomas Huth wrote: >> The file "qemu-options.h", "qemu-options.hx" and "qemu-options-wrapper.h" >> in the main directory are currently without maintainer according to our >> get_maintainers.pl script. Considering that the command line options are >> a public interface and thus quite important, this is quite a bad state. >> Add an entry for these files which is maintained by Paolo, since the >> option handling is tightly coupled with the code in vl.c (that Paolo >> already handles via the "Main loop" entry). >> >> And since I'm interested in the command line interface of QEMU, add >> myself (and also Markus) as reviewer here. > > Sometimes, a file is really only touched as part of changes to other > files. Command line options can be changed by block device patches, > vl.c patches, or something else. I think changes to qemu-options* alone Yes. But when the file in question is an important interface, having someone in charge of interface design aspects makes sense all the same. Without that, consistency is left to chance. Of course, you might say that command line consistency is FUBAR anyway. You'd have a point. > are quite rare, and I'd rather not have more stuff added to my misc tree > (and my inbox)... Same here. I serve in such a role for the QAPI schema, together with Eric. I feel like I'm pretty consistently failing at keeping up (pardon the pun). I haven't thrown in the towel only because I also feel the guidance I manage to provide there is better than nothing. But I'd rather not take on more. Note that once my project to QAPIfy the CLI (on hold due to other obligations) is complete, the CLI will be specified in the QAPI schema, effectively dumping it on the QAPI schema maintainers. I guess I'm a sucker for punishment. Just not right now, please.
diff --git a/MAINTAINERS b/MAINTAINERS index 3275cc6..66c13f2 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1522,6 +1522,14 @@ F: tests/test-qemu-opts.c F: util/keyval.c F: util/qemu-option.c +Generic command line options +M: Paolo Bonzini <pbonzini@redhat.com> +R: Markus Armbruster <armbru@redhat.com> +R: Thomas Huth <thuth@redhat.com> +S: Maintained +F: qemu-options* +F: vl.c + Coverity model M: Markus Armbruster <armbru@redhat.com> S: Supported
The file "qemu-options.h", "qemu-options.hx" and "qemu-options-wrapper.h" in the main directory are currently without maintainer according to our get_maintainers.pl script. Considering that the command line options are a public interface and thus quite important, this is quite a bad state. Add an entry for these files which is maintained by Paolo, since the option handling is tightly coupled with the code in vl.c (that Paolo already handles via the "Main loop" entry). And since I'm interested in the command line interface of QEMU, add myself (and also Markus) as reviewer here. Signed-off-by: Thomas Huth <thuth@redhat.com> --- v2: Create a separate entry MAINTAINERS | 8 ++++++++ 1 file changed, 8 insertions(+)