On Fri, Jul 2, 2021 at 5:26 PM G S Niteesh Babu <niteesh.gs@gmail.com>
wrote:
> Added dependencies for the upcoming AQMP TUI under the optional
> 'tui' group.
>
> The same dependencies have also been added under the devel group
> since no work around has been found for optional groups to imply
> other optional groups.
>
> Signed-off-by: G S Niteesh Babu <niteesh.gs@gmail.com>
>
One thing that I notice is that if you already have a .tox environment,
adding new dependencies won't cause .tox to regenerate its environments,
forcing me to delete .tox and try again.
Maybe it's possible to update the Makefile so that changes to 'setup.cfg'
will coerce Tox to rebuild?
Looks good otherwise, thank you for taking the time to re-order your
patches for the list, and congrats on your first submission :)
--js
> ---
> python/Pipfile.lock | 12 ++++++++++++
> python/setup.cfg | 7 +++++++
> 2 files changed, 19 insertions(+)
>
> diff --git a/python/Pipfile.lock b/python/Pipfile.lock
> index 8ab41a3f60..76cf1e4930 100644
> --- a/python/Pipfile.lock
> +++ b/python/Pipfile.lock
> @@ -289,6 +289,18 @@
> "markers": "python_version < '3.8'",
> "version": "==3.10.0.0"
> },
> + "urwid": {
> + "hashes": [
> +
> "sha256:588bee9c1cb208d0906a9f73c613d2bd32c3ed3702012f51efe318a3f2127eae"
> + ],
> + "version": "==2.1.2"
> + },
> + "urwid-readline": {
> + "hashes": [
> +
> "sha256:018020cbc864bb5ed87be17dc26b069eae2755cb29f3a9c569aac3bded1efaf4"
> + ],
> + "version": "==0.13"
> + },
> "virtualenv": {
> "hashes": [
>
> "sha256:14fdf849f80dbb29a4eb6caa9875d476ee2a5cf76a5f5415fa2f1606010ab467",
> diff --git a/python/setup.cfg b/python/setup.cfg
> index 1a552d672a..c62803bffc 100644
> --- a/python/setup.cfg
> +++ b/python/setup.cfg
> @@ -44,11 +44,18 @@ devel =
> mypy >= 0.770
> pylint >= 2.8.0
> tox >= 3.18.0
> + urwid >= 2.1.2
> + urwid-readline >= 0.13
>
> # Provides qom-fuse functionality
> fuse =
> fusepy >= 2.0.4
>
> +# AQMP TUI dependencies
> +tui =
> + urwid >= 2.1.2
> + urwid-readline >= 0.13
> +
> [options.entry_points]
> console_scripts =
> qom = qemu.qmp.qom:main
> --
> 2.17.1
>
>