public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: "Petr Kubizňák" <kubiznak@2n.com>,
	openembedded-core@lists.openembedded.org
Subject: Re: [OE-core][PATCH v4 06/10] webkitgtk: add missing dependencies
Date: Fri, 31 Mar 2023 13:06:45 +0100	[thread overview]
Message-ID: <e1408db180e41666a38260a9f45f6edf7f725192.camel@linuxfoundation.org> (raw)
In-Reply-To: <20230331114634.3578244-6-kubiznak@2n.com>

On Fri, 2023-03-31 at 13:46 +0200, Petr Kubizňák wrote:
> When gobject-introspection feature is disabled, gettext and
> glib-2.0-native dependencies are not pulled in, which causes failures
> in do_compile due to missing xgettext and gdbus-codegen.
> 
> Signed-off-by: Petr Kubizňák <kubiznak@2n.com>
> ---
>  meta/recipes-sato/webkit/webkitgtk_2.38.5.bb | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-sato/webkit/webkitgtk_2.38.5.bb b/meta/recipes-sato/webkit/webkitgtk_2.38.5.bb
> index e52920883a..b49ecf2a31 100644
> --- a/meta/recipes-sato/webkit/webkitgtk_2.38.5.bb
> +++ b/meta/recipes-sato/webkit/webkitgtk_2.38.5.bb
> @@ -17,7 +17,7 @@ SRC_URI = "https://www.webkitgtk.org/releases/${BPN}-${PV}.tar.xz \
>             "
>  SRC_URI[sha256sum] = "40c20c43022274df5893f22b1054fa894c3eea057389bb08aee08c5b0bb0c1a7"
>  
> -inherit cmake pkgconfig gobject-introspection perlnative features_check upstream-version-is-even gi-docgen
> +inherit cmake pkgconfig gobject-introspection perlnative features_check upstream-version-is-even gi-docgen gettext
>  
>  ANY_OF_DISTRO_FEATURES = "${GTK3DISTROFEATURES}"
>  REQUIRED_DISTRO_FEATURES = "${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'opengl', '', d)}"
> @@ -38,6 +38,7 @@ DEPENDS += " \
>            libnotify \
>            gstreamer1.0 \
>            gstreamer1.0-plugins-base \
> +          glib-2.0-native \
>            "
>  
>  PACKAGECONFIG_SOUP ?= "soup3"

If the dependency is just xgettext then it may just be a DEPENDS +=
"gettext-native"  which is a lower requirement than the full gettext
target code?

Cheers,

Richard



  reply	other threads:[~2023-03-31 12:06 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-31 11:46 [OE-core][PATCH v4 01/10] gobject-introspection: reduce dependencies Petr Kubizňák
2023-03-31 11:46 ` [OE-core][PATCH v4 02/10] graphene: add gobject-types PACKAGECONFIG Petr Kubizňák
2023-03-31 11:46 ` [OE-core][PATCH v4 03/10] python3-pygobject: depend on gobject-introspection Petr Kubizňák
2023-03-31 11:46 ` [OE-core][PATCH v4 04/10] gconf: add missing dependencies Petr Kubizňák
2023-03-31 11:46 ` [OE-core][PATCH v4 05/10] avahi: " Petr Kubizňák
2023-03-31 11:46 ` [OE-core][PATCH v4 06/10] webkitgtk: " Petr Kubizňák
2023-03-31 12:06   ` Richard Purdie [this message]
2023-04-03 10:31     ` Petr Kubizňák - 2N
2023-04-03 10:43       ` Richard Purdie
2023-03-31 11:46 ` [OE-core][PATCH v4 07/10] harfbuzz: depend on glib-2.0-native Petr Kubizňák
2023-03-31 11:46 ` [OE-core][PATCH v4 08/10] json-glib: " Petr Kubizňák
2023-03-31 11:46 ` [OE-core][PATCH v4 09/10] libgudev: " Petr Kubizňák
2023-03-31 11:46 ` [OE-core][PATCH v4 10/10] at-spi2-core: " Petr Kubizňák
2023-04-03 20:14 ` [OE-core][PATCH v4 01/10] gobject-introspection: reduce dependencies Richard Purdie
2023-04-04  7:07   ` Petr Kubizňák - 2N
2023-04-05  7:05     ` Petr Kubizňák - 2N
2023-04-05  8:19       ` Richard Purdie

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=e1408db180e41666a38260a9f45f6edf7f725192.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=kubiznak@2n.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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