On Fri, 21 Jun 2024 at 04:36, Simon Glass <s...@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 <s...@chromium.org> > Fixes: b617611b27a ("binman: capsule: Add support for generating...") > ---
Acked-by: Sughosh Ganu <sughosh.g...@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 >