Re: [meta-qt5][PATCH] qtbase.inc: Enable accessibility by default

[Jonathan Liu <net147@gmail.com>]

On 15/05/2014 5:04 AM, Otavio Salvador wrote:
> Hello folks,
>
> On Mon, May 5, 2014 at 9:21 AM, Otavio Salvador <otavio@ossystems.com.br> wrote:
>> On Sun, May 4, 2014 at 6:47 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
>>> On Mon, Apr 28, 2014 at 12:27:39PM -0300, Otavio Salvador wrote:
>>>> qtdeclarative requires accessibility to be enabled and it is added by
>>>> default to the toolchain so we ought to have it enabled to ensure the
>>>> default toolchain generation works.
>>>>
>>>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> ...
>>> As I told you on gtalk, I would prefer default to stay as minimal as
>>> posible, why don't you change packagegroup-qt5-toolchain-target.bb to
>>> use RRECOMMENDS instead of RDEPENDS so that missing
>>> qtquickcontrols-qmlplugins package doesn't break it when it's not
>>> available (because it's empty)?
>> I don't have a strong opinion for either case however I think we ought
>> to know what other meta-qt5 users think about it.
>>
>> In support to this patch addition I think we ought to provide the most
>> used features of Qt5 working out of box to users have a good first
>> use. Special cases can customize it per need basis. I think QML is
>> common enough for us to provide full support for it by default.
>
> Martin and I have different views on this topic and I'd like to merge
> or drop this patch. Could people comment on this one?
>
If changing to RRECOMMENDS resolves the issue I vote for that option.
It's better to start minimal and add only what you need when creating an 
image.

Regards,
Jonathan