On Wed, Oct 08, 2025 at 01:58:11PM -0400, Stefan Hajnoczi wrote: > On Wed, Oct 08, 2025 at 08:35:41AM +0200, Paolo Bonzini wrote: > > Sort imports automatically, to keep the coding style more uniform. > > > > Signed-off-by: Paolo Bonzini <[email protected]> > > --- > > python/tests/tracetool-isort.sh | 4 ++++ > > scripts/tracetool.py | 5 ++--- > > scripts/tracetool/backend/dtrace.py | 1 - > > scripts/tracetool/backend/ftrace.py | 3 +-- > > scripts/tracetool/backend/log.py | 3 +-- > > scripts/tracetool/backend/simple.py | 1 - > > scripts/tracetool/backend/syslog.py | 3 +-- > > scripts/tracetool/backend/ust.py | 1 - > > scripts/tracetool/format/d.py | 2 +- > > scripts/tracetool/format/log_stap.py | 1 - > > scripts/tracetool/format/stap.py | 1 - > > 11 files changed, 10 insertions(+), 15 deletions(-) > > create mode 100755 python/tests/tracetool-isort.sh > > > > diff --git a/python/tests/tracetool-isort.sh > > b/python/tests/tracetool-isort.sh > > new file mode 100755 > > index 00000000000..b23f3d48448 > > --- /dev/null > > +++ b/python/tests/tracetool-isort.sh > > @@ -0,0 +1,4 @@ > > +#!/bin/sh -e > > +# SPDX-License-Identifier: GPL-2.0-or-later > > + > > +python3 -m isort --sp . -c ../scripts/tracetool/ > > I wonder why python/tests/isort.sh doesn't already cover this with its > `python3 -m isort -c scripts/` line? > > Also, why the --settings-path (--sp) option that python/tests/isort.sh > doesn't use? > > It would be great to have just 1 script that runs isort on all Python > code in QEMU.
IMHO all of the shell scripts should really just go away. They live in their own world outside the rest of our meson test setup which means developers rarely even know they exist, let alone run them on submissions. I've proposed removing them in favour of meson rules earlier this year: https://lists.gnu.org/archive/html/qemu-devel/2025-02/msg04920.html With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
