[Buildroot] [PATCH 1/1] qtquick1: removed OpenGL dependencies

[Wojciech Sleńska]

2014-05-06 21:21 GMT+02:00 Thomas Petazzoni
<thomas.petazzoni at free-electrons.com>:
> Dear Wojciech Slenska,
>
> On Mon,  5 May 2014 19:06:46 +0200, Wojciech Slenska wrote:
>> QtQuick1 module doesn't need OpenGl libs and declarative module.
>> This module can be used to port qt4 qml apps to qt5. Patch has been tested with linuxfb output plugin.
>>
>> Signed-off-by: Wojciech Slenska <wojciech.slenska at gmail.com>
>
> Thanks for this patch. Can qt5quick1 optionally depend on
> qt5declarative? If it's the case, then the .mk file should have
> something like:
>
> ifeq ($(select BR2_PACKAGE_QT5DECLARATIVE),y)
> QT5QUICK1_DEPENDENCIES += qt5declarative
> endif
>

Hello Thomas,

Thanks for your comment.

I'm using qtquick1 library on following HW configurations: TI AM3517
with EGL support and Freescale imx28 with linuxfb plugin. I neither of
these configurations QtQuick2 (qt5declarative) is needed.
Only different thing is that when we build qt with EGL support,
qtquick1 also need EGL libs to work.

In qt5 we can use two versions of QML:
qtquick1 (libQt5Declarative.so) -> Can be used to port old apps to qt5
or for processors which doesn't have OpenGl. For those version opengl
isn't needed, but can be used. I'm using it on ARM9 without opengl
(imx28)
qtquick2 (libQt5Quick.so) -> New version which needs OpenGL to work.
I'm using it on TI AM proccessor.
I never saw any dependency between those two libs.

Best Regards,
Wojciech Slenska

> Fatih, could you review this patch?
>
> Thanks!
>
> Thomas
>
>> ---
>>  package/qt5/qt5quick1/Config.in    |    8 ++------
>>  package/qt5/qt5quick1/qt5quick1.mk |    2 +-
>>  2 files changed, 3 insertions(+), 7 deletions(-)
>>
>> diff --git a/package/qt5/qt5quick1/Config.in b/package/qt5/qt5quick1/Config.in
>> index 30c2e79..59b6c34 100644
>> --- a/package/qt5/qt5quick1/Config.in
>> +++ b/package/qt5/qt5quick1/Config.in
>> @@ -5,11 +5,8 @@ config BR2_PACKAGE_QT5QUICK1
>>       select BR2_PACKAGE_QT5BASE_WIDGETS
>>       select BR2_PACKAGE_QT5SCRIPT
>>       select BR2_PACKAGE_QT5XMLPATTERNS
>> -     select BR2_PACKAGE_QT5DECLARATIVE
>>       # This module does not support static linking
>>       depends on !BR2_PREFER_STATIC_LIB
>> -     depends on BR2_PACKAGE_HAS_LIBEGL
>> -     depends on BR2_PACKAGE_HAS_LIBGLES
>>       depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
>>       help
>>         Qt is a cross-platform application and UI framework for
>> @@ -19,7 +16,6 @@ config BR2_PACKAGE_QT5QUICK1
>>
>>         http://qt-project.org
>>
>> -comment "qt5quick1 needs a toolchain w/ dynamic library and an OpenGL-capable backend"
>> +comment "qt5quick1 needs a toolchain w/ dynamic library"
>>       depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
>> -     depends on !BR2_PACKAGE_HAS_LIBEGL || !BR2_PACKAGE_HAS_LIBGLES || \
>> -             BR2_PREFER_STATIC_LIB
>> +     depends on BR2_PREFER_STATIC_LIB
>> diff --git a/package/qt5/qt5quick1/qt5quick1.mk b/package/qt5/qt5quick1/qt5quick1.mk
>> index 624b130..e9d7a64 100644
>> --- a/package/qt5/qt5quick1/qt5quick1.mk
>> +++ b/package/qt5/qt5quick1/qt5quick1.mk
>> @@ -7,7 +7,7 @@
>>  QT5QUICK1_VERSION = $(QT5_VERSION)
>>  QT5QUICK1_SITE = $(QT5_SITE)
>>  QT5QUICK1_SOURCE = qtquick1-opensource-src-$(QT5QUICK1_VERSION).tar.xz
>> -QT5QUICK1_DEPENDENCIES = qt5base qt5xmlpatterns qt5script qt5declarative \
>> +QT5QUICK1_DEPENDENCIES = qt5base qt5xmlpatterns qt5script \
>>       $(if $(BR2_PACKAGE_QT5WEBKIT),qt5webkit)
>>  QT5QUICK1_INSTALL_STAGING = YES
>>
>
>
>
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com