Re: [PATCH v3 0/2] cmd: ufetch: print Net feature when NET_LWIP=y, prettyfy if NET=n

[Casey Connolly <casey.connolly@linaro.org>]

On 05/05/2026 11:14, Quentin Schulz wrote:
> Make sure the Net feature is printed when NET_LWIP is enabled, not only
> when NET_LEGACY is.
> 
> Only print comma and following whitespace as separator if there was a
> feature printed before feature to be printed.
> 
> Signed-off-by: Quentin Schulz <quentin.schulz@cherry.de>

Reviewed-by: Casey Connolly <casey.connolly@linaro.org>

> ---
> Changes in v3:
> - migrated from bool plus ternary operator for setting the separator per
>   printf to a const char* which contains the separator to print, as
>   suggested by Simon,
> - Link to v2: https://patch.msgid.link/20260504-ufetch-net-v2-0-01bec4ca7169@cherry.de
> 
> Changes in v2:
> - added R-b trailers,
> - fixed spurious colon character before separator in cat,
> - Link to v1: https://patch.msgid.link/20260428-ufetch-net-v1-0-5d58c7618239@cherry.de
> 
> ---
> Quentin Schulz (2):
>       cmd: ufetch: show net feature when NET_LWIP is selected
>       cmd: ufetch: only show comma separator if there was a previous feature
> 
>  cmd/ufetch.c | 29 ++++++++++++++++++++---------
>  1 file changed, 20 insertions(+), 9 deletions(-)
> ---
> base-commit: 57460fde4ba84d67de4066ee1141e89e8139ed23
> change-id: 20260428-ufetch-net-1ba7553a20c7
> 
> Best regards,
> --  
> Quentin Schulz <quentin.schulz@cherry.de>
> 

-- 
// Casey (she/her)