[PATCH v2] efi_loader: Disable ANSI output for tests

Tom Rini trini at konsulko.com
Sat May 10 17:05:23 CEST 2025


On Sat, May 10, 2025 at 02:54:38PM +0200, Simon Glass wrote:
> We don't want ANSI escape-sequences written in tests since it is a pain
> to check the output with ut_assert_nextline() et al.
> 
> Provide a way to tests to request that these characters not be sent.
> 
> Add a proper function comment while we are here, to encourage others.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
> Changes in v2:
> - Use the term 'ANSI escape-sequences'
> 
>  include/efi_loader.h         | 21 ++++++++++++++++++++-
>  lib/efi_loader/efi_console.c | 26 +++++++++++++++++---------
>  2 files changed, 37 insertions(+), 10 deletions(-)

Since we keep coming back to this, here's the email where *why* there
are ANSI escape sequences is explained along with Heinrich's suggestion
of a fix for your concern:
https://lore.kernel.org/all/ba46d2c2-38ac-411a-a48c-429295cf6efb@gmx.de/

I assume that this, rather than repeating and reposting this solution
again is still what Heinrich prefers.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20250510/862f56df/attachment.sig>


More information about the U-Boot mailing list