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 <[email protected]>

Reviewed-by: Casey Connolly <[email protected]>

> ---
> 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/[email protected]
> 
> Changes in v2:
> - added R-b trailers,
> - fixed spurious colon character before separator in cat,
> - Link to v1: 
> https://patch.msgid.link/[email protected]
> 
> ---
> 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 <[email protected]>
> 

-- 
// Casey (she/her)

Reply via email to