All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: Carlos Hernandez <ceh@ti.com>
Cc: "meta-arago@arago-project.org" <meta-arago@arago-project.org>,
	"Cooper Jr., Franklin" <fcooper@ti.com>
Subject: Re: [PATCH] Define arago-test-image
Date: Thu, 06 Dec 2012 15:33:56 -0500	[thread overview]
Message-ID: <20121206203356.GD17596@denix.org> (raw)
In-Reply-To: <50C0FFE3.2090304@ti.com>

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
> 


  reply	other threads:[~2012-12-06 20:34 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-05 19:54 [PATCH] Define arago-test-image Carlos Hernandez
2012-12-06 19:30 ` Cooper Jr., Franklin
2012-12-06 20:28   ` Carlos Hernandez
2012-12-06 20:33     ` Denys Dmytriyenko [this message]
2012-12-06 22:23       ` [PATCH v2] " Carlos Hernandez
2012-12-06 23:52         ` Denys Dmytriyenko
2012-12-07 13:59           ` Hernandez, Carlos

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20121206203356.GD17596@denix.org \
    --to=denys@ti.com \
    --cc=ceh@ti.com \
    --cc=fcooper@ti.com \
    --cc=meta-arago@arago-project.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.