From: John Snow <jsnow@redhat.com>
To: qemu-devel@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>,
Eduardo Habkost <eduardo@habkost.net>,
Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>,
qemu-block@nongnu.org, Markus Armbruster <armbru@redhat.com>,
Hanna Reitz <hreitz@redhat.com>, Cleber Rosa <crosa@redhat.com>,
John Snow <jsnow@redhat.com>
Subject: [RFC PATCH 0/9] Python: Switch to externally hosted qemu.qmp dependency
Date: Thu, 16 Dec 2021 21:29:30 -0500 [thread overview]
Message-ID: <20211217022939.279559-1-jsnow@redhat.com> (raw)
Based-on: <20211215193939.3768033-1-jsnow@redhat.com>
Hi, this series is part of an effort to publish the qemu.qmp package on
PyPI. It is the last of three series to complete this work:
(1) Switch the new Async QMP library in to python/qemu/qmp
(2) Fork python/qemu/qmp out into its own repository,
with updated GitLab CI/CD targets to build packages.
--> (3) Update qemu.git to install qemu.qmp from PyPI,
and then delete python/qemu/qmp.
This series is meant to apply to qemu.git after the standalone
repository for qemu.qmp is established and the library is uploaded to
PyPI.
However, I thought I'd share it now so that everyone can see what kind
of work is involved post-split on the qemu.git end.
RFC notes:
- The version pinning and Pipenv patches will have to be redone later
with proper versions and checksums and so on, but this is a workable
example you can experiment with in the meantime.
- The testenv.py changes to use a VENV could be bolstered even further
to use a checksum of requirements.txt to make that system a little bit
more robust, but I wanted to share the WIP draft to get some early
impressions.
... And that's it! These three series complete the split and leave us
with a published QMP library on PyPI.
Happy New Years, everyone. See you in 2022 :)
John Snow (9):
Python: Update mypy dependency to >= 0.780
Python: update isort dependency
scripts/qmp: Update 'qmp-shell' forwarder stub
scripts/qmp: update remaining forwarder stubs
scripts/qmp: delete qmp.py script stub
scripts: remove sys.path hacks for qemu/qemu.qmp
iotests/testenv: initialize an iotests venv
Python: delete qemu.qmp
(WIP) Python: update Pipfile
python/Pipfile.lock | 86 ++-
python/qemu/qmp/__init__.py | 59 --
python/qemu/qmp/aqmp_tui.py | 652 -----------------
python/qemu/qmp/error.py | 50 --
python/qemu/qmp/events.py | 717 -------------------
python/qemu/qmp/legacy.py | 319 ---------
python/qemu/qmp/message.py | 209 ------
python/qemu/qmp/models.py | 146 ----
python/qemu/qmp/protocol.py | 922 -------------------------
python/qemu/qmp/py.typed | 0
python/qemu/qmp/qmp_client.py | 655 ------------------
python/qemu/qmp/qmp_shell.py | 537 --------------
python/qemu/qmp/util.py | 217 ------
python/qemu/utils/qom_fuse.py | 1 -
python/setup.cfg | 31 +-
python/tests/protocol.py | 583 ----------------
scripts/device-crash-test | 11 +-
scripts/qmp/qemu-ga-client | 40 +-
scripts/qmp/qmp | 11 -
scripts/qmp/qmp-shell | 39 +-
scripts/qmp/qom-fuse | 40 +-
scripts/qmp/qom-get | 40 +-
scripts/qmp/qom-list | 40 +-
scripts/qmp/qom-set | 40 +-
scripts/qmp/qom-tree | 41 +-
scripts/render_block_graph.py | 13 +-
scripts/simplebench/bench_block_job.py | 11 +-
tests/qemu-iotests/requirements.txt | 3 +
tests/qemu-iotests/testenv.py | 74 +-
29 files changed, 412 insertions(+), 5175 deletions(-)
delete mode 100644 python/qemu/qmp/__init__.py
delete mode 100644 python/qemu/qmp/aqmp_tui.py
delete mode 100644 python/qemu/qmp/error.py
delete mode 100644 python/qemu/qmp/events.py
delete mode 100644 python/qemu/qmp/legacy.py
delete mode 100644 python/qemu/qmp/message.py
delete mode 100644 python/qemu/qmp/models.py
delete mode 100644 python/qemu/qmp/protocol.py
delete mode 100644 python/qemu/qmp/py.typed
delete mode 100644 python/qemu/qmp/qmp_client.py
delete mode 100644 python/qemu/qmp/qmp_shell.py
delete mode 100644 python/qemu/qmp/util.py
delete mode 100644 python/tests/protocol.py
delete mode 100755 scripts/qmp/qmp
create mode 100644 tests/qemu-iotests/requirements.txt
--
2.31.1
next reply other threads:[~2021-12-17 2:31 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-17 2:29 John Snow [this message]
2021-12-17 2:29 ` [RFC PATCH 1/9] Python: Update mypy dependency to >= 0.780 John Snow
2021-12-17 2:29 ` [RFC PATCH 2/9] Python: update isort dependency John Snow
2021-12-17 2:29 ` [RFC PATCH 3/9] scripts/qmp: Update 'qmp-shell' forwarder stub John Snow
2021-12-17 2:29 ` [RFC PATCH 4/9] scripts/qmp: update remaining forwarder stubs John Snow
2021-12-17 2:29 ` [RFC PATCH 5/9] scripts/qmp: delete qmp.py script stub John Snow
2021-12-17 2:29 ` [RFC PATCH 6/9] scripts: remove sys.path hacks for qemu/qemu.qmp John Snow
2021-12-17 2:29 ` [RFC PATCH 7/9] iotests/testenv: initialize an iotests venv John Snow
2021-12-17 2:29 ` [RFC PATCH 8/9] Python: delete qemu.qmp John Snow
2021-12-17 2:29 ` [RFC PATCH 9/9] (WIP) Python: update Pipfile John Snow
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211217022939.279559-1-jsnow@redhat.com \
--to=jsnow@redhat.com \
--cc=armbru@redhat.com \
--cc=crosa@redhat.com \
--cc=eduardo@habkost.net \
--cc=hreitz@redhat.com \
--cc=kwolf@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=vsementsov@virtuozzo.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).