From 05dec37488e27da292c8ff51e936b46eac708bd4 Mon Sep 17 00:00:00 2001 From: Jonas Karlman Date: Tue, 18 Jul 2023 20:34:34 +0000 Subject: [PATCH] binman: Update missing optional external blob warning text Make it more clear that the missing external blob is optional in the printed warning message. Signed-off-by: Jonas Karlman Reviewed-by: Simon Glass --- tools/binman/control.py | 2 +- tools/binman/ftest.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/binman/control.py b/tools/binman/control.py index d1ee1d69a9..8bc666a0c7 100644 --- a/tools/binman/control.py +++ b/tools/binman/control.py @@ -674,7 +674,7 @@ def ProcessImage(image, update_fdt, write_map, get_contents=True, image.CheckOptional(optional_list) if optional_list: tout.warning( - "Image '%s' is missing external blobs but is still functional: %s" % + "Image '%s' is missing optional external blobs but is still functional: %s" % (image.name, ' '.join([e.name for e in optional_list]))) _ShowHelpForMissingBlobs(optional_list) diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index 1cfa349d38..36c72bed35 100644 --- a/tools/binman/ftest.py +++ b/tools/binman/ftest.py @@ -6390,7 +6390,7 @@ fdt fdtmap Extract the devicetree blob from the fdtmap err = stderr.getvalue() self.assertRegex( err, - "Image '.*' is missing external blobs but is still functional: missing") + "Image '.*' is missing optional external blobs but is still functional: missing") def testSectionInner(self): """Test an inner section with a size""" -- 2.39.5