All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Define arago-test-image
@ 2012-12-05 19:54 Carlos Hernandez
  2012-12-06 19:30 ` Cooper Jr., Franklin
  0 siblings, 1 reply; 7+ messages in thread
From: Carlos Hernandez @ 2012-12-05 19:54 UTC (permalink / raw)
  To: meta-arago

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

This filesystem image includes test appplications used by
Linux validation team.

Please consider it for meta-arago project. Thanks,
Carlos





[-- Attachment #2: 0001-Define-arago-test-image.patch --]
[-- Type: text/x-patch, Size: 3147 bytes --]

From 3b2e5e76100574e64bfb6d90035e2fa88236fc67 Mon Sep 17 00:00:00 2001
From: Carlos Hernandez <ceh@ti.com>
Date: Tue, 4 Dec 2012 15:31:40 -0500
Subject: [PATCH 01/12] Define arago-test-image

This filesystem image includes test appplications used by
Linux validation team

Signed-off-by: Carlos Hernandez <ceh@ti.com>
---
 .../recipes-core/images/arago-test-image.bb        |   21 ++++++++
 .../recipes-core/tasks/task-arago-test-addons.bb   |   49 ++++++++++++++++++++
 2 files changed, 70 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..9cf07f9
--- /dev/null
+++ b/meta-arago-distro/recipes-core/images/arago-test-image.bb
@@ -0,0 +1,21 @@
+# 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-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..08d80f1
--- /dev/null
+++ b/meta-arago-distro/recipes-core/tasks/task-arago-test-addons.bb
@@ -0,0 +1,49 @@
+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 = " \
+    dt \
+    gdbserver \
+    oprofile \
+    nbench-byte \
+    trace-cmd \
+    bridge-utils \
+    "
+
+TEST_ADDONS_TI = " \
+    "
+
+# Add am-benchmarks for supported devices.
+TEST_ADDONS_TI_append_omapl138 = " arm-benchmarks"
+TEST_ADDONS_TI_append_omap3 = " arm-benchmarks"
+TEST_ADDONS_TI_append_ti33x = " arm-benchmarks"
+TEST_ADDONS_TI_append_ti811x = " arm-benchmarks"
+TEST_ADDONS_TI_append_ti813x = " arm-benchmarks"
+TEST_ADDONS_TI_append_ti814x = " arm-benchmarks"
+TEST_ADDONS_TI_append_ti816x = " arm-benchmarks"
+TEST_ADDONS_TI_append_dm365 = " arm-benchmarks"
+
+# Add PRU examples for am180x-evm devices
+TEST_ADDONS_TI_append_am180x-evm = " ti-pru-sw-examples"
+
+CRYPTO_SUPPORT_TSU = "\
+    openssl \
+    wpa-supplicant \
+    dropbear \
+    openssh-sftp-server \
+    openssl-misc \
+    "
+
+RDEPENDS_${PN} = "\
+    ${TEST_ADDONS} \
+    ${TEST_ADDONS_TI} \
+    ${CRYPTO_SUPPORT_TSU} \
+    "
-- 
1.7.0.4


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] Define arago-test-image
  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
  0 siblings, 1 reply; 7+ messages in thread
From: Cooper Jr., Franklin @ 2012-12-06 19:30 UTC (permalink / raw)
  To: Hernandez, Carlos, meta-arago@arago-project.org

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
> 
> 
> 



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] Define arago-test-image
  2012-12-06 19:30 ` Cooper Jr., Franklin
@ 2012-12-06 20:28   ` Carlos Hernandez
  2012-12-06 20:33     ` Denys Dmytriyenko
  0 siblings, 1 reply; 7+ messages in thread
From: Carlos Hernandez @ 2012-12-06 20:28 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org

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.

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
>>
>>
>>



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] Define arago-test-image
  2012-12-06 20:28   ` Carlos Hernandez
@ 2012-12-06 20:33     ` Denys Dmytriyenko
  2012-12-06 22:23       ` [PATCH v2] " Carlos Hernandez
  0 siblings, 1 reply; 7+ messages in thread
From: Denys Dmytriyenko @ 2012-12-06 20:33 UTC (permalink / raw)
  To: Carlos Hernandez; +Cc: meta-arago@arago-project.org, Cooper Jr., Franklin

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
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] Define arago-test-image
  2012-12-06 20:33     ` Denys Dmytriyenko
@ 2012-12-06 22:23       ` Carlos Hernandez
  2012-12-06 23:52         ` Denys Dmytriyenko
  0 siblings, 1 reply; 7+ messages in thread
From: Carlos Hernandez @ 2012-12-06 22:23 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: meta-arago@arago-project.org, Cooper Jr., Franklin

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

Please find v2 attached.

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
>>


[-- Attachment #2: 0001-Define-arago-test-image.patch --]
[-- Type: text/x-patch, Size: 2369 bytes --]

From 390814230637c81f4c5e95b49f869d40857e3663 Mon Sep 17 00:00:00 2001
From: Carlos Hernandez <ceh@ti.com>
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 <ceh@ti.com>
---
 .../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


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] Define arago-test-image
  2012-12-06 22:23       ` [PATCH v2] " Carlos Hernandez
@ 2012-12-06 23:52         ` Denys Dmytriyenko
  2012-12-07 13:59           ` Hernandez, Carlos
  0 siblings, 1 reply; 7+ messages in thread
From: Denys Dmytriyenko @ 2012-12-06 23:52 UTC (permalink / raw)
  To: Carlos Hernandez; +Cc: meta-arago@arago-project.org, Cooper Jr., Franklin

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 <ceh@ti.com>
> 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 <ceh@ti.com>
> ---
>  .../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



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] Define arago-test-image
  2012-12-06 23:52         ` Denys Dmytriyenko
@ 2012-12-07 13:59           ` Hernandez, Carlos
  0 siblings, 0 replies; 7+ messages in thread
From: Hernandez, Carlos @ 2012-12-07 13:59 UTC (permalink / raw)
  To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org, Cooper Jr., Franklin

The main reason to have task-arago-test-addons is to avoid affecting TISDK builds that already include task-arago-test. 
I also don't see any harm in having both, if give us more flexibility.

Carlos

> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Thursday, December 06, 2012 6:52 PM
> To: Hernandez, Carlos
> Cc: Dmytriyenko, Denys; meta-arago@arago-project.org; Cooper Jr., Franklin
> Subject: Re: [meta-arago] [PATCH v2] Define arago-test-image
> 
> 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 <ceh@ti.com>
> > 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 <ceh@ti.com>
> > ---
> >  .../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



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2012-12-07 13:59 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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
2012-12-06 22:23       ` [PATCH v2] " Carlos Hernandez
2012-12-06 23:52         ` Denys Dmytriyenko
2012-12-07 13:59           ` Hernandez, Carlos

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.