Message ID | 20200129231402.23384-1-philmd@redhat.com (mailing list archive) |
---|---|
Headers | show |
Series | python: Explicit usage of Python 3 | expand |
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?
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.
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. >
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>