From: Michael Olbrich <mol@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Marco Felsch <m.felsch@pengutronix.de>
Subject: Re: [ptxdist] [APPLIED] qt5: drop qtdeclarative opengl dependency
Date: Sat, 20 Jun 2020 00:04:11 +0200 [thread overview]
Message-ID: <E1jmP7L-0002Td-Of@dude02.lab.pengutronix.de> (raw)
In-Reply-To: <20200312113516.21154-2-m.felsch@pengutronix.de>
Thanks, applied as cb83037d89db53190f58358aae36fd050b384455.
Michael
[sent from post-receive hook]
On Sat, 20 Jun 2020 00:04:11 +0200, Marco Felsch <m.felsch@pengutronix.de> wrote:
> Since 5.14 the qtdeclarative can be build with -no-opengl option so we
> can remove the dependency. Now all 3D modules need to add the opengl
> dependency by there own. We also must ensure that all 3D modules are
> skipped if -no-opengl is specified.
>
> The particles module depends on the opengl support and the module isn't
> build if the support is missing. So we need a way to handle this too.
>
> Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
> Message-Id: <20200312113516.21154-2-m.felsch@pengutronix.de>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
>
> diff --git a/rules/qt5.in b/rules/qt5.in
> index 5590b623792d..4ba7e6837c21 100644
> --- a/rules/qt5.in
> +++ b/rules/qt5.in
> @@ -321,6 +321,7 @@ menu "Modules "
>
> config QT5_MODULE_QT3D
> bool
> + select QT5_OPENGL
> select QT5_MODULE_QTDECLARATIVE
> select QT5_MODULE_QTGRAPHICALEFFECTS
> prompt "Qt3D"
> @@ -408,7 +409,6 @@ config QT5_MODULE_QTDATAVIS3D_QUICK
>
> config QT5_MODULE_QTDECLARATIVE
> bool
> - select QT5_OPENGL
> prompt "QtDeclarative"
>
> if QT5_MODULE_QTDECLARATIVE
> @@ -435,6 +435,10 @@ config QT5_MODULE_QTDECLARATIVE_QUICK_DEBUG
> bool
> default QT5_MODULE_QTDECLARATIVE_DEBUG
>
> +config QT5_MODULE_QTDECLARATIVE_QUICK_PARTICLES
> + bool
> + default QT5_OPENGL
> +
> endif
>
> config QT5_MODULE_QTDECLARATIVE_ACCESSIBILITY
> diff --git a/rules/qt5.make b/rules/qt5.make
> index 4f7ada8ad87d..aa9676af77ea 100644
> --- a/rules/qt5.make
> +++ b/rules/qt5.make
> @@ -453,7 +453,7 @@ QT5_LIBS-$(PTXCONF_QT5_MODULE_QTDECLARATIVE_QUICK) += Qt5QmlWorkerScript
> QT5_LIBS-$(PTXCONF_QT5_MODULE_QTDECLARATIVE_QUICK) += Qt5Quick
> QT5_LIBS-$(PTXCONF_QT5_MODULE_QTDECLARATIVE_QUICK) += Qt5QuickShapes
> QT5_LIBS-$(PTXCONF_QT5_MODULE_QTDECLARATIVE_QUICK_WIDGETS) += Qt5QuickWidgets
> -QT5_LIBS-$(PTXCONF_QT5_MODULE_QTDECLARATIVE_QUICK) += Qt5QuickParticles
> +QT5_LIBS-$(PTXCONF_QT5_MODULE_QTDECLARATIVE_QUICK_PARTICLES) += Qt5QuickParticles
> ifdef PTXCONF_QT5_TEST
> QT5_LIBS-$(PTXCONF_QT5_MODULE_QTDECLARATIVE_QUICK) += Qt5QuickTest
> endif
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
next prev parent reply other threads:[~2020-06-19 22:04 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-12 11:35 [ptxdist] [RFC PATCH 1/3] qt5: version bump 5.12.6 -> 5.14.1 Marco Felsch
2020-03-12 11:35 ` [ptxdist] [RFC PATCH 2/3] qt5: drop qtdeclarative opengl dependency Marco Felsch
2020-06-19 22:04 ` Michael Olbrich [this message]
2020-03-12 11:35 ` [ptxdist] [RFC PATCH 3/3] qt5: add capability to build from git super project Marco Felsch
2020-04-10 10:48 ` [ptxdist] [RFC PATCH 1/3] qt5: version bump 5.12.6 -> 5.14.1 Marco Felsch
2020-06-19 22:04 ` [ptxdist] [APPLIED] " Michael Olbrich
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=E1jmP7L-0002Td-Of@dude02.lab.pengutronix.de \
--to=mol@pengutronix.de \
--cc=m.felsch@pengutronix.de \
--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