[PATCH] u_boot_pylib: Ensure subprocess is closed down
Simon Glass
sjg at chromium.org
Thu Nov 2 23:49:05 CET 2023
It isn't clear why we need to have two different paths for closing down
the pipe. Unify them and use the Python to avoid this warning:
subprocess.py:1127: ResourceWarning: subprocess 83531 is still running
Note that this code appears to originally have come from [1] and was
committed into the ChromeOS chromiumos/platform/crosutils repo in the
bin/cros_image_to_target.py file. The addition of the extra code path
came later, so that is chosen for the fixes tag.
[1] https://codereview.chromium.org/3391008
Signed-off-by: Simon Glass <sjg at chromium.org>
Fixes: a10fd93cbc patman: Make command methods return a CommandResult
---
tools/u_boot_pylib/command.py | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
Applied to u-boot-dm, thanks!
More information about the U-Boot
mailing list