[meta-freescale] [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

Otavio Salvador otavio at ossystems.com.br
Thu Feb 19 03:08:55 PST 2015


On Thu, Feb 19, 2015 at 9:03 AM, Andreas Müller
<schnitzeltony at googlemail.com> wrote:
> On Thu, Feb 19, 2015 at 11:41 AM, Alexandre Belloni
> <alexandre.belloni at free-electrons.com> wrote:
>> I just want to mention that removing icu will break any application
>> using qtwebkit (but not qtwebkit itself) when building with a distro
>> that does not add icu to the qtbase PACKAGECONFIG (like poky).
>>
>> I'm not saying this shouldn't be done, I'm just warning that we may see
>> some reports of broken applications soon ;)
>>
> I think we share same opinion here:
>
> * This should not be the job of a BSP
> * qtdeclarative fails to configure without icu and qtwebkit depends on
> it. qtdeclarative requiring icu is mentioned in qtbase.inc

Agreed. So I think I will amend the commit log as follows:

qtbase: Remove distribution-related settings

The use of TSLib, ICU and availability of examples is a distribution
configuration and not a BSP thing. Drop this settings from the
bbappend.

WARNING
-------

   This breaks QtDeclarative and QtWebKit based application,
   as OE-Core and Poky distros does not add 'icu' to the
   DISTRO_FEATURES list.

Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>


-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


More information about the meta-freescale mailing list