On Fri, 21 Jun 2024 at 04:36, Simon Glass sjg@chromium.org wrote:
There is no need to remove input files. It makes it harder to diagnose failures. Keep the payload file.
There is no test for this condition, but one could be added.
Signed-off-by: Simon Glass sjg@chromium.org Fixes: b617611b27a ("binman: capsule: Add support for generating...")
Acked-by: Sughosh Ganu sughosh.ganu@linaro.org
Although I don't think this warrants a Fixes tag. It is not fixing any issue as such.
-sughosh
(no changes since v1)
tools/binman/etype/efi_capsule.py | 1 - 1 file changed, 1 deletion(-)
diff --git a/tools/binman/etype/efi_capsule.py b/tools/binman/etype/efi_capsule.py index 47da5da324b..03e55cbc4d9 100644 --- a/tools/binman/etype/efi_capsule.py +++ b/tools/binman/etype/efi_capsule.py @@ -148,7 +148,6 @@ class Entry_efi_capsule(Entry_section): self.fw_version, self.oem_flags) if ret is not None:
os.remove(payload) return tools.read_file(capsule_fname) else: # Bintool is missing; just use the input data as the output-- 2.34.1