Re: [yocto-patches] [psplash][PATCH 2/2] Add configure flags to set colors

["Gaël PORTAY" <gael.portay@gmail.com>]

Dear Maintainers,

I am just bumping this for visibility, I am sorry if this is not
appropriate.

Kind Regards,
Gaël

On Thu Nov 20, 2025 at 10:03 PM CET, Gaël PORTAY via lists.yoctoproject.org wrote:
> Added --with-text-color, --with-background-color, --with-bar-color, and
> --with-bar-background-color configure options to set colors without
> patching the code. Default colors are 0x6d,0x6d,0x70 for any text output
> and bar indicator, and 0xec,0xec,0xe1 for overall background.
>
> Signed-off-by: Gaël PORTAY <gael.portay+rtone@gmail.com>
> ---
>  configure.ac     | 32 ++++++++++++++++++++++++++++++++
>  psplash-colors.h |  8 ++++++++
>  2 files changed, 40 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index a64990a..d01afce 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -53,6 +53,38 @@ AS_IF([test "x$with_progress_bar_border_size" != "xyes"], [
>      EXTRA_GCC_FLAGS="$EXTRA_GCC_FLAGS -DPSPLASH_PROGRESS_BAR_BORDER_SIZE=$with_progress_bar_border_size"
>  ])
>  
> +AC_ARG_WITH([text-color],
> +    AS_HELP_STRING([--with-text-color=RED,GREEN,BLUE], [Set color of any text output (default is 0x6d,0x6d,0x70)]),
> +    [with_text_color=$withval],
> +    [with_text_color=0x6d,0x6d,0x70])
> +AS_IF([test "x$with_text_color" != "xyes"], [
> +    EXTRA_GCC_FLAGS="$EXTRA_GCC_FLAGS -DPSPLASH_TEXT_COLOR=$with_text_color"
> +])
> +
> +AC_ARG_WITH([background-color],
> +    AS_HELP_STRING([--with-background-color=RED,GREEN,BLUE], [Set color of overall background (default is 0xec,0xec,0xe1)]),
> +    [with_background_color=$withval],
> +    [with_background_color=0xec,0xec,0xe1])
> +AS_IF([test "x$with_background_color" != "xyes"], [
> +    EXTRA_GCC_FLAGS="$EXTRA_GCC_FLAGS -DPSPLASH_BACKGROUND_COLOR=$with_background_color"
> +])
> +
> +AC_ARG_WITH([bar-color],
> +    AS_HELP_STRING([--with-bar-color=RED,GREEN,BLUE], [Set color of the progress bar indicator (default is 0x6d,0x6d,0x70)]),
> +    [with_bar_color=$withval],
> +    [with_bar_color=0x6d,0x6d,0x70])
> +AS_IF([test "x$with_bar_color" != "xyes"], [
> +    EXTRA_GCC_FLAGS="$EXTRA_GCC_FLAGS -DPSPLASH_BAR_COLOR=$with_bar_color"
> +])
> +
> +AC_ARG_WITH([bar-background-color],
> +    AS_HELP_STRING([--with-bar-background-color=RED,GREEN,BLUE], [Set color of the progress bar background (default is 0xec,0xec,0xe1)]),
> +    [with_bar_background_color=$withval],
> +    [with_bar_background_color=0xec,0xec,0xe1])
> +AS_IF([test "x$with_bar_background_color" != "xyes"], [
> +    EXTRA_GCC_FLAGS="$EXTRA_GCC_FLAGS -DPSPLASH_BAR_BACKGROUND_COLOR=$with_bar_background_color"
> +])
> +
>  AC_ARG_ENABLE([img-fullscreen],
>      AS_HELP_STRING([--enable-img-fullscreen], [Enable the logo image in fullscreen mode)]),
>      [enable_img_fullscreen=$enableval],
> diff --git a/psplash-colors.h b/psplash-colors.h
> index 82a9893..d6d19af 100644
> --- a/psplash-colors.h
> +++ b/psplash-colors.h
> @@ -12,15 +12,23 @@
>  #define _HAVE_PSPLASH_COLORS_H
>  
>  /* This is the overall background color */
> +#ifndef PSPLASH_BACKGROUND_COLOR
>  #define PSPLASH_BACKGROUND_COLOR 0xec,0xec,0xe1
> +#endif
>  
>  /* This is the color of any text output */
> +#ifndef PSPLASH_TEXT_COLOR
>  #define PSPLASH_TEXT_COLOR 0x6d,0x6d,0x70
> +#endif
>  
>  /* This is the color of the progress bar indicator */
> +#ifdef PSPLASH_BAR_COLOR
>  #define PSPLASH_BAR_COLOR 0x6d,0x6d,0x70
> +#endif
>  
>  /* This is the color of the progress bar background */
> +#ifdef PSPLASH_BAR_BACKGROUND_COLOR
>  #define PSPLASH_BAR_BACKGROUND_COLOR 0xec,0xec,0xe1
> +#endif
>  
>  #endif