All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: Adrian Ratiu <adrian.ratiu@collabora.com>
Cc: openembeded-devel <openembedded-devel@lists.openembedded.org>
Subject: Re: [meta-oe] [PATCH] renderdoc: add x11 to REQUIRED_DISTRO_FEATURES
Date: Fri, 4 Oct 2019 12:12:48 +0200	[thread overview]
Message-ID: <20191004101248.GA1983@jama> (raw)
In-Reply-To: <20191004100835.1700461-1-adrian.ratiu@collabora.com>

[-- Attachment #1: Type: text/plain, Size: 1418 bytes --]

On Fri, Oct 04, 2019 at 01:08:35PM +0300, Adrian Ratiu wrote:
> This recipe depends on having x11 enabled so we add it to fix:
> 
> ERROR: Nothing PROVIDES 'libxcb' (but
> meta-oe/meta-oe/recipes-graphics/renderdoc/renderdoc_1.4.bb
> DEPENDS on or otherwise requires it)
> libxcb was skipped: missing required distro feature 'x11' (not in DISTRO_FEATURES)

Thanks for quick update!

Sorry I haven't mentioned it in the (late) review comment, but this will
also need distro_features_check in the inherit for
REQUIRED_DISTRO_FEATURES variable to work.

Cheers,

> Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
> ---
>  meta-oe/recipes-graphics/renderdoc/renderdoc_1.4.bb | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/meta-oe/recipes-graphics/renderdoc/renderdoc_1.4.bb b/meta-oe/recipes-graphics/renderdoc/renderdoc_1.4.bb
> index b6e0c92f6..dd5e4058c 100644
> --- a/meta-oe/recipes-graphics/renderdoc/renderdoc_1.4.bb
> +++ b/meta-oe/recipes-graphics/renderdoc/renderdoc_1.4.bb
> @@ -10,6 +10,8 @@ SRC_URI = "git://github.com/baldurk/${BPN}.git;protocol=http;branch=v1.x \
>  "
>  S = "${WORKDIR}/git"
>  
> +REQUIRED_DISTRO_FEATURES = "x11"
> +
>  DEPENDS += "virtual/libx11 virtual/libgl libxcb xcb-util-keysyms vim-native"
>  
>  RDEPENDS_${PN} = "libxcb xcb-util-keysyms"
> -- 
> 2.23.0
> 

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 201 bytes --]

      reply	other threads:[~2019-10-04 10:12 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-04 10:08 [meta-oe] [PATCH] renderdoc: add x11 to REQUIRED_DISTRO_FEATURES Adrian Ratiu
2019-10-04 10:12 ` Martin Jansa [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=20191004101248.GA1983@jama \
    --to=martin.jansa@gmail.com \
    --cc=adrian.ratiu@collabora.com \
    --cc=openembedded-devel@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.