Make it more clear that the missing external blob is optional in the printed warning message.
Signed-off-by: Jonas Karlman <jo...@kwiboo.se> --- v2: - New patch 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 25e66814837d..3560cadba4c2 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 e53181afb78a..152f800c676c 100644 --- a/tools/binman/ftest.py +++ b/tools/binman/ftest.py @@ -6330,7 +6330,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.41.0