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
signature.asc
Description: PGP signature