[libvirt PATCH] tools: Fix style issues in virt-qemu-sev-validate

Andrea Bolognani abologna at redhat.com
Thu Dec 8 18:04:44 UTC 2022


On Thu, Dec 08, 2022 at 05:09:39PM +0000, Daniel P. Berrangé wrote:
> On Thu, Dec 08, 2022 at 06:06:23PM +0100, Andrea Bolognani wrote:
> > The script had an incorrect interpreter line until commit
> > f6a19d7264bb, so the flake8 check would not realize it needed
> > to pick it up and these issues, some of which were present it
> > the very first version that was committed, were not being
> > reported.
>
> Hmm, if flake8 is going to be restricted to /usr/bin/env python,
> then we ought to have a syntax check to forbid /usr/bin/python*,
> to prevent missing the flake8 check.

That would certainly be good to have.

Regardless of the flake8 check, you simply don't want to hardcode a
path to the Python interpreter there, as it will prevent the script
from working on FreeBSD and macOS at the very least.

-- 
Andrea Bolognani / Red Hat / Virtualization



More information about the libvir-list mailing list