On Thu, Aug 15, 2024 at 02:25:15PM -0600, Simon Glass wrote:

> Tidy up most of these warnings. Remaining are four of these:
> 
>    R0914: Too many local variables
> 
> which can only by fixed by splitting things into functions, so that is
> left for another time.
> 
> Part of this change was done by the flynt tool.
> 
> Signed-off-by: Simon Glass <s...@chromium.org>
[snip]
> -    u_boot_utils.run_and_log(cons, 'qemu-img create %s 20M' % fname)
> -    u_boot_utils.run_and_log(cons, 'sudo sfdisk %s' % fname,
> +    u_boot_utils.run_and_log(cons, f'qemu-img create {fname} 20M')
> +    u_boot_utils.run_and_log(cons, f'sudo sfdisk {fname}',
>                               stdin=spec.encode('utf-8'))

This should be re-done on top of Richard's work (once he fixes the
pylint issues it introduces) at
https://patchwork.ozlabs.org/project/uboot/patch/20240802093322.15240-1-rich...@nod.at/

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to