From: Michael Tretter <m.tretter@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Benjamin Gaignard <benjamin.gaignard@collabora.com>
Subject: Re: [ptxdist] [PATCH] weston: build color management module
Date: Tue, 3 Sep 2024 14:54:41 +0200 [thread overview]
Message-ID: <ZtcHEUIRLf-os7St@pengutronix.de> (raw)
In-Reply-To: <010201919dedf670-fb51031f-4588-459a-965b-b272683a339f-000000@eu-west-1.amazonses.com>
Hi Benjamin,
On Thu, 29 Aug 2024 11:38:23 +0000, Benjamin Gaignard wrote:
> If LCMS package dependency is selected build weston color management
> module.
>
> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@collabora.com>
> ---
> rules/weston.in | 1 +
> rules/weston.make | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/rules/weston.in b/rules/weston.in
> index 10f0e9172..22752aef0 100644
> --- a/rules/weston.in
> +++ b/rules/weston.in
> @@ -40,6 +40,7 @@ menuconfig WESTON
> select PIPEWIRE if WESTON_BACKEND_PIPEWIRE
> select NEATVNC if WESTON_BACKEND_VNC
> select PAM if WESTON_BACKEND_VNC
> + select LCMS
> prompt "weston "
> help
> Wayland compositor reference implementation
> diff --git a/rules/weston.make b/rules/weston.make
> index 35ddab180..908f76f0b 100644
> --- a/rules/weston.make
> +++ b/rules/weston.make
> @@ -45,7 +45,7 @@ WESTON_CONF_OPT := \
> -Dbackend-vnc=$(call ptx/truefalse,PTXCONF_WESTON_BACKEND_VNC) \
> -Dbackend-wayland=$(call ptx/truefalse,PTXCONF_WESTON_GL) \
> -Dbackend-x11=false \
> - -Dcolor-management-lcms=false \
> + -Dcolor-management-lcms=$(call ptx/truefalse, PTXCONF_LCMS) \
With this change, it's not possible to build Weston without
color-management-lcms, because PTXCONF_LCMS is always selected and this
always will be set to true.
Please add a separate option PTXCONF_WESTON_COLOR_MANAGEMENT_LCMS (or
similar) that enables this option, and select LCMS only if
PTXCONF_WESTON_COLOR_MANAGEMENT_LCMS is set.
Michael
> -Ddemo-clients=$(call ptx/truefalse,PTXCONF_WESTON_IVISHELL_EXAMPLE) \
> -Ddeprecated-color-management-colord=false \
> -Ddeprecated-color-management-static=false \
> --
> 2.43.0
>
>
>
prev parent reply other threads:[~2024-09-03 12:55 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-29 11:38 Benjamin Gaignard
2024-09-03 12:54 ` Michael Tretter [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZtcHEUIRLf-os7St@pengutronix.de \
--to=m.tretter@pengutronix.de \
--cc=benjamin.gaignard@collabora.com \
--cc=ptxdist@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox