Re: [meta-qt5][PATCH v5 2/2] packagegroup-qt5-toolchain-target: include all modules for development

[Jonathan Liu <net147@gmail.com>]

On 19/04/2014 1:29 PM, Otavio Salvador wrote:
> On Sat, Apr 19, 2014 at 12:17 AM, Jonathan Liu <net147@gmail.com> wrote:
>> On 19/04/2014 12:27 PM, Otavio Salvador wrote:
>>> Hello Jonathan,
>>>
>>> On Sun, Mar 30, 2014 at 6:52 PM, Jonathan Liu <net147@gmail.com> wrote:
>>>> On 13/03/2014 1:01 PM, Otavio Salvador wrote:
>>>>> On Wed, Mar 12, 2014 at 7:52 PM, Jonathan Liu <net147@gmail.com> wrote:
>>>>>> This adds the necessary target packages for development with all of the
>>>>>> Qt 5 modules.
>>>>>>
>>>>>> Signed-off-by: Jonathan Liu <net147@gmail.com>
>>>>> | Computing transaction...error: Can't install
>>>>> qtwayland-dev-5.2.1+git0+573d0ee5ba-r0.0@cortexa9hf_vfp_neon: no
>>>>> package provides qtwayland = 5.2.1+git0+573d0ee5ba-r0.0
>>>>> |
>>>>> | Saving cache...
>>>>> |
>>>>> | WARNING:
>>>>>
>>>>> /home/otavio/hacking/customer/companytec/yocto/build/tmp/work/wandboard_solo-oel-linux-gnueabi/qsiv-demo-image/1.0-r0/temp/run.populate_sdk_image.29400:1
>>>>> exit 1 from
>>>>> |   smart --data-dir=${target_rootfs}/var/lib/smart install -y
>>>>> ${pkgs_to_install}
>>>>> | DEBUG: Python function do_populate_sdk finished
>>>>> | ERROR: Function failed: populate_sdk_image (log file is located at
>>>>>
>>>>>
>>>>> /home/otavio/hacking/customer/companytec/yocto/build/tmp/work/wandboard_solo-oel-linux-gnueabi/qsiv-demo-image/1.0-r0/temp/lo
>>>>>
>>>>> Please fix this and test the combinations of enable/disable features.
>>>>> They seem not well tested.
>>>>>
>>>> It looks like qtwayland is not being built properly in your configration.
>>>>
>>>> Can you provide configuration and steps to reproduce this failure?
>>>> I have not had any luck in reproducing it.
>>> The wayland packages cannot be installed if wayland distro feature is
>>> not available.
>>>
>> By default, DISTRO_FEATURES does not contain wayland so I wonder if it makes
>> sense for to add wayland to DISTRO_FEATURES if it is not available.
>> Is there a better method than checking if DISTRO_FEATURES has wayland for
>> qtwayland?
> You can check for x11 or wayland in this case. But I think qtwayland
> is useless without wayland in distro features, isn't it?
>
I already do check for wayland in DISTRO_FEATURES before including 
qtwayland packages:
${@base_contains('DISTRO_FEATURES', 'wayland', '${USE_WAYLAND}', '', d)}

Is this not enough?

I could perhaps change USE_RUBY, USE_WAYLAND, and USE_X11 to assign with 
?= instead of = so that the packages depending on those can be excluded 
from the SDK if desired by setting them to "" in local.conf.

Regards,
Jonathan