[Buildroot] [PATCH 2/2] Qwt package : added dependency to Qt5 besides Qt.

[Arnout Vandecappelle <arnout@mind.be>]

On 22-01-16 19:19, Yann E. MORIN wrote:
> David, All,
> 
> On 2016-01-22 18:44 +0100, David Picard spake thusly:
[snip]
>> @@ -21,10 +23,14 @@ config BR2_PACKAGE_QWT_OPENGL
>>  	bool "OpenGL support"
>>  	depends on BR2_PACKAGE_HAS_LIBGLES
>>  	depends on BR2_PACKAGE_HAS_LIBEGL
>> -	select BR2_PACKAGE_QT_OPENGL_ES
>> +	select BR2_PACKAGE_QT_OPENGL_ES if BR2_PACKAGE_QT
>> +	select BR2_PACKAGE_QT5BASE_OPENGL_LIB if BR2_PACKAGE_QT5BASE
> 
> BR2_PACKAGE_QT5BASE_OPENGL_LIB depends on BR2_PACKAGE_QT5BASE_OPENGL, so
> it should have been selected too.
> 
> However, I'm a bit worried about the complexity here, and I'd rather
> that we depend on such support rather than select it. So I'd prefer to
> see:
> 
>     depends on BR2_PACKAGE_QT_OPENGL_ES || BR2_PACKAGE_QT5BASE_OPENGL_LIB

 In that case, it can probably be removed completely and turned into an
automatic dependency (i.e. just in the .mk file).

 I also wouldn't bother with legacy handling when this symbol is removed. The
legacy .config file will already contain BR2_PACKAGE_QT_OPENGL_ES, so once the
handling is automatic, you just have to run 'make oldconfig' and it will Just
Work (TM).


 Regards,
 Arnout

[snip]


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF