All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [meta-qt5][PATCH] qt5: package Qt Quick Designer files separately
Date: Wed, 9 Sep 2015 20:02:37 +0200	[thread overview]
Message-ID: <20150909180237.GC2390@jama> (raw)
In-Reply-To: <1441186359-14111-1-git-send-email-samuli.piippo@theqtcompany.com>

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

On Wed, Sep 02, 2015 at 12:32:39PM +0300, Samuli Piippo wrote:
> Some modules (at least Qt Quick Controls and Qt Canvas 3d) have extra
> files for Qt Quick Designer integration, which are not needed in the image.
> Create new ${PN}-qmldesigner package that includes those files instead of
> ${PN}-qmlplugins.

This breaks qtwebkit:

ERROR: Nothing RPROVIDES 'qtwebkit-qmldesigner' (but
/home/jenkins/oe/world/shr-core/meta-qt5/recipes-qt/qt5/qtwebkit_git.bb
RDEPENDS on or otherwise requires it)

NOTE: Runtime target 'qtwebkit-qmldesigner' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['qtwebkit-qmldesigner']

NOTE: Runtime target 'qtquick1-dev' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['qtquick1-dev',
'qtwebkit', 'qtwebkit-qmldesigner']
...

it's caused by qtwebkit redefinition of PACKAGES variable:
# remove default ${PN}-examples-dbg ${PN}-examples set in qt5.inc,
# because it conflicts with ${PN} from separate webkit-examples recipe
PACKAGES = "${PN}-dbg ${PN}-staticdev ${PN}-dev ${PN}-doc ${PN}-locale
${PACKAGE_BEFORE_PN} ${PN} ${PN}-qmlplugins-dbg ${PN}-tools-dbg
${PN}-plugins-dbg ${PN}-qmlplugins ${PN}-tools ${PN}-plugins
${PN}-mkspecs "


Can you change qtwebkit_git.bb to use PACKAGES_remove and resend tested
patch?

Regards,

> ---
>  recipes-qt/qt5/qt5.inc | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/recipes-qt/qt5/qt5.inc b/recipes-qt/qt5/qt5.inc
> index 4c58ad6..17d8fd7 100644
> --- a/recipes-qt/qt5/qt5.inc
> +++ b/recipes-qt/qt5/qt5.inc
> @@ -45,7 +45,7 @@ INSANE_SKIP_${PN}-examples += "libdir"
>  INSANE_SKIP_${PN}-examples-dbg += "libdir"
>  INSANE_SKIP_${PN}-examples-dev += "libdir"
>  
> -PACKAGES =. "${PN}-qmlplugins-dbg ${PN}-tools-dbg ${PN}-plugins-dbg ${PN}-qmlplugins ${PN}-tools ${PN}-plugins ${PN}-mkspecs ${PN}-examples-dev ${PN}-examples-staticdev ${PN}-examples-dbg ${PN}-examples "
> +PACKAGES =. "${PN}-qmlplugins-dbg ${PN}-tools-dbg ${PN}-plugins-dbg ${PN}-qmldesigner ${PN}-qmlplugins ${PN}-tools ${PN}-plugins ${PN}-mkspecs ${PN}-examples-dev ${PN}-examples-staticdev ${PN}-examples-dbg ${PN}-examples "
>  
>  ALLOW_EMPTY_${PN} = "1"
>  ALLOW_EMPTY_${PN}-dbg = "1"
> @@ -58,6 +58,7 @@ RRECOMMENDS_${PN}_class-target = " \
>  RRECOMMENDS_${PN}-dev = " \
>      ${PN} \
>      ${PN}-mkspecs \
> +    ${PN}-qmldesigner \
>      "
>  
>  RRECOMMENDS_${PN}-dbg = " \
> @@ -85,6 +86,11 @@ FILES_${PN}-plugins-dbg = " \
>  "
>  
>  # extra packages
> +FILES_${PN}-qmldesigner += " \
> +    ${OE_QMAKE_PATH_QML}/*/designer \
> +    ${OE_QMAKE_PATH_QML}/*/*/designer \
> +"
> +
>  # qmlplugins 1-4 levels of subdirs qmldir + *{.so,*.qmltypes,*.qml,*js}
>  FILES_${PN}-qmlplugins = " \
>      ${OE_QMAKE_PATH_QML}/*/*${SOLIBSDEV} \
> -- 
> 1.9.1
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

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

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

      reply	other threads:[~2015-09-09 18:01 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-02  9:32 [meta-qt5][PATCH] qt5: package Qt Quick Designer files separately Samuli Piippo
2015-09-09 18:02 ` 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=20150909180237.GC2390@jama \
    --to=martin.jansa@gmail.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.