From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from vms173007pub.verizon.net (vms173007pub.verizon.net [206.46.173.7]) by arago-project.org (Postfix) with ESMTP id 6E1C152A93 for ; Thu, 6 Dec 2012 23:52:43 +0000 (UTC) Received: from gandalf.denix.org ([unknown] [71.191.205.12]) by vms173007.mailsrvcs.net (Sun Java(tm) System Messaging Server 7u2-7.02 32bit (built Apr 16 2009)) with ESMTPA id <0MEM00J8FVN2REI3@vms173007.mailsrvcs.net> for meta-arago@arago-project.org; Thu, 06 Dec 2012 17:52:30 -0600 (CST) Received: by gandalf.denix.org (Postfix, from userid 1000) id E927920101; Thu, 06 Dec 2012 18:52:13 -0500 (EST) Date: Thu, 06 Dec 2012 18:52:13 -0500 From: Denys Dmytriyenko To: Carlos Hernandez Message-id: <20121206235213.GE17596@denix.org> References: <50BFA673.6040800@ti.com> <8F29D6B095ED194EA1980491A5E029710C342343@DFLE09.ent.ti.com> <50C0FFE3.2090304@ti.com> <20121206203356.GD17596@denix.org> <50C11ADD.5010803@ti.com> MIME-version: 1.0 In-reply-to: <50C11ADD.5010803@ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-arago@arago-project.org" , "Cooper Jr., Franklin" Subject: Re: [PATCH v2] Define arago-test-image 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: Thu, 06 Dec 2012 23:52:44 -0000 Content-type: text/plain; charset=us-ascii Content-disposition: inline On Thu, Dec 06, 2012 at 05:23:25PM -0500, Carlos Hernandez wrote: > Please find v2 attached. Sorry, but any reason you need a whole new task (task-arago-test-addons) in addition to the one you are already adding for the test (task-arago-test)? Right now there's only 1 package listed there (bridge-utils) - do you plan on addin more to that task later on? Denys > Thanks, Carlos On 12/06/2012 03:33 PM, Denys Dmytriyenko wrote: > > On Thu, Dec 06, 2012 at 03:28:19PM -0500, Carlos Hernandez wrote: > >> Franklin, > >> > >> That's a fair point. I did it to keep some independence between the > >> arago-test and arago-tisdk images but I'm OK including > >> task-arago-tisdk-addons instead of task-arago-test-addons. I guess we > >> could separate into different task files only if needed in the future. > >> > >> One task that will need to be added to task-arago-tisdk-addons is > >> bridge-utils. > >> bridge-utils is currently only included by task-arago-test-addons. > >> > >> Let me know if you are OK adding bridge-utils and I can submit another > >> patch with these changes. > > You could also add bridge-utils to your task-arago-test recipe. > > > > Or, you could just add bridge-utils besides the regular task-arago-tisdk-addons > > to IMAGE_INSTALL in your test image recipe: > > > > IMAGE_INSTALL += "\ > > task-arago-base \ > > task-arago-console \ > > task-arago-base-tisdk \ > > task-arago-tisdk-connectivity \ > > task-arago-tisdk-crypto \ > > task-arago-tisdk-multimedia \ > > task-arago-tisdk-addons \ > > task-arago-test \ > > bridge-utils \ > > " > > > > Denys > > > >> Thanks for the review, > >> Carlos > >> > >> On 12/06/2012 02:30 PM, Cooper Jr., Franklin wrote: > >>> Can't task-arago-tisdk-addons be used instead of creating this new task-arago-test-addons recipe? The majority of the content in arago-test-addons is provided by task-arago-addons and the remainder is provided by tasks your pulling into the test image recipe. Other than having a slightly bloated file system with additional tools/libraries is there any reason you can't reuse some of the other tasks? > >>> > >>>> -----Original Message----- > >>>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago- > >>>> bounces@arago-project.org] On Behalf Of Hernandez, Carlos > >>>> Sent: Wednesday, December 05, 2012 1:54 PM > >>>> To: meta-arago@arago-project.org > >>>> Subject: [meta-arago] [PATCH] Define arago-test-image > >>>> > >>>> This filesystem image includes test appplications used by Linux validation team. > >>>> > >>>> Please consider it for meta-arago project. Thanks, Carlos > >>>> > >>>> > >>>> > >> _______________________________________________ > >> meta-arago mailing list > >> meta-arago@arago-project.org > >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago > >> > > From 390814230637c81f4c5e95b49f869d40857e3663 Mon Sep 17 00:00:00 2001 > From: Carlos Hernandez > Date: Thu, 6 Dec 2012 17:31:45 -0500 > Subject: [PATCH] Define arago-test-image > > This filesystem image includes test appplications used by > Linux validation team. > > Signed-off-by: Carlos Hernandez > --- > .../recipes-core/images/arago-test-image.bb | 22 ++++++++++++++++++++ > .../recipes-core/tasks/task-arago-test-addons.bb | 22 ++++++++++++++++++++ > 2 files changed, 44 insertions(+), 0 deletions(-) > create mode 100644 meta-arago-distro/recipes-core/images/arago-test-image.bb > create mode 100644 meta-arago-distro/recipes-core/tasks/task-arago-test-addons.bb > > diff --git a/meta-arago-distro/recipes-core/images/arago-test-image.bb b/meta-arago-distro/recipes-core/images/arago-test-image.bb > new file mode 100644 > index 0000000..3f52729 > --- /dev/null > +++ b/meta-arago-distro/recipes-core/images/arago-test-image.bb > @@ -0,0 +1,22 @@ > +# Arago System Test image > +# gives you test applications > +LICENSE = "MIT" > +LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58" > + > +require arago-image.inc > + > +COMPATIBLE_MACHINE = "omap3|dm365|omapl138|ti813x|ti814x|ti816x|omap4|ti33x" > + > +IMAGE_INSTALL += "\ > + task-arago-base \ > + task-arago-console \ > + task-arago-base-tisdk \ > + task-arago-tisdk-connectivity \ > + task-arago-tisdk-crypto \ > + task-arago-tisdk-multimedia \ > + task-arago-tisdk-addons \ > + task-arago-test \ > + task-arago-test-addons \ > + " > + > +export IMAGE_BASENAME = "arago-test-image" > diff --git a/meta-arago-distro/recipes-core/tasks/task-arago-test-addons.bb b/meta-arago-distro/recipes-core/tasks/task-arago-test-addons.bb > new file mode 100644 > index 0000000..00e9ff6 > --- /dev/null > +++ b/meta-arago-distro/recipes-core/tasks/task-arago-test-addons.bb > @@ -0,0 +1,22 @@ > +DESCRIPTION = "Task to install additional utilities/demos for test image" > +LICENSE = "MIT" > +LIC_FILES_CHKSUM = "file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420" > +PR = "r0" > + > +ALLOW_EMPTY = "1" > + > +inherit task > +PACKAGE_ARCH = "${MACHINE_ARCH}" > + > + > +TEST_ADDONS = " \ > + bridge-utils \ > + " > + > +TEST_ADDONS_TI = " \ > + " > + > +RDEPENDS_${PN} = "\ > + ${TEST_ADDONS} \ > + ${TEST_ADDONS_TI} \ > + " > -- > 1.7.0.4 > > _______________________________________________ > meta-arago mailing list > meta-arago@arago-project.org > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago