From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from comal.ext.ti.com (comal.ext.ti.com [198.47.26.152]) by arago-project.org (Postfix) with ESMTPS id A6CFC5208B for ; Wed, 17 Feb 2016 22:18:19 +0000 (UTC) Received: from dflxv15.itg.ti.com ([128.247.5.124]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id u1HMII5R013519 for ; Wed, 17 Feb 2016 16:18:18 -0600 Received: from DLEE71.ent.ti.com (dlee71.ent.ti.com [157.170.170.114]) by dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id u1HMII8B023938 for ; Wed, 17 Feb 2016 16:18:18 -0600 Received: from dflp33.itg.ti.com (10.64.6.16) by DLEE71.ent.ti.com (157.170.170.114) with Microsoft SMTP Server id 14.3.224.2; Wed, 17 Feb 2016 16:18:18 -0600 Received: from [10.218.109.201] (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id u1HMIIk8008147; Wed, 17 Feb 2016 16:18:18 -0600 Message-ID: <56C4F1AA.5060609@ti.com> Date: Wed, 17 Feb 2016 17:18:18 -0500 From: Jacob Stiffler User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Denys Dmytriyenko References: <1455561794-2206-1-git-send-email-j-stiffler@ti.com> <56C21EA9.3090209@ti.com> <20160215190458.GG31480@edge> <56C237D5.9030907@ti.com> <20160215210210.GK31480@edge> In-Reply-To: <20160215210210.GK31480@edge> Cc: meta-arago@arago-project.org Subject: Re: [PATCH 1/2] arago.conf: Remove qtwayland from opencv PACKAGECONFIG for keystone X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 17 Feb 2016 22:18:19 -0000 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Should I mark opencv as machine-specific in arago.conf? On 2/15/2016 4:02 PM, Denys Dmytriyenko wrote: > On Mon, Feb 15, 2016 at 03:40:53PM -0500, Jacob Stiffler wrote: >> >> 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. > Well, not exactly. The original 3.0 recipe that we added to meta-arago was > backported from meta-oe/master as is. > > Then, 3.1 update came along and this one is completely custom, as no other > layer has 3.1 and the dependency on qtwayland is also something that we added. > > >> We use the qtwayland component for the GUI in the video analytics demo. > Ok, by the looks of it, enabling WITH_QT option in opencv allows it to work > with QT and open windows in opencv-enabled apps... > > >>>> 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 >>>>> --- >>>>> 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