[PATCH 00/10] python: Explicit usage of Python 3

Philippe Mathieu-Daudé posted 10 patches 5 years, 9 months ago
Test docker-quick@centos7 passed
Test FreeBSD passed
Test docker-mingw@fedora passed
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200129231402.23384-1-philmd@redhat.com
Maintainers: Stefan Hajnoczi <stefanha@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Michael Roth <mdroth@linux.vnet.ibm.com>, Fam Zheng <fam@euphon.net>, "Alex Bennée" <alex.bennee@linaro.org>, Cleber Rosa <crosa@redhat.com>, Eduardo Habkost <ehabkost@redhat.com>, "Philippe Mathieu-Daudé" <philmd@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Max Reitz <mreitz@redhat.com>, Markus Armbruster <armbru@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <rth@twiddle.net>, Juan Quintela <quintela@redhat.com>
There is a newer version of this series
scripts/analyse-9p-simpletrace.py                | 2 +-
scripts/analyse-locks-simpletrace.py             | 2 +-
scripts/decodetree.py                            | 2 +-
scripts/device-crash-test                        | 2 +-
scripts/kvm/kvm_flightrecorder                   | 2 +-
scripts/minikconf.py                             | 1 +
scripts/qapi-gen.py                              | 2 +-
scripts/qmp/qemu-ga-client                       | 2 +-
scripts/qmp/qmp                                  | 2 +-
scripts/qmp/qmp-shell                            | 2 +-
scripts/qmp/qom-fuse                             | 2 +-
scripts/render_block_graph.py                    | 2 +-
scripts/replay-dump.py                           | 2 +-
scripts/simpletrace.py                           | 2 +-
scripts/tracetool.py                             | 2 +-
scripts/tracetool/__init__.py                    | 1 -
scripts/tracetool/backend/__init__.py            | 1 -
scripts/tracetool/backend/dtrace.py              | 1 -
scripts/tracetool/backend/ftrace.py              | 1 -
scripts/tracetool/backend/log.py                 | 1 -
scripts/tracetool/backend/simple.py              | 1 -
scripts/tracetool/backend/syslog.py              | 1 -
scripts/tracetool/backend/ust.py                 | 1 -
scripts/tracetool/format/__init__.py             | 1 -
scripts/tracetool/format/c.py                    | 1 -
scripts/tracetool/format/d.py                    | 1 -
scripts/tracetool/format/h.py                    | 1 -
scripts/tracetool/format/log_stap.py             | 1 -
scripts/tracetool/format/simpletrace_stap.py     | 1 -
scripts/tracetool/format/stap.py                 | 1 -
scripts/tracetool/format/tcg_h.py                | 1 -
scripts/tracetool/format/tcg_helper_c.py         | 1 -
scripts/tracetool/format/tcg_helper_h.py         | 1 -
scripts/tracetool/format/tcg_helper_wrapper_h.py | 1 -
scripts/tracetool/format/ust_events_c.py         | 1 -
scripts/tracetool/format/ust_events_h.py         | 1 -
scripts/tracetool/transform.py                   | 1 -
scripts/tracetool/vcpu.py                        | 1 -
scripts/vmstate-static-checker.py                | 2 +-
tests/acceptance/virtio_seg_max_adjust.py        | 1 -
tests/acceptance/x86_cpu_model_versions.py       | 1 -
tests/docker/travis.py                           | 2 +-
tests/qapi-schema/test-qapi.py                   | 2 +-
tests/qemu-iotests/030                           | 2 +-
tests/qemu-iotests/040                           | 2 +-
tests/qemu-iotests/041                           | 2 +-
tests/qemu-iotests/044                           | 2 +-
tests/qemu-iotests/045                           | 2 +-
tests/qemu-iotests/055                           | 2 +-
tests/qemu-iotests/056                           | 2 +-
tests/qemu-iotests/057                           | 2 +-
tests/qemu-iotests/065                           | 2 +-
tests/qemu-iotests/093                           | 2 +-
tests/qemu-iotests/096                           | 2 +-
tests/qemu-iotests/118                           | 2 +-
tests/qemu-iotests/124                           | 2 +-
tests/qemu-iotests/129                           | 2 +-
tests/qemu-iotests/132                           | 2 +-
tests/qemu-iotests/136                           | 2 +-
tests/qemu-iotests/139                           | 2 +-
tests/qemu-iotests/147                           | 2 +-
tests/qemu-iotests/148                           | 2 +-
tests/qemu-iotests/149                           | 2 +-
tests/qemu-iotests/151                           | 2 +-
tests/qemu-iotests/152                           | 2 +-
tests/qemu-iotests/155                           | 2 +-
tests/qemu-iotests/163                           | 2 +-
tests/qemu-iotests/165                           | 2 +-
tests/qemu-iotests/169                           | 2 +-
tests/qemu-iotests/194                           | 2 +-
tests/qemu-iotests/196                           | 2 +-
tests/qemu-iotests/199                           | 2 +-
tests/qemu-iotests/202                           | 2 +-
tests/qemu-iotests/203                           | 2 +-
tests/qemu-iotests/205                           | 2 +-
tests/qemu-iotests/206                           | 2 +-
tests/qemu-iotests/207                           | 2 +-
tests/qemu-iotests/208                           | 2 +-
tests/qemu-iotests/209                           | 2 +-
tests/qemu-iotests/210                           | 2 +-
tests/qemu-iotests/211                           | 2 +-
tests/qemu-iotests/212                           | 2 +-
tests/qemu-iotests/213                           | 2 +-
tests/qemu-iotests/216                           | 2 +-
tests/qemu-iotests/218                           | 2 +-
tests/qemu-iotests/219                           | 2 +-
tests/qemu-iotests/222                           | 2 +-
tests/qemu-iotests/224                           | 2 +-
tests/qemu-iotests/228                           | 2 +-
tests/qemu-iotests/234                           | 2 +-
tests/qemu-iotests/235                           | 2 +-
tests/qemu-iotests/236                           | 2 +-
tests/qemu-iotests/237                           | 2 +-
tests/qemu-iotests/238                           | 2 +-
tests/qemu-iotests/242                           | 2 +-
tests/qemu-iotests/245                           | 2 +-
tests/qemu-iotests/246                           | 2 +-
tests/qemu-iotests/248                           | 2 +-
tests/qemu-iotests/254                           | 2 +-
tests/qemu-iotests/255                           | 2 +-
tests/qemu-iotests/256                           | 2 +-
tests/qemu-iotests/257                           | 2 +-
tests/qemu-iotests/258                           | 2 +-
tests/qemu-iotests/260                           | 2 +-
tests/qemu-iotests/262                           | 2 +-
tests/qemu-iotests/264                           | 2 +-
tests/qemu-iotests/266                           | 2 +-
tests/qemu-iotests/277                           | 2 +-
tests/qemu-iotests/280                           | 2 +-
tests/qemu-iotests/281                           | 2 +-
tests/qemu-iotests/check                         | 2 +-
tests/qemu-iotests/nbd-fault-injector.py         | 2 +-
tests/qemu-iotests/qcow2.py                      | 2 +-
tests/qemu-iotests/qed.py                        | 2 +-
tests/vm/basevm.py                               | 1 -
tests/vm/centos                                  | 2 +-
tests/vm/fedora                                  | 2 +-
tests/vm/freebsd                                 | 2 +-
tests/vm/netbsd                                  | 2 +-
tests/vm/openbsd                                 | 2 +-
tests/vm/ubuntu.i386                             | 2 +-
121 files changed, 95 insertions(+), 120 deletions(-)
mode change 100755 => 100644 tests/acceptance/virtio_seg_max_adjust.py
mode change 100755 => 100644 tests/vm/basevm.py
[PATCH 00/10] python: Explicit usage of Python 3
Posted by Philippe Mathieu-Daudé 5 years, 9 months ago
Hello,

These are mechanical sed patches used to convert the
code base to Python 3, as suggested on this thread:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg675024.html

Regards,

Phil.

Philippe Mathieu-Daudé (10):
  scripts: Explicit usage of Python 3
  tests/qemu-iotests: Explicit usage of Python 3
  tests: Explicit usage of Python 3
  scripts/minikconf: Explicit usage of Python 3
  tests/acceptance: Remove shebang header
  scripts/tracetool: Remove shebang header
  tests/vm: Remove shebang header
  tests/qemu-iotests: Explicit usage of Python 3
  scripts: Explicit usage of Python 3
  tests/qemu-iotests/check: Update to match Python 3 interpreter

 scripts/analyse-9p-simpletrace.py                | 2 +-
 scripts/analyse-locks-simpletrace.py             | 2 +-
 scripts/decodetree.py                            | 2 +-
 scripts/device-crash-test                        | 2 +-
 scripts/kvm/kvm_flightrecorder                   | 2 +-
 scripts/minikconf.py                             | 1 +
 scripts/qapi-gen.py                              | 2 +-
 scripts/qmp/qemu-ga-client                       | 2 +-
 scripts/qmp/qmp                                  | 2 +-
 scripts/qmp/qmp-shell                            | 2 +-
 scripts/qmp/qom-fuse                             | 2 +-
 scripts/render_block_graph.py                    | 2 +-
 scripts/replay-dump.py                           | 2 +-
 scripts/simpletrace.py                           | 2 +-
 scripts/tracetool.py                             | 2 +-
 scripts/tracetool/__init__.py                    | 1 -
 scripts/tracetool/backend/__init__.py            | 1 -
 scripts/tracetool/backend/dtrace.py              | 1 -
 scripts/tracetool/backend/ftrace.py              | 1 -
 scripts/tracetool/backend/log.py                 | 1 -
 scripts/tracetool/backend/simple.py              | 1 -
 scripts/tracetool/backend/syslog.py              | 1 -
 scripts/tracetool/backend/ust.py                 | 1 -
 scripts/tracetool/format/__init__.py             | 1 -
 scripts/tracetool/format/c.py                    | 1 -
 scripts/tracetool/format/d.py                    | 1 -
 scripts/tracetool/format/h.py                    | 1 -
 scripts/tracetool/format/log_stap.py             | 1 -
 scripts/tracetool/format/simpletrace_stap.py     | 1 -
 scripts/tracetool/format/stap.py                 | 1 -
 scripts/tracetool/format/tcg_h.py                | 1 -
 scripts/tracetool/format/tcg_helper_c.py         | 1 -
 scripts/tracetool/format/tcg_helper_h.py         | 1 -
 scripts/tracetool/format/tcg_helper_wrapper_h.py | 1 -
 scripts/tracetool/format/ust_events_c.py         | 1 -
 scripts/tracetool/format/ust_events_h.py         | 1 -
 scripts/tracetool/transform.py                   | 1 -
 scripts/tracetool/vcpu.py                        | 1 -
 scripts/vmstate-static-checker.py                | 2 +-
 tests/acceptance/virtio_seg_max_adjust.py        | 1 -
 tests/acceptance/x86_cpu_model_versions.py       | 1 -
 tests/docker/travis.py                           | 2 +-
 tests/qapi-schema/test-qapi.py                   | 2 +-
 tests/qemu-iotests/030                           | 2 +-
 tests/qemu-iotests/040                           | 2 +-
 tests/qemu-iotests/041                           | 2 +-
 tests/qemu-iotests/044                           | 2 +-
 tests/qemu-iotests/045                           | 2 +-
 tests/qemu-iotests/055                           | 2 +-
 tests/qemu-iotests/056                           | 2 +-
 tests/qemu-iotests/057                           | 2 +-
 tests/qemu-iotests/065                           | 2 +-
 tests/qemu-iotests/093                           | 2 +-
 tests/qemu-iotests/096                           | 2 +-
 tests/qemu-iotests/118                           | 2 +-
 tests/qemu-iotests/124                           | 2 +-
 tests/qemu-iotests/129                           | 2 +-
 tests/qemu-iotests/132                           | 2 +-
 tests/qemu-iotests/136                           | 2 +-
 tests/qemu-iotests/139                           | 2 +-
 tests/qemu-iotests/147                           | 2 +-
 tests/qemu-iotests/148                           | 2 +-
 tests/qemu-iotests/149                           | 2 +-
 tests/qemu-iotests/151                           | 2 +-
 tests/qemu-iotests/152                           | 2 +-
 tests/qemu-iotests/155                           | 2 +-
 tests/qemu-iotests/163                           | 2 +-
 tests/qemu-iotests/165                           | 2 +-
 tests/qemu-iotests/169                           | 2 +-
 tests/qemu-iotests/194                           | 2 +-
 tests/qemu-iotests/196                           | 2 +-
 tests/qemu-iotests/199                           | 2 +-
 tests/qemu-iotests/202                           | 2 +-
 tests/qemu-iotests/203                           | 2 +-
 tests/qemu-iotests/205                           | 2 +-
 tests/qemu-iotests/206                           | 2 +-
 tests/qemu-iotests/207                           | 2 +-
 tests/qemu-iotests/208                           | 2 +-
 tests/qemu-iotests/209                           | 2 +-
 tests/qemu-iotests/210                           | 2 +-
 tests/qemu-iotests/211                           | 2 +-
 tests/qemu-iotests/212                           | 2 +-
 tests/qemu-iotests/213                           | 2 +-
 tests/qemu-iotests/216                           | 2 +-
 tests/qemu-iotests/218                           | 2 +-
 tests/qemu-iotests/219                           | 2 +-
 tests/qemu-iotests/222                           | 2 +-
 tests/qemu-iotests/224                           | 2 +-
 tests/qemu-iotests/228                           | 2 +-
 tests/qemu-iotests/234                           | 2 +-
 tests/qemu-iotests/235                           | 2 +-
 tests/qemu-iotests/236                           | 2 +-
 tests/qemu-iotests/237                           | 2 +-
 tests/qemu-iotests/238                           | 2 +-
 tests/qemu-iotests/242                           | 2 +-
 tests/qemu-iotests/245                           | 2 +-
 tests/qemu-iotests/246                           | 2 +-
 tests/qemu-iotests/248                           | 2 +-
 tests/qemu-iotests/254                           | 2 +-
 tests/qemu-iotests/255                           | 2 +-
 tests/qemu-iotests/256                           | 2 +-
 tests/qemu-iotests/257                           | 2 +-
 tests/qemu-iotests/258                           | 2 +-
 tests/qemu-iotests/260                           | 2 +-
 tests/qemu-iotests/262                           | 2 +-
 tests/qemu-iotests/264                           | 2 +-
 tests/qemu-iotests/266                           | 2 +-
 tests/qemu-iotests/277                           | 2 +-
 tests/qemu-iotests/280                           | 2 +-
 tests/qemu-iotests/281                           | 2 +-
 tests/qemu-iotests/check                         | 2 +-
 tests/qemu-iotests/nbd-fault-injector.py         | 2 +-
 tests/qemu-iotests/qcow2.py                      | 2 +-
 tests/qemu-iotests/qed.py                        | 2 +-
 tests/vm/basevm.py                               | 1 -
 tests/vm/centos                                  | 2 +-
 tests/vm/fedora                                  | 2 +-
 tests/vm/freebsd                                 | 2 +-
 tests/vm/netbsd                                  | 2 +-
 tests/vm/openbsd                                 | 2 +-
 tests/vm/ubuntu.i386                             | 2 +-
 121 files changed, 95 insertions(+), 120 deletions(-)
 mode change 100755 => 100644 tests/acceptance/virtio_seg_max_adjust.py
 mode change 100755 => 100644 tests/vm/basevm.py

-- 
2.21.1


Re: [PATCH 00/10] python: Explicit usage of Python 3
Posted by Stefan Hajnoczi 5 years, 9 months ago
On Thu, Jan 30, 2020 at 12:13:52AM +0100, Philippe Mathieu-Daudé wrote:
> Hello,
> 
> These are mechanical sed patches used to convert the
> code base to Python 3, as suggested on this thread:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg675024.html
> 
> Regards,
> 
> Phil.
> 
> Philippe Mathieu-Daudé (10):
>   scripts: Explicit usage of Python 3
>   tests/qemu-iotests: Explicit usage of Python 3
>   tests: Explicit usage of Python 3
>   scripts/minikconf: Explicit usage of Python 3
>   tests/acceptance: Remove shebang header
>   scripts/tracetool: Remove shebang header
>   tests/vm: Remove shebang header
>   tests/qemu-iotests: Explicit usage of Python 3
>   scripts: Explicit usage of Python 3
>   tests/qemu-iotests/check: Update to match Python 3 interpreter
> 
>  scripts/analyse-9p-simpletrace.py                | 2 +-
>  scripts/analyse-locks-simpletrace.py             | 2 +-
>  scripts/decodetree.py                            | 2 +-
>  scripts/device-crash-test                        | 2 +-
>  scripts/kvm/kvm_flightrecorder                   | 2 +-
>  scripts/minikconf.py                             | 1 +
>  scripts/qapi-gen.py                              | 2 +-
>  scripts/qmp/qemu-ga-client                       | 2 +-
>  scripts/qmp/qmp                                  | 2 +-
>  scripts/qmp/qmp-shell                            | 2 +-
>  scripts/qmp/qom-fuse                             | 2 +-
>  scripts/render_block_graph.py                    | 2 +-
>  scripts/replay-dump.py                           | 2 +-
>  scripts/simpletrace.py                           | 2 +-
>  scripts/tracetool.py                             | 2 +-
>  scripts/tracetool/__init__.py                    | 1 -
>  scripts/tracetool/backend/__init__.py            | 1 -
>  scripts/tracetool/backend/dtrace.py              | 1 -
>  scripts/tracetool/backend/ftrace.py              | 1 -
>  scripts/tracetool/backend/log.py                 | 1 -
>  scripts/tracetool/backend/simple.py              | 1 -
>  scripts/tracetool/backend/syslog.py              | 1 -
>  scripts/tracetool/backend/ust.py                 | 1 -
>  scripts/tracetool/format/__init__.py             | 1 -
>  scripts/tracetool/format/c.py                    | 1 -
>  scripts/tracetool/format/d.py                    | 1 -
>  scripts/tracetool/format/h.py                    | 1 -
>  scripts/tracetool/format/log_stap.py             | 1 -
>  scripts/tracetool/format/simpletrace_stap.py     | 1 -
>  scripts/tracetool/format/stap.py                 | 1 -
>  scripts/tracetool/format/tcg_h.py                | 1 -
>  scripts/tracetool/format/tcg_helper_c.py         | 1 -
>  scripts/tracetool/format/tcg_helper_h.py         | 1 -
>  scripts/tracetool/format/tcg_helper_wrapper_h.py | 1 -
>  scripts/tracetool/format/ust_events_c.py         | 1 -
>  scripts/tracetool/format/ust_events_h.py         | 1 -
>  scripts/tracetool/transform.py                   | 1 -
>  scripts/tracetool/vcpu.py                        | 1 -
>  scripts/vmstate-static-checker.py                | 2 +-
>  tests/acceptance/virtio_seg_max_adjust.py        | 1 -
>  tests/acceptance/x86_cpu_model_versions.py       | 1 -
>  tests/docker/travis.py                           | 2 +-
>  tests/qapi-schema/test-qapi.py                   | 2 +-
>  tests/qemu-iotests/030                           | 2 +-
>  tests/qemu-iotests/040                           | 2 +-
>  tests/qemu-iotests/041                           | 2 +-
>  tests/qemu-iotests/044                           | 2 +-
>  tests/qemu-iotests/045                           | 2 +-
>  tests/qemu-iotests/055                           | 2 +-
>  tests/qemu-iotests/056                           | 2 +-
>  tests/qemu-iotests/057                           | 2 +-
>  tests/qemu-iotests/065                           | 2 +-
>  tests/qemu-iotests/093                           | 2 +-
>  tests/qemu-iotests/096                           | 2 +-
>  tests/qemu-iotests/118                           | 2 +-
>  tests/qemu-iotests/124                           | 2 +-
>  tests/qemu-iotests/129                           | 2 +-
>  tests/qemu-iotests/132                           | 2 +-
>  tests/qemu-iotests/136                           | 2 +-
>  tests/qemu-iotests/139                           | 2 +-
>  tests/qemu-iotests/147                           | 2 +-
>  tests/qemu-iotests/148                           | 2 +-
>  tests/qemu-iotests/149                           | 2 +-
>  tests/qemu-iotests/151                           | 2 +-
>  tests/qemu-iotests/152                           | 2 +-
>  tests/qemu-iotests/155                           | 2 +-
>  tests/qemu-iotests/163                           | 2 +-
>  tests/qemu-iotests/165                           | 2 +-
>  tests/qemu-iotests/169                           | 2 +-
>  tests/qemu-iotests/194                           | 2 +-
>  tests/qemu-iotests/196                           | 2 +-
>  tests/qemu-iotests/199                           | 2 +-
>  tests/qemu-iotests/202                           | 2 +-
>  tests/qemu-iotests/203                           | 2 +-
>  tests/qemu-iotests/205                           | 2 +-
>  tests/qemu-iotests/206                           | 2 +-
>  tests/qemu-iotests/207                           | 2 +-
>  tests/qemu-iotests/208                           | 2 +-
>  tests/qemu-iotests/209                           | 2 +-
>  tests/qemu-iotests/210                           | 2 +-
>  tests/qemu-iotests/211                           | 2 +-
>  tests/qemu-iotests/212                           | 2 +-
>  tests/qemu-iotests/213                           | 2 +-
>  tests/qemu-iotests/216                           | 2 +-
>  tests/qemu-iotests/218                           | 2 +-
>  tests/qemu-iotests/219                           | 2 +-
>  tests/qemu-iotests/222                           | 2 +-
>  tests/qemu-iotests/224                           | 2 +-
>  tests/qemu-iotests/228                           | 2 +-
>  tests/qemu-iotests/234                           | 2 +-
>  tests/qemu-iotests/235                           | 2 +-
>  tests/qemu-iotests/236                           | 2 +-
>  tests/qemu-iotests/237                           | 2 +-
>  tests/qemu-iotests/238                           | 2 +-
>  tests/qemu-iotests/242                           | 2 +-
>  tests/qemu-iotests/245                           | 2 +-
>  tests/qemu-iotests/246                           | 2 +-
>  tests/qemu-iotests/248                           | 2 +-
>  tests/qemu-iotests/254                           | 2 +-
>  tests/qemu-iotests/255                           | 2 +-
>  tests/qemu-iotests/256                           | 2 +-
>  tests/qemu-iotests/257                           | 2 +-
>  tests/qemu-iotests/258                           | 2 +-
>  tests/qemu-iotests/260                           | 2 +-
>  tests/qemu-iotests/262                           | 2 +-
>  tests/qemu-iotests/264                           | 2 +-
>  tests/qemu-iotests/266                           | 2 +-
>  tests/qemu-iotests/277                           | 2 +-
>  tests/qemu-iotests/280                           | 2 +-
>  tests/qemu-iotests/281                           | 2 +-
>  tests/qemu-iotests/check                         | 2 +-
>  tests/qemu-iotests/nbd-fault-injector.py         | 2 +-
>  tests/qemu-iotests/qcow2.py                      | 2 +-
>  tests/qemu-iotests/qed.py                        | 2 +-
>  tests/vm/basevm.py                               | 1 -
>  tests/vm/centos                                  | 2 +-
>  tests/vm/fedora                                  | 2 +-
>  tests/vm/freebsd                                 | 2 +-
>  tests/vm/netbsd                                  | 2 +-
>  tests/vm/openbsd                                 | 2 +-
>  tests/vm/ubuntu.i386                             | 2 +-
>  121 files changed, 95 insertions(+), 120 deletions(-)
>  mode change 100755 => 100644 tests/acceptance/virtio_seg_max_adjust.py
>  mode change 100755 => 100644 tests/vm/basevm.py
> 
> -- 
> 2.21.1
> 

Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
Re: [PATCH 00/10] python: Explicit usage of Python 3
Posted by Vladimir Sementsov-Ogievskiy 5 years, 9 months ago
First, thanks for handling this!

30.01.2020 2:13, Philippe Mathieu-Daudé wrote:
> Hello,
> 
> These are mechanical sed patches used to convert the
> code base to Python 3, as suggested on this thread:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg675024.html
> 
> Regards,
> 
> Phil.
> 
> Philippe Mathieu-Daudé (10):
>    scripts: Explicit usage of Python 3
>    tests/qemu-iotests: Explicit usage of Python 3
>    tests: Explicit usage of Python 3
>    scripts/minikconf: Explicit usage of Python 3
>    tests/acceptance: Remove shebang header
>    scripts/tracetool: Remove shebang header
>    tests/vm: Remove shebang header
>    tests/qemu-iotests: Explicit usage of Python 3
>    scripts: Explicit usage of Python 3
>    tests/qemu-iotests/check: Update to match Python 3 interpreter
> 

Could you please not use same subject for different patches? Such things are hard to manage during patch porting from version to version.

Also, will you update checkpatch.pl, to avoid appearing unversioned python again?

-- 
Best regards,
Vladimir

Re: [PATCH 00/10] python: Explicit usage of Python 3
Posted by Philippe Mathieu-Daudé 5 years, 9 months ago
On 1/30/20 3:02 PM, Vladimir Sementsov-Ogievskiy wrote:
> First, thanks for handling this!
> 
> 30.01.2020 2:13, Philippe Mathieu-Daudé wrote:
>> Hello,
>>
>> These are mechanical sed patches used to convert the
>> code base to Python 3, as suggested on this thread:
>> https://www.mail-archive.com/qemu-devel@nongnu.org/msg675024.html
>>
>> Regards,
>>
>> Phil.
>>
>> Philippe Mathieu-Daudé (10):
>>    scripts: Explicit usage of Python 3
>>    tests/qemu-iotests: Explicit usage of Python 3
>>    tests: Explicit usage of Python 3
>>    scripts/minikconf: Explicit usage of Python 3
>>    tests/acceptance: Remove shebang header
>>    scripts/tracetool: Remove shebang header
>>    tests/vm: Remove shebang header
>>    tests/qemu-iotests: Explicit usage of Python 3
>>    scripts: Explicit usage of Python 3
>>    tests/qemu-iotests/check: Update to match Python 3 interpreter
>>
> 
> Could you please not use same subject for different patches? Such things 
> are hard to manage during patch porting from version to version.

I can change but I'm not understanding what you want.

> 
> Also, will you update checkpatch.pl, to avoid appearing unversioned 
> python again?

I'm not sure I can because checkpatch.pl is written in Perl, but I'll try.


Re: [PATCH 00/10] python: Explicit usage of Python 3
Posted by Vladimir Sementsov-Ogievskiy 5 years, 9 months ago
30.01.2020 18:04, Philippe Mathieu-Daudé wrote:
> On 1/30/20 3:02 PM, Vladimir Sementsov-Ogievskiy wrote:
>> First, thanks for handling this!
>>
>> 30.01.2020 2:13, Philippe Mathieu-Daudé wrote:
>>> Hello,
>>>
>>> These are mechanical sed patches used to convert the
>>> code base to Python 3, as suggested on this thread:
>>> https://www.mail-archive.com/qemu-devel@nongnu.org/msg675024.html
>>>
>>> Regards,
>>>
>>> Phil.
>>>
>>> Philippe Mathieu-Daudé (10):
>>>    scripts: Explicit usage of Python 3
>>>    tests/qemu-iotests: Explicit usage of Python 3
>>>    tests: Explicit usage of Python 3
>>>    scripts/minikconf: Explicit usage of Python 3
>>>    tests/acceptance: Remove shebang header
>>>    scripts/tracetool: Remove shebang header
>>>    tests/vm: Remove shebang header
>>>    tests/qemu-iotests: Explicit usage of Python 3
>>>    scripts: Explicit usage of Python 3
>>>    tests/qemu-iotests/check: Update to match Python 3 interpreter
>>>
>>
>> Could you please not use same subject for different patches? Such things are hard to manage during patch porting from version to version.
> 
> I can change but I'm not understanding what you want.

I just want different subjects for different patches (you use the same for 01 and 09, for 02 and 08), if it possible :) But I don't insist.

> 
>>
>> Also, will you update checkpatch.pl, to avoid appearing unversioned python again?
> 
> I'm not sure I can because checkpatch.pl is written in Perl, but I'll try.
> 


-- 
Best regards,
Vladimir