On 2/4/20 3:11 PM, Wainer dos Santos Moschetta wrote:
> I started fixing an issue on exception handling which in some places
> currently use the deprecated (in Python 3.3) `socket.error`. Then I
> ended up delinting the module code and making some improvements.
>
> Changes in v2:
> - Rebased to master. No conflicts.
> - Added docstring describing the allowed value of the new `timeout`
> option in accept() (patch 03) [jsnow]
> - Set the new `timeout` option to 15.0 by default [philmd]
>
> v1: https://www.mail-archive.com/qemu-devel@nongnu.org/msg667479.html
>
> Git:
> - Tree: https://github.com/wainersm/qemu
> - Branch: python_qmp_sockets_error-v2
>
> CI:
> - Travis (FAIL): https://travis-ci.org/wainersm/qemu/builds/645583812
> Jobs failures aren't related with these changes
>
>
> Wainer dos Santos Moschetta (5):
> python/qemu: qmp: Replace socket.error with OSError
> python/qemu: Delint the qmp module
> python/qemu: qmp: Make accept()'s timeout configurable
> python/qemu: qmp: Make QEMUMonitorProtocol a context manager
> python/qemu: qmp: Remove unnused attributes
>
> python/qemu/qmp.py | 97 +++++++++++++++++++++++++++++++++-------------
> 1 file changed, 71 insertions(+), 26 deletions(-)
>
Patches 3 and 4 applied to my python-next tree:
https://gitlab.com/philmd/qemu/commits/python-next