[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns |
Date: |
Thu, 14 Jan 2021 14:02:17 +0000 |
On Wed, Jan 13, 2021 at 07:20:42PM -0500, John Snow wrote:
> On 1/13/21 6:45 PM, BALATON Zoltan wrote:
> >
> > Please don't. That would break all scripts that already call qemu using
> > single dash options. Forcing everyone to edit their scripts is not nice.
> > How about also accepting single dash in qemu-img for consistency then
> > you can use whatever you want in docs as long as single dash still works
> > for consistency and backward compatibility.
>
> That's not what I meant -- I meant for documentation purposes. QEMU can keep
> accepting single dash, but we should standardize on the spelling for
> purposes of interactive messages, documentation, etc.
I wasn't aware of the fact that some of the utilities are sensitive to
'--' vs '-'! I'm in favor of consistently using '--' in documentation
but allowing both for backwards compatibility where '-' is currently
supported.
If we are in agreement, then let's:
1. Add a section to CODING_STYLE.rst or other developer documentation
documenting this.
2. Convert existing documentation to use '--'. This will make it more
consistent and also avoid confusion about '-' vs '--'.
John: Do you have time to do these things as part of your QEMU
command-line and API improvement work?
Thanks,
Stefan
signature.asc
Description: PGP signature
- [PATCH 1/2] trace: document how to specify multiple --trace patterns, (continued)
- [PATCH 1/2] trace: document how to specify multiple --trace patterns, Stefan Hajnoczi, 2021/01/12
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, Philippe Mathieu-Daudé, 2021/01/12
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, BALATON Zoltan, 2021/01/12
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, John Snow, 2021/01/13
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, Eric Blake, 2021/01/13
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, John Snow, 2021/01/13
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, BALATON Zoltan, 2021/01/13
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, John Snow, 2021/01/13
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns,
Stefan Hajnoczi <=
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, BALATON Zoltan, 2021/01/14
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, Stefan Hajnoczi, 2021/01/14
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, Eric Blake, 2021/01/19
- Re: [PATCH 1/2] trace: document how to specify multiple --trace patterns, John Snow, 2021/01/14
[PATCH 2/2] trace: update docs with meson build information, Stefan Hajnoczi, 2021/01/12