diff mbox series

[PATCH/next,v1] package/qt5declarative: move the qt5gui select from qt5declarative to qt5quick

Message ID 20211114135722.12386-1-ps.report@gmx.net
State Accepted
Headers show
Series [PATCH/next,v1] package/qt5declarative: move the qt5gui select from qt5declarative to qt5quick | expand

Commit Message

Peter Seiderer Nov. 14, 2021, 1:57 p.m. UTC
As requested/suggested by Sergey Kuzminov([1]) move the qt5gui select from
qt5declarative to qt5quick.

The qt5declarative (without qt5quick) compiles and installs fine without
qt5gui enabled and code inspection of the qt5declarative souce package *.pro
files shows the unconditionally gui dependencies only in for doc-snippets
and/or test projectes or real the qt5quick moduls.

[1] https://lore.kernel.org/buildroot/6fd05f97-7fb9-96d8-cca0-5063be8ad620@gmail.com

Reported-by: "Кузьминов Сергей" <kuzminov.sergey81@gmail.com>
Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
Notes:
  - @Sergey: I used the ASCII version of your name, hope it is o.k. for you?
---
 package/qt5/qt5declarative/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Petazzoni Nov. 14, 2021, 2:14 p.m. UTC | #1
On Sun, 14 Nov 2021 14:57:22 +0100
Peter Seiderer <ps.report@gmx.net> wrote:

> As requested/suggested by Sergey Kuzminov([1]) move the qt5gui select from
> qt5declarative to qt5quick.
> 
> The qt5declarative (without qt5quick) compiles and installs fine without
> qt5gui enabled and code inspection of the qt5declarative souce package *.pro
> files shows the unconditionally gui dependencies only in for doc-snippets
> and/or test projectes or real the qt5quick moduls.
> 
> [1] https://lore.kernel.org/buildroot/6fd05f97-7fb9-96d8-cca0-5063be8ad620@gmail.com
> 
> Reported-by: "Кузьминов Сергей" <kuzminov.sergey81@gmail.com>
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
> Notes:
>   - @Sergey: I used the ASCII version of your name, hope it is o.k. for you?
> ---
>  package/qt5/qt5declarative/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied to next, thanks.

Thomas
diff mbox series

Patch

diff --git a/package/qt5/qt5declarative/Config.in b/package/qt5/qt5declarative/Config.in
index 3ba9224c22..5ec83daa27 100644
--- a/package/qt5/qt5declarative/Config.in
+++ b/package/qt5/qt5declarative/Config.in
@@ -1,7 +1,6 @@ 
 config BR2_PACKAGE_QT5DECLARATIVE
 	bool "qt5declarative"
 	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
-	select BR2_PACKAGE_QT5BASE_GUI
 	help
 	  Qt is a cross-platform application and UI framework for
 	  developers using C++.
@@ -16,6 +15,7 @@  if BR2_PACKAGE_QT5DECLARATIVE
 
 config BR2_PACKAGE_QT5DECLARATIVE_QUICK
 	bool "quick module"
+	select BR2_PACKAGE_QT5BASE_GUI
 	select BR2_PACKAGE_QT5BASE_OPENGL if BR2_PACKAGE_QT5_GL_AVAILABLE
 
 endif