* [Qemu-devel] [PATCH] Add missing parameters to mon option documentation
@ 2017-11-09 12:19 Vicente Jimenez Aguilar
2017-11-09 15:06 ` Daniel P. Berrange
2017-11-09 15:47 ` Marc-André Lureau
0 siblings, 2 replies; 3+ messages in thread
From: Vicente Jimenez Aguilar @ 2017-11-09 12:19 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial, Vicente Jimenez Aguilar
Documentation missed 'mon' option's 'pretty' and 'default' parameters
Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
---
qemu-options.hx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/qemu-options.hx b/qemu-options.hx
index 3728e9b4dd..72cf48a8e5 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -3477,9 +3477,9 @@ Like -qmp but uses pretty JSON formatting.
ETEXI
DEF("mon", HAS_ARG, QEMU_OPTION_mon, \
- "-mon [chardev=]name[,mode=readline|control]\n", QEMU_ARCH_ALL)
+ "-mon [chardev=]name[,mode=readline|control][,pretty[=on|off]][,default[=on|off]]\n", QEMU_ARCH_ALL)
STEXI
-@item -mon [chardev=]name[,mode=readline|control]
+@item -mon [chardev=]name[,mode=readline|control][,pretty[=on|off]][,default[=on|off]]
@findex -mon
Setup monitor on chardev @var{name}.
ETEXI
--
2.14.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] Add missing parameters to mon option documentation
2017-11-09 12:19 [Qemu-devel] [PATCH] Add missing parameters to mon option documentation Vicente Jimenez Aguilar
@ 2017-11-09 15:06 ` Daniel P. Berrange
2017-11-09 15:47 ` Marc-André Lureau
1 sibling, 0 replies; 3+ messages in thread
From: Daniel P. Berrange @ 2017-11-09 15:06 UTC (permalink / raw)
To: Vicente Jimenez Aguilar; +Cc: qemu-devel, qemu-trivial
On Thu, Nov 09, 2017 at 01:19:03PM +0100, Vicente Jimenez Aguilar wrote:
> Documentation missed 'mon' option's 'pretty' and 'default' parameters
>
> Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
> ---
> qemu-options.hx | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 3728e9b4dd..72cf48a8e5 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -3477,9 +3477,9 @@ Like -qmp but uses pretty JSON formatting.
> ETEXI
>
> DEF("mon", HAS_ARG, QEMU_OPTION_mon, \
> - "-mon [chardev=]name[,mode=readline|control]\n", QEMU_ARCH_ALL)
> + "-mon [chardev=]name[,mode=readline|control][,pretty[=on|off]][,default[=on|off]]\n", QEMU_ARCH_ALL)
> STEXI
> -@item -mon [chardev=]name[,mode=readline|control]
> +@item -mon [chardev=]name[,mode=readline|control][,pretty[=on|off]][,default[=on|off]]
The 'default' option is deprecated and unused, so intentionally not
documented
https://qemu.weilnetz.de/doc/qemu-doc.html#g_t_002dmon-default_003don-_0028since-2_002e4_002e0_0029
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] Add missing parameters to mon option documentation
2017-11-09 12:19 [Qemu-devel] [PATCH] Add missing parameters to mon option documentation Vicente Jimenez Aguilar
2017-11-09 15:06 ` Daniel P. Berrange
@ 2017-11-09 15:47 ` Marc-André Lureau
1 sibling, 0 replies; 3+ messages in thread
From: Marc-André Lureau @ 2017-11-09 15:47 UTC (permalink / raw)
To: Vicente Jimenez Aguilar; +Cc: QEMU, qemu trival
Hi
On Thu, Nov 9, 2017 at 1:19 PM, Vicente Jimenez Aguilar
<googuy@gmail.com> wrote:
> Documentation missed 'mon' option's 'pretty' and 'default' parameters
>
> Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
> ---
> qemu-options.hx | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 3728e9b4dd..72cf48a8e5 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -3477,9 +3477,9 @@ Like -qmp but uses pretty JSON formatting.
> ETEXI
>
> DEF("mon", HAS_ARG, QEMU_OPTION_mon, \
> - "-mon [chardev=]name[,mode=readline|control]\n", QEMU_ARCH_ALL)
> + "-mon [chardev=]name[,mode=readline|control][,pretty[=on|off]][,default[=on|off]]\n", QEMU_ARCH_ALL)
> STEXI
> -@item -mon [chardev=]name[,mode=readline|control]
> +@item -mon [chardev=]name[,mode=readline|control][,pretty[=on|off]][,default[=on|off]]
"default" is deprecated since commit 06ac27f (2.4.0). It does nothing
anymore, so should probably not be documented.
"pretty" was added in commit 39eaab9ac2a82f. You write some
documentation for it based on the commit message perhaps?
thanks
> @findex -mon
> Setup monitor on chardev @var{name}.
> ETEXI
> --
> 2.14.1
>
>
--
Marc-André Lureau
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-11-09 15:47 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-09 12:19 [Qemu-devel] [PATCH] Add missing parameters to mon option documentation Vicente Jimenez Aguilar
2017-11-09 15:06 ` Daniel P. Berrange
2017-11-09 15:47 ` Marc-André Lureau
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.