[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH RFC 02/32] scripts/qmp: Fix shebang and imports
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH RFC 02/32] scripts/qmp: Fix shebang and imports |
Date: |
Tue, 26 May 2020 17:55:48 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 5/14/20 7:53 AM, John Snow wrote:
> There's more wrong with these scripts; They are in various stages of
> disrepair. That's beyond the scope of this current patchset.
>
> This just mechanically corrects the imports and the shebangs, as part of
> ensuring that the python/qemu/lib refactoring didn't break anything
> needlessly.
>
> Signed-off-by: John Snow <address@hidden>
Whoops, I address this in a patch series I'm working on, too. I'll keep
my patch in there for now until this one makes it in, or vice-versa.
https://www.mail-archive.com/address@hidden/msg66801.html
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> scripts/qmp/qmp | 4 +++-
> scripts/qmp/qom-fuse | 4 +++-
> scripts/qmp/qom-get | 6 ++++--
> scripts/qmp/qom-list | 4 +++-
> scripts/qmp/qom-set | 6 ++++--
> scripts/qmp/qom-tree | 6 ++++--
> 6 files changed, 21 insertions(+), 9 deletions(-)
>
> diff --git a/scripts/qmp/qmp b/scripts/qmp/qmp
> index 0625fc2aba..5981f7c414 100755
> --- a/scripts/qmp/qmp
> +++ b/scripts/qmp/qmp
> @@ -11,7 +11,9 @@
> # See the COPYING file in the top-level directory.
>
> import sys, os
> -from qmp import QEMUMonitorProtocol
> +
> +sys.path.append(os.path.join(os.path.dirname(__file__), '..', '..',
> 'python'))
> +from qemu.lib import QEMUMonitorProtocol
>
> def print_response(rsp, prefix=[]):
> if type(rsp) == list:
> diff --git a/scripts/qmp/qom-fuse b/scripts/qmp/qom-fuse
> index 6bada2c33d..6006f1d562 100755
> --- a/scripts/qmp/qom-fuse
> +++ b/scripts/qmp/qom-fuse
> @@ -15,7 +15,9 @@ import fuse, stat
> from fuse import Fuse
> import os, posix
> from errno import *
> -from qmp import QEMUMonitorProtocol
> +
> +sys.path.append(os.path.join(os.path.dirname(__file__), '..', '..',
> 'python'))
> +from qemu.lib import QEMUMonitorProtocol
>
> fuse.fuse_python_api = (0, 2)
>
> diff --git a/scripts/qmp/qom-get b/scripts/qmp/qom-get
> index 007b4cd442..1d04d1b119 100755
> --- a/scripts/qmp/qom-get
> +++ b/scripts/qmp/qom-get
> @@ -1,4 +1,4 @@
> -#!/usr/bin/python
> +#!/usr/bin/env python3
> ##
> # QEMU Object Model test tools
> #
> @@ -13,7 +13,9 @@
>
> import sys
> import os
> -from qmp import QEMUMonitorProtocol
> +
> +sys.path.append(os.path.join(os.path.dirname(__file__), '..', '..',
> 'python'))
> +from qemu.lib import QEMUMonitorProtocol
>
> cmd, args = sys.argv[0], sys.argv[1:]
> socket_path = None
> diff --git a/scripts/qmp/qom-list b/scripts/qmp/qom-list
> index 03bda3446b..94e3bc21ba 100755
> --- a/scripts/qmp/qom-list
> +++ b/scripts/qmp/qom-list
> @@ -1,4 +1,4 @@
> -#!/usr/bin/python
> +#!/usr/bin/env python3
> ##
> # QEMU Object Model test tools
> #
> @@ -13,6 +13,8 @@
>
> import sys
> import os
> +
> +sys.path.append(os.path.join(os.path.dirname(__file__), '..', '..',
> 'python'))
> from qmp import QEMUMonitorProtocol
>
> cmd, args = sys.argv[0], sys.argv[1:]
> diff --git a/scripts/qmp/qom-set b/scripts/qmp/qom-set
> index c37fe78b00..a78080a4fc 100755
> --- a/scripts/qmp/qom-set
> +++ b/scripts/qmp/qom-set
> @@ -1,4 +1,4 @@
> -#!/usr/bin/python
> +#!/usr/bin/env python3
> ##
> # QEMU Object Model test tools
> #
> @@ -13,7 +13,9 @@
>
> import sys
> import os
> -from qmp import QEMUMonitorProtocol
> +
> +sys.path.append(os.path.join(os.path.dirname(__file__), '..', '..',
> 'python'))
> +from qemu.lib import QEMUMonitorProtocol
>
> cmd, args = sys.argv[0], sys.argv[1:]
> socket_path = None
> diff --git a/scripts/qmp/qom-tree b/scripts/qmp/qom-tree
> index 1c8acf61e7..4866c15581 100755
> --- a/scripts/qmp/qom-tree
> +++ b/scripts/qmp/qom-tree
> @@ -1,4 +1,4 @@
> -#!/usr/bin/python
> +#!/usr/bin/env python3
> ##
> # QEMU Object Model test tools
> #
> @@ -15,7 +15,9 @@
>
> import sys
> import os
> -from qmp import QEMUMonitorProtocol
> +
> +sys.path.append(os.path.join(os.path.dirname(__file__), '..', '..',
> 'python'))
> +from qemu.lib import QEMUMonitorProtocol
>
> cmd, args = sys.argv[0], sys.argv[1:]
> socket_path = None
>
- Re: [PATCH RFC 01/32] python/qemu: create qemu.lib module, (continued)
- Re: [PATCH RFC 01/32] python/qemu: create qemu.lib module, Daniel P . Berrangé, 2020/05/26
- Re: [PATCH RFC 01/32] python/qemu: create qemu.lib module, Philippe Mathieu-Daudé, 2020/05/26
- Re: [PATCH RFC 01/32] python/qemu: create qemu.lib module, Daniel P . Berrangé, 2020/05/26
- Re: [PATCH RFC 01/32] python/qemu: create qemu.lib module, John Snow, 2020/05/27
- Re: [PATCH RFC 01/32] python/qemu: create qemu.lib module, Daniel P . Berrangé, 2020/05/27
[PATCH RFC 02/32] scripts/qmp: Fix shebang and imports, John Snow, 2020/05/14
- Re: [PATCH RFC 02/32] scripts/qmp: Fix shebang and imports,
Philippe Mathieu-Daudé <=
[PATCH RFC 03/32] python//machine.py: remove bare except, John Snow, 2020/05/14
Re: [PATCH RFC 03/32] python//machine.py: remove bare except, Philippe Mathieu-Daudé, 2020/05/26
[PATCH RFC 04/32] python/qemu/lib: delint, add pylintrc, John Snow, 2020/05/14
[PATCH RFC 05/32] python/qemu/lib: delint; add flake8 config, John Snow, 2020/05/14