diff mbox series

[Concept,01/16] buildman: Fix f-string in the delta column header

Message ID 20260109015323.3411528-2-sjg@u-boot.org
State New
Headers
Series test: pxe: Add some decent tests for the PXE/extlinux parser |

Commit Message

Simon Glass Jan. 9, 2026, 1:53 a.m. UTC 
  From: Simon Glass <simon.glass@canonical.com>

The '+' sign format specifier is only valid for numeric types to force
showing the sign. Using it with a string literal causes a ValueError.

Remove the invalid '+' from the "delta" column header format specifier.

Fixes: 34567e06b6a7 ("buildman: Convert to f-strings in builder.py")
Co-developed-by: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

 tools/buildman/builder.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index d92748b01e9..b7343b57c0d 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -1219,7 +1219,7 @@  class Builder:
                f'{args[0]}/{args[1]}, grow: {args[2]}/{args[3]} bytes: '
                f'{args[4]}/{args[5]} ({args[6]})')
         tprint(f'{indent}  {"function":<38s} {"old":>7s} {"new":>7s} '
-               f'{"delta":>+7s}')
+               f'{"delta":>7s}')
         for diff, name in delta:
             if diff:
                 color = self.col.RED if diff > 0 else self.col.GREEN