[Buildroot] [PATCH 2/5] package/qt5webkit: remove obsolete package

[Julien CORJON]

Gary, All,

Le 21/03/2016 16:56, Gary Bisson a écrit :
> Julien, All,
>
> On Thu, Mar 17, 2016 at 10:42 AM, Julien Corjon <corjon.j at ecagroup.com> wrote:
>>
>> Signed-off-by: Julien Corjon <corjon.j at ecagroup.com>
>> ---
>>   package/qt5/Config.in                |  1 -
>>   package/qt5/qt5webkit/Config.in      | 23 --------------
>>   package/qt5/qt5webkit/qt5webkit.hash |  2 --
>>   package/qt5/qt5webkit/qt5webkit.mk   | 58 ------------------------------------
>>   4 files changed, 84 deletions(-)
>>   delete mode 100644 package/qt5/qt5webkit/Config.in
>>   delete mode 100644 package/qt5/qt5webkit/qt5webkit.hash
>>   delete mode 100644 package/qt5/qt5webkit/qt5webkit.mk
>
> Are we sure we want to remove this package actually? The release says:
> "With Qt 5.6 the following modules are no longer part of the release
> packages, but users can still build them from source:
> - Qt WebKit
> - Qt Declarative (Qt Quick 1)"
>
> But the project has a qt5.6.0 branch just like the other modules:
> https://github.com/qtproject/qtwebkit/commits/5.6.0

You're almost right, the project have a 5.6 branch but 5.6.0 version has 
not been tagged on deprecated modules. If we want to keep those in 
buildroot we have now to "randomly" choose a commit.

>
> Especially since we don't have any replacement I wouldn't merge this
> patch just yet. Plus webkit can run on a platform without GPU whereas
> WebEngine depends on OpenGL, I wouldn't mind keeping it, for i.MX7
> platforms for instance.

I agree that platform without OpenGL support lose web browser ability 
from Qt and for that we should probably keep webkit in buildroot 
packages. But I'm not sure we should keep it in official Qt section 
since there not officially supported by Digia and Qt community.

>
> The package would need to be modified to use the github project
> instead of the release archives though.

If we use Qt repository we should use the official one 
(http://code.qt.io/cgit/qt/qtwebkit.git/) instead of github.

>
> Let me know your thoughts.
>
> Regards,
> Gary
>

Regards,

Julien