]> git.ipfire.org Git - thirdparty/u-boot.git/commitdiff
tools: rmboard.py: Fix conversion from run_pipe to new helper
authorTom Rini <trini@konsulko.com>
Fri, 20 Jun 2025 16:25:02 +0000 (10:25 -0600)
committerTom Rini <trini@konsulko.com>
Fri, 27 Jun 2025 16:02:19 +0000 (10:02 -0600)
When this utility was converted from run_pipe and to the new output
helper, two problems were introduced. First, the conversion for calling
"git rm -f" wasn't correct. Change this to match the other conversions.
Second, the final call we do we need to construct the list because we
print that command for the user to use to inspect remaining references.

Fixes: 3d094ce28a22 ("u_boot_pylib: Add a function to run a single command")
Signed-off-by: Tom Rini <trini@konsulko.com>
tools/rmboard.py

index 594fd89b8d789bb543dd1ef7c7fba5b584a24ee7..21d68c5726181de1e6d14b55321b43ab9892b7f1 100755 (executable)
@@ -112,8 +112,7 @@ def rm_board(board):
                 rm_kconfig_include(fname)
 
     # Remove unwanted files
-    cmd = ['git', 'rm', '-r'] + real
-    stdout = command.output(*cmd, capture=True)
+    stdout = command.output('git', 'rm', '-r', *real)
 
     ## Change the messages as needed
     msg = '''arm: Remove %s board
@@ -130,7 +129,8 @@ Remove it.
 
     # Check if the board is mentioned anywhere else. The user will need to deal
     # with this
-    print(command.output('git', 'grep', '-il', board, raise_on_error=False))
+    cmd = ['git', 'grep', '-il', board]
+    print(command.output(*cmd, raise_on_error=False))
     print(' '.join(cmd))
 
 for board in sys.argv[1:]: