Re: [PATCH v4 16/17] python: update README.rst to reflect qemu.qmp's removal

["Daniel P. Berrangé" <berrange@redhat.com>]

On Mon, Jan 19, 2026 at 04:27:42PM -0500, John Snow wrote:
> It is no longer simply possible to just use this directory as if it were
> an installed package now that it has dependencies, so update the README
> to reflect this.
> 
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
>  python/README.rst | 50 ++++++++++++++++++++++++++++++++++++++++-------
>  1 file changed, 43 insertions(+), 7 deletions(-)
> 
> diff --git a/python/README.rst b/python/README.rst
> index d62e71528d2..2accfd7760e 100644
> --- a/python/README.rst
> +++ b/python/README.rst
> @@ -3,7 +3,17 @@ QEMU Python Tooling
>  
>  This directory houses Python tooling used by the QEMU project to build,
>  configure, and test QEMU. It is organized by namespace (``qemu``), and
> -then by package (e.g. ``qemu/machine``, ``qemu/qmp``, etc).
> +then by package (e.g. ``qemu/machine``, ``qemu/utils``, etc).
> +
> +These tools and libraries are installed to the QEMU configure-time
> +Python virtual environment by default (see qemu.git/pythondeps.toml
> +"tooling" group), and are available for use by any Python script
> +executed by the build system. To have these libraries available for
> +manual invocations of scripts, use of the "run" script executable in
> +your build directory is recommended.

s/executable//, it is just a "script"

> @@ -61,8 +81,24 @@ invoke them without installation, you can invoke e.g.:
>  
>  ``> PYTHONPATH=~/src/qemu/python python3 -m qemu.qmp.qmp_shell``
>  
> +Or, with the runscript available in the QEMU build directory, simply:

s/runscript/run script/

> +
> +``> $builddir/run qmp-shell``
> +
>  The mappings between console script name and python module path can be
> -found in ``setup.cfg``.
> +found in ``setup.cfg``, but the console scripts available are listed
> +here for reference:
> +
> +* ``qemu-ga-client``
> +* ``qmp-shell``
> +* ``qmp-shell-wrap``
> +* ``qmp-tui`` (prototype urwid interface for async QMP)
> +* ``qom``
> +* ``qom-fuse`` (requires fusepy to be installed!)
> +* ``qom-get``
> +* ``qom-list``
> +* ``qom-set``
> +* ``qom-tree``

Those scripts all exist in $GIT/scripts/qmp/.  Are you saying that
they get installed into the venv now ? If so, should this series
also delete scripts/qmp ?

With 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 :|