Re: [PATCH 1/2] arago.conf: Remove qtwayland from opencv PACKAGECONFIG for keystone

From: Jacob Stiffler <j-stiffler@ti.com>
To: Denys Dmytriyenko <denys@ti.com>
Cc: meta-arago@arago-project.org
Subject: Re: [PATCH 1/2] arago.conf: Remove qtwayland from opencv PACKAGECONFIG for keystone
Date: Mon, 15 Feb 2016 15:40:53 -0500	[thread overview]
Message-ID: <56C237D5.9030907@ti.com> (raw)
In-Reply-To: <20160215190458.GG31480@edge>

[-- Attachment #1: Type: text/plain, Size: 2527 bytes --]

On 2/15/2016 2:04 PM, Denys Dmytriyenko wrote:
> On Mon, Feb 15, 2016 at 01:53:29PM -0500, Jacob Stiffler wrote:
>> This may be more of an RFC as I have some concerns.
>>
>> I am concerned with this patch as it adds machine-specific
>> configuration for opencv, but the recipe is not marked as
>> machine-specific.
>>
>> Will this pose an issue?
> Yeah, I just replied on this part.
>
>
>> Now I understand that if opencv packaging is done well enough, that
>> we should still be able to build the qtwayland parts of opencv, but
>> do not install them in the keystone filesystem so that wayland is
>> not brought in. However, wayland has a dependency on the ddk, which
>> is not compatible with keystone machines. Thus the opencv build will
>> be broken for keystone machines.
>>
>> Any suggestions?
> Quick question - where did 3.1 recipe come from and what exactly qtwayland
> dependency do?

The 3.1 recipe was created by Hongmei and based on 
http://cgit.openembedded.org/cgit.cgi/meta-openembedded/tree/meta-oe/recipes-support/opencv/opencv_2.4.bb?h=master.

We use the qtwayland component for the GUI in the video analytics demo.

>
>> On 2/15/2016 1:43 PM, Jacob Stiffler wrote:
>>> * keystone machines do not have a display, hence they do not support
>>>    wayland
>>> * opencv adds qtwayland support since wayland is in DISTRO_FEATURES
>>> * Remove qtwayland from opencv PACKAGECONFIG for keystone machines
>>>
>>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>>> ---
>>>   meta-arago-distro/conf/distro/arago.conf | 3 +++
>>>   1 file changed, 3 insertions(+)
>>>
>>> diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
>>> index 325e112..38c7887 100644
>>> --- a/meta-arago-distro/conf/distro/arago.conf
>>> +++ b/meta-arago-distro/conf/distro/arago.conf
>>> @@ -58,6 +58,9 @@ PACKAGECONFIG_append_pn-qtmultimedia  = " gstreamer"
>>>   # Prefer OpenSSL over GnuTLS for curl due to TSU exemptions
>>>   PACKAGECONFIG_pn-curl = "${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6", "", d)} ssl zlib"
>>> +# Remove qtwayland from openvcv PACKAGECONFIG for keystone
>>> +PACKAGECONFIG_remove_pn-opencv_keystone = "qtwayland"
>>> +
>>>   # Enable UI and scripting for perf
>>>   PERF_FEATURES_ENABLE ?= "perf-scripting perf-tui"
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

[-- Attachment #2: Type: text/html, Size: 24855 bytes --]