Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH 0/1] rt-tests: Add src/backfire to FILES_${PN}
@ 2012-03-15 22:00 Darren Hart
  2012-03-15 22:00 ` [PATCH 1/1] " Darren Hart
  2012-03-16  3:35 ` [PATCH 0/1] " Saul Wold
  0 siblings, 2 replies; 13+ messages in thread
From: Darren Hart @ 2012-03-15 22:00 UTC (permalink / raw)
  To: openembedded-core

The following changes since commit d4f1ef6b8297826230a68e535098709d07de1f48:

  qt4: package static libraries correctly to fix warnings (2012-03-15 13:33:37 +0000)

are available in the git repository at:
  git://git.yoctoproject.org/user-contrib/dvhart/oe-core dvhart/updates
  http://git.yoctoproject.org/cgit.cgi/user-contrib/dvhart/oe-core/log/?h=dvhart/updates

Darren Hart (1):
  rt-tests: Add src/backfire to FILES_${PN}

 meta/recipes-rt/rt-tests/rt-tests_0.83.bb |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

-- 
1.7.5.4




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

* [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-15 22:00 [PATCH 0/1] rt-tests: Add src/backfire to FILES_${PN} Darren Hart
@ 2012-03-15 22:00 ` Darren Hart
  2012-03-16 15:33   ` Darren Hart
  2012-03-16 16:06   ` Saul Wold
  2012-03-16  3:35 ` [PATCH 0/1] " Saul Wold
  1 sibling, 2 replies; 13+ messages in thread
From: Darren Hart @ 2012-03-15 22:00 UTC (permalink / raw)
  To: openembedded-core

Fixes [YOCTO #2114]

The rt-tests recipe was issuing the following WARNINGs:

WARNING: For recipe rt-tests, the following files/directories were installed
but not shipped in any package:
WARNING:   /usr/src
WARNING:   /usr/src/backfire
WARNING:   /usr/src/backfire/backfire.c
WARNING:   /usr/src/backfire/Makefile

Add ${prefix}/src/backfire to FILES_${PN} to include these files in the package.
These depend on the target kernel and are meant to be able to be rebuilt as
needed.  Ideally we would also build this module and include it with rt-tests or
the linux-yocto-rt kernel. For now, conform with the intent of the rt-tests
Makefile and avoid the WARNINGs.

Signed-off-by: Darren Hart <dvhart@linux.intel.com>
---
 meta/recipes-rt/rt-tests/rt-tests_0.83.bb |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
index 78d51b3..6425027 100644
--- a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
+++ b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
@@ -11,6 +11,7 @@ SRCREV = "5f1e84f8b015df3ff950056494134eca3f640d70"
 
 # git -> 0.83 needs a PE bump
 PE = "1"
+PR = "r1"
 
 SRC_URI = "git://github.com/clrkwllms/rt-tests.git"
 
@@ -25,3 +26,5 @@ do_install() {
         oe_runmake install DESTDIR=${D} SBINDIR=${sbindir} MANDIR=${mandir} \
                            INCLUDEDIR=${includedir}
 }
+
+FILES_${PN} += "${prefix}/src/backfire"
-- 
1.7.5.4




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

* Re: [PATCH 0/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-15 22:00 [PATCH 0/1] rt-tests: Add src/backfire to FILES_${PN} Darren Hart
  2012-03-15 22:00 ` [PATCH 1/1] " Darren Hart
@ 2012-03-16  3:35 ` Saul Wold
  2012-03-16  5:32   ` Darren Hart
  1 sibling, 1 reply; 13+ messages in thread
From: Saul Wold @ 2012-03-16  3:35 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer; +Cc: Darren Hart

On 03/15/2012 03:00 PM, Darren Hart wrote:
> The following changes since commit d4f1ef6b8297826230a68e535098709d07de1f48:
>
>    qt4: package static libraries correctly to fix warnings (2012-03-15 13:33:37 +0000)
>
> are available in the git repository at:
>    git://git.yoctoproject.org/user-contrib/dvhart/oe-core dvhart/updates
>    http://git.yoctoproject.org/cgit.cgi/user-contrib/dvhart/oe-core/log/?h=dvhart/updates
>
Not sure what's going on here, but this is always harder for me to get 
to then the standard contrib branches.

It turns out it really dvhart/dvhart/updates, not dvhart/updates

Sau!

> Darren Hart (1):
>    rt-tests: Add src/backfire to FILES_${PN}
>
>   meta/recipes-rt/rt-tests/rt-tests_0.83.bb |    3 +++
>   1 files changed, 3 insertions(+), 0 deletions(-)
>



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

* Re: [PATCH 0/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-16  3:35 ` [PATCH 0/1] " Saul Wold
@ 2012-03-16  5:32   ` Darren Hart
  0 siblings, 0 replies; 13+ messages in thread
From: Darren Hart @ 2012-03-16  5:32 UTC (permalink / raw)
  To: Saul Wold; +Cc: Patches and discussions about the oe-core layer



On 03/15/2012 08:35 PM, Saul Wold wrote:
> On 03/15/2012 03:00 PM, Darren Hart wrote:
>> The following changes since commit d4f1ef6b8297826230a68e535098709d07de1f48:
>>
>>    qt4: package static libraries correctly to fix warnings (2012-03-15 13:33:37 +0000)
>>
>> are available in the git repository at:
>>    git://git.yoctoproject.org/user-contrib/dvhart/oe-core dvhart/updates
>>    http://git.yoctoproject.org/cgit.cgi/user-contrib/dvhart/oe-core/log/?h=dvhart/updates
>>
> Not sure what's going on here, but this is always harder for me to get 
> to then the standard contrib branches.
> 
> It turns out it really dvhart/dvhart/updates, not dvhart/updates

No, the branch is dvhart/updates:

$ git clone git://git.yoctoproject.org/user-contrib/dvhart/oe-core -b dvhart/updates
$ cd oe-core
$ git show HEAD
commit 5ca651eb19c3fcb64de2ce405de0bc7fa2a8c90b
Author: Darren Hart <dvhart@linux.intel.com>
Date:   Thu Mar 15 14:51:22 2012 -0700

    rt-tests: Add src/backfire to FILES_${PN}
...


So I wonder what you are seeing? Maybe you call the new remote "dvhart" ?
If so, then yes, you would refer to the commit as remote/branch which would
be dvhart/dvhart/updates. So yeah, I don't need the dvhart in the branch
name since this is a personal repo... just an old habit.

I also see that I haven't got this made public yet. I'll fix that.

--
Darren

> 
> Sau!
> 
>> Darren Hart (1):
>>    rt-tests: Add src/backfire to FILES_${PN}
>>
>>   meta/recipes-rt/rt-tests/rt-tests_0.83.bb |    3 +++
>>   1 files changed, 3 insertions(+), 0 deletions(-)
>>

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel



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

* Re: [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-15 22:00 ` [PATCH 1/1] " Darren Hart
@ 2012-03-16 15:33   ` Darren Hart
  2012-03-16 15:55     ` Darren Hart
  2012-03-16 16:06   ` Saul Wold
  1 sibling, 1 reply; 13+ messages in thread
From: Darren Hart @ 2012-03-16 15:33 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

Hrm, this silences the warnings, but the backfire sources are not
installed on the rootfs....

Anyone see what is wrong with the below? Is there some magic that purges
/usr/src in a minimal build?

--
Darren

On 03/15/2012 03:00 PM, Darren Hart wrote:
> Fixes [YOCTO #2114]
> 
> The rt-tests recipe was issuing the following WARNINGs:
> 
> WARNING: For recipe rt-tests, the following files/directories were installed
> but not shipped in any package:
> WARNING:   /usr/src
> WARNING:   /usr/src/backfire
> WARNING:   /usr/src/backfire/backfire.c
> WARNING:   /usr/src/backfire/Makefile
> 
> Add ${prefix}/src/backfire to FILES_${PN} to include these files in the package.
> These depend on the target kernel and are meant to be able to be rebuilt as
> needed.  Ideally we would also build this module and include it with rt-tests or
> the linux-yocto-rt kernel. For now, conform with the intent of the rt-tests
> Makefile and avoid the WARNINGs.
> 
> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
> ---
>  meta/recipes-rt/rt-tests/rt-tests_0.83.bb |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
> index 78d51b3..6425027 100644
> --- a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
> +++ b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
> @@ -11,6 +11,7 @@ SRCREV = "5f1e84f8b015df3ff950056494134eca3f640d70"
>  
>  # git -> 0.83 needs a PE bump
>  PE = "1"
> +PR = "r1"
>  
>  SRC_URI = "git://github.com/clrkwllms/rt-tests.git"
>  
> @@ -25,3 +26,5 @@ do_install() {
>          oe_runmake install DESTDIR=${D} SBINDIR=${sbindir} MANDIR=${mandir} \
>                             INCLUDEDIR=${includedir}
>  }
> +
> +FILES_${PN} += "${prefix}/src/backfire"

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel



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

* Re: [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-16 15:33   ` Darren Hart
@ 2012-03-16 15:55     ` Darren Hart
  2012-03-16 16:26       ` Richard Purdie
  0 siblings, 1 reply; 13+ messages in thread
From: Darren Hart @ 2012-03-16 15:55 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 03/16/2012 08:33 AM, Darren Hart wrote:
> Hrm, this silences the warnings, but the backfire sources are not
> installed on the rootfs....
> 
> Anyone see what is wrong with the below? Is there some magic that purges
> /usr/src in a minimal build?

I was adding "/usr/src/backfire" to FILES_${PN}. It seems one should add
"/usr/src/backfire/*" to actually get the files.

However, I am surprised that the warnings went away and the files were
still not packaged when I used "/usr/src/backfire". Is this expected
behavior?

I have pushed the new version to the same branch using
"/usr/src/backfire/*". Built,booted,and verified on qemux86 using
core-image-rt.

--
Darren

> 
> --
> Darren
> 
> On 03/15/2012 03:00 PM, Darren Hart wrote:
>> Fixes [YOCTO #2114]
>>
>> The rt-tests recipe was issuing the following WARNINGs:
>>
>> WARNING: For recipe rt-tests, the following files/directories were installed
>> but not shipped in any package:
>> WARNING:   /usr/src
>> WARNING:   /usr/src/backfire
>> WARNING:   /usr/src/backfire/backfire.c
>> WARNING:   /usr/src/backfire/Makefile
>>
>> Add ${prefix}/src/backfire to FILES_${PN} to include these files in the package.
>> These depend on the target kernel and are meant to be able to be rebuilt as
>> needed.  Ideally we would also build this module and include it with rt-tests or
>> the linux-yocto-rt kernel. For now, conform with the intent of the rt-tests
>> Makefile and avoid the WARNINGs.
>>
>> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
>> ---
>>  meta/recipes-rt/rt-tests/rt-tests_0.83.bb |    3 +++
>>  1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
>> index 78d51b3..6425027 100644
>> --- a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
>> +++ b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
>> @@ -11,6 +11,7 @@ SRCREV = "5f1e84f8b015df3ff950056494134eca3f640d70"
>>  
>>  # git -> 0.83 needs a PE bump
>>  PE = "1"
>> +PR = "r1"
>>  
>>  SRC_URI = "git://github.com/clrkwllms/rt-tests.git"
>>  
>> @@ -25,3 +26,5 @@ do_install() {
>>          oe_runmake install DESTDIR=${D} SBINDIR=${sbindir} MANDIR=${mandir} \
>>                             INCLUDEDIR=${includedir}
>>  }
>> +
>> +FILES_${PN} += "${prefix}/src/backfire"
> 

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel



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

* Re: [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-15 22:00 ` [PATCH 1/1] " Darren Hart
  2012-03-16 15:33   ` Darren Hart
@ 2012-03-16 16:06   ` Saul Wold
  2012-03-16 16:16     ` Darren Hart
  2012-03-16 16:24     ` Richard Purdie
  1 sibling, 2 replies; 13+ messages in thread
From: Saul Wold @ 2012-03-16 16:06 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer; +Cc: Darren Hart

On 03/15/2012 03:00 PM, Darren Hart wrote:
> Fixes [YOCTO #2114]
>
> The rt-tests recipe was issuing the following WARNINGs:
>
> WARNING: For recipe rt-tests, the following files/directories were installed
> but not shipped in any package:
> WARNING:   /usr/src
> WARNING:   /usr/src/backfire
> WARNING:   /usr/src/backfire/backfire.c
> WARNING:   /usr/src/backfire/Makefile
>
> Add ${prefix}/src/backfire to FILES_${PN} to include these files in the package.
> These depend on the target kernel and are meant to be able to be rebuilt as
> needed.  Ideally we would also build this module and include it with rt-tests or
> the linux-yocto-rt kernel. For now, conform with the intent of the rt-tests
> Makefile and avoid the WARNINGs.
>
> Signed-off-by: Darren Hart<dvhart@linux.intel.com>
> ---
>   meta/recipes-rt/rt-tests/rt-tests_0.83.bb |    3 +++
>   1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
> index 78d51b3..6425027 100644
> --- a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
> +++ b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
> @@ -11,6 +11,7 @@ SRCREV = "5f1e84f8b015df3ff950056494134eca3f640d70"
>
>   # git ->  0.83 needs a PE bump
>   PE = "1"
> +PR = "r1"
>
>   SRC_URI = "git://github.com/clrkwllms/rt-tests.git"
>
> @@ -25,3 +26,5 @@ do_install() {
>           oe_runmake install DESTDIR=${D} SBINDIR=${sbindir} MANDIR=${mandir} \
>                              INCLUDEDIR=${includedir}
>   }
> +
> +FILES_${PN} += "${prefix}/src/backfire"
Darren,

I looked closer and you need you use ${D}${prefix}/src here instead, 
that will get you what you want.

Sau!



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

* Re: [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-16 16:06   ` Saul Wold
@ 2012-03-16 16:16     ` Darren Hart
  2012-03-16 16:24     ` Richard Purdie
  1 sibling, 0 replies; 13+ messages in thread
From: Darren Hart @ 2012-03-16 16:16 UTC (permalink / raw)
  To: Saul Wold; +Cc: Patches and discussions about the oe-core layer



On 03/16/2012 09:06 AM, Saul Wold wrote:
> On 03/15/2012 03:00 PM, Darren Hart wrote:
>> Fixes [YOCTO #2114]
>>
>> The rt-tests recipe was issuing the following WARNINGs:
>>
>> WARNING: For recipe rt-tests, the following files/directories were installed
>> but not shipped in any package:
>> WARNING:   /usr/src
>> WARNING:   /usr/src/backfire
>> WARNING:   /usr/src/backfire/backfire.c
>> WARNING:   /usr/src/backfire/Makefile
>>
>> Add ${prefix}/src/backfire to FILES_${PN} to include these files in the package.
>> These depend on the target kernel and are meant to be able to be rebuilt as
>> needed.  Ideally we would also build this module and include it with rt-tests or
>> the linux-yocto-rt kernel. For now, conform with the intent of the rt-tests
>> Makefile and avoid the WARNINGs.
>>
>> Signed-off-by: Darren Hart<dvhart@linux.intel.com>
>> ---
>>   meta/recipes-rt/rt-tests/rt-tests_0.83.bb |    3 +++
>>   1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
>> index 78d51b3..6425027 100644
>> --- a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
>> +++ b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
>> @@ -11,6 +11,7 @@ SRCREV = "5f1e84f8b015df3ff950056494134eca3f640d70"
>>
>>   # git ->  0.83 needs a PE bump
>>   PE = "1"
>> +PR = "r1"
>>
>>   SRC_URI = "git://github.com/clrkwllms/rt-tests.git"
>>
>> @@ -25,3 +26,5 @@ do_install() {
>>           oe_runmake install DESTDIR=${D} SBINDIR=${sbindir} MANDIR=${mandir} \
>>                              INCLUDEDIR=${includedir}
>>   }
>> +
>> +FILES_${PN} += "${prefix}/src/backfire"
> Darren,
> 
> I looked closer and you need you use ${D}${prefix}/src here instead, 
> that will get you what you want.


Hrm, that isn't consistent with the rest of the usage I see in oe-core:

$ git grep 'FILES_${PN}.*${prefix}'
classes/cross-canadian.bbclass:FILES_${PN} = "${prefix}"
classes/cross-canadian.bbclass:FILES_${PN}-dbg += "${prefix}/.debug \
recipes-devtools/binutils/binutils.inc:FILES_${PN}-dbg += "${prefix}/${TARGET_SYS}/
recipes-devtools/rpm/rpm_5.4.0.bb:FILES_${PN}-build = "${prefix}/src/rpm \
recipes-devtools/tcltk/tcl_8.5.11.bb:FILES_${PN} += "${prefix}/lib/tcl8.5 ${prefix}
recipes-sato/puzzles/puzzles_r9375.bb:FILES_${PN} = "${prefix}/games/* ${datadir}/a
recipes-sato/puzzles/puzzles_r9375.bb:FILES_${PN}-dbg += "${prefix}/games/.debug"

Also, it works as intended if I use "${prefix}/src/backfire/*" and this is
consistent with existing usage.

What lead you to suggest "${D}${prefix}" ?

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel



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

* Re: [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-16 16:06   ` Saul Wold
  2012-03-16 16:16     ` Darren Hart
@ 2012-03-16 16:24     ` Richard Purdie
  1 sibling, 0 replies; 13+ messages in thread
From: Richard Purdie @ 2012-03-16 16:24 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer; +Cc: Darren Hart

On Fri, 2012-03-16 at 09:06 -0700, Saul Wold wrote:
> On 03/15/2012 03:00 PM, Darren Hart wrote:
> > Fixes [YOCTO #2114]
> >
> > The rt-tests recipe was issuing the following WARNINGs:
> >
> > WARNING: For recipe rt-tests, the following files/directories were installed
> > but not shipped in any package:
> > WARNING:   /usr/src
> > WARNING:   /usr/src/backfire
> > WARNING:   /usr/src/backfire/backfire.c
> > WARNING:   /usr/src/backfire/Makefile
> >
> > Add ${prefix}/src/backfire to FILES_${PN} to include these files in the package.
> > These depend on the target kernel and are meant to be able to be rebuilt as
> > needed.  Ideally we would also build this module and include it with rt-tests or
> > the linux-yocto-rt kernel. For now, conform with the intent of the rt-tests
> > Makefile and avoid the WARNINGs.
> >
> > Signed-off-by: Darren Hart<dvhart@linux.intel.com>
> > ---
> >   meta/recipes-rt/rt-tests/rt-tests_0.83.bb |    3 +++
> >   1 files changed, 3 insertions(+), 0 deletions(-)
> >
> > diff --git a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
> > index 78d51b3..6425027 100644
> > --- a/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
> > +++ b/meta/recipes-rt/rt-tests/rt-tests_0.83.bb
> > @@ -11,6 +11,7 @@ SRCREV = "5f1e84f8b015df3ff950056494134eca3f640d70"
> >
> >   # git ->  0.83 needs a PE bump
> >   PE = "1"
> > +PR = "r1"
> >
> >   SRC_URI = "git://github.com/clrkwllms/rt-tests.git"
> >
> > @@ -25,3 +26,5 @@ do_install() {
> >           oe_runmake install DESTDIR=${D} SBINDIR=${sbindir} MANDIR=${mandir} \
> >                              INCLUDEDIR=${includedir}
> >   }
> > +
> > +FILES_${PN} += "${prefix}/src/backfire"
> Darren,
> 
> I looked closer and you need you use ${D}${prefix}/src here instead, 
> that will get you what you want.

You should not need ${D} here...

Cheers,

Richard




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

* Re: [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-16 15:55     ` Darren Hart
@ 2012-03-16 16:26       ` Richard Purdie
  2012-03-16 16:28         ` Darren Hart
  0 siblings, 1 reply; 13+ messages in thread
From: Richard Purdie @ 2012-03-16 16:26 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Fri, 2012-03-16 at 08:55 -0700, Darren Hart wrote:
> On 03/16/2012 08:33 AM, Darren Hart wrote:
> > Hrm, this silences the warnings, but the backfire sources are not
> > installed on the rootfs....
> > 
> > Anyone see what is wrong with the below? Is there some magic that purges
> > /usr/src in a minimal build?
> 
> I was adding "/usr/src/backfire" to FILES_${PN}. It seems one should add
> "/usr/src/backfire/*" to actually get the files.
> 
> However, I am surprised that the warnings went away and the files were
> still not packaged when I used "/usr/src/backfire". Is this expected
> behavior?
> 
> I have pushed the new version to the same branch using
> "/usr/src/backfire/*". Built,booted,and verified on qemux86 using
> core-image-rt.

FWIW with your original patch I'm seeing things being correctly packaged
in the main package (rt-tests) which is consistent with the warning
message being fixed.

Cheers,

Richard




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

* Re: [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-16 16:26       ` Richard Purdie
@ 2012-03-16 16:28         ` Darren Hart
  2012-03-16 17:11           ` Richard Purdie
  0 siblings, 1 reply; 13+ messages in thread
From: Darren Hart @ 2012-03-16 16:28 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer



On 03/16/2012 09:26 AM, Richard Purdie wrote:
> On Fri, 2012-03-16 at 08:55 -0700, Darren Hart wrote:
>> On 03/16/2012 08:33 AM, Darren Hart wrote:
>>> Hrm, this silences the warnings, but the backfire sources are not
>>> installed on the rootfs....
>>>
>>> Anyone see what is wrong with the below? Is there some magic that purges
>>> /usr/src in a minimal build?
>>
>> I was adding "/usr/src/backfire" to FILES_${PN}. It seems one should add
>> "/usr/src/backfire/*" to actually get the files.
>>
>> However, I am surprised that the warnings went away and the files were
>> still not packaged when I used "/usr/src/backfire". Is this expected
>> behavior?
>>
>> I have pushed the new version to the same branch using
>> "/usr/src/backfire/*". Built,booted,and verified on qemux86 using
>> core-image-rt.
> 
> FWIW with your original patch I'm seeing things being correctly packaged
> in the main package (rt-tests) which is consistent with the warning
> message being fixed.

I see them appear in the rt-tests "package" dir, but not in the rootfs
unless I append the "/*".

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel



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

* Re: [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-16 16:28         ` Darren Hart
@ 2012-03-16 17:11           ` Richard Purdie
  2012-03-16 17:59             ` Darren Hart
  0 siblings, 1 reply; 13+ messages in thread
From: Richard Purdie @ 2012-03-16 17:11 UTC (permalink / raw)
  To: Darren Hart; +Cc: Patches and discussions about the oe-core layer

On Fri, 2012-03-16 at 09:28 -0700, Darren Hart wrote:
> 
> On 03/16/2012 09:26 AM, Richard Purdie wrote:
> > On Fri, 2012-03-16 at 08:55 -0700, Darren Hart wrote:
> >> On 03/16/2012 08:33 AM, Darren Hart wrote:
> >>> Hrm, this silences the warnings, but the backfire sources are not
> >>> installed on the rootfs....
> >>>
> >>> Anyone see what is wrong with the below? Is there some magic that purges
> >>> /usr/src in a minimal build?
> >>
> >> I was adding "/usr/src/backfire" to FILES_${PN}. It seems one should add
> >> "/usr/src/backfire/*" to actually get the files.
> >>
> >> However, I am surprised that the warnings went away and the files were
> >> still not packaged when I used "/usr/src/backfire". Is this expected
> >> behavior?
> >>
> >> I have pushed the new version to the same branch using
> >> "/usr/src/backfire/*". Built,booted,and verified on qemux86 using
> >> core-image-rt.
> > 
> > FWIW with your original patch I'm seeing things being correctly packaged
> > in the main package (rt-tests) which is consistent with the warning
> > message being fixed.
> 
> I see them appear in the rt-tests "package" dir, but not in the rootfs
> unless I append the "/*".

If I view the contents of the rt-tests*.rpm in deploy/rpm/xxx/, I can
see the files listed. This is with current master.

Please check what you tested ;-)

Cheers,

Richard




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

* Re: [PATCH 1/1] rt-tests: Add src/backfire to FILES_${PN}
  2012-03-16 17:11           ` Richard Purdie
@ 2012-03-16 17:59             ` Darren Hart
  0 siblings, 0 replies; 13+ messages in thread
From: Darren Hart @ 2012-03-16 17:59 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer



On 03/16/2012 10:11 AM, Richard Purdie wrote:
> On Fri, 2012-03-16 at 09:28 -0700, Darren Hart wrote:
>>
>> On 03/16/2012 09:26 AM, Richard Purdie wrote:
>>> On Fri, 2012-03-16 at 08:55 -0700, Darren Hart wrote:
>>>> On 03/16/2012 08:33 AM, Darren Hart wrote:
>>>>> Hrm, this silences the warnings, but the backfire sources are not
>>>>> installed on the rootfs....
>>>>>
>>>>> Anyone see what is wrong with the below? Is there some magic that purges
>>>>> /usr/src in a minimal build?
>>>>
>>>> I was adding "/usr/src/backfire" to FILES_${PN}. It seems one should add
>>>> "/usr/src/backfire/*" to actually get the files.
>>>>
>>>> However, I am surprised that the warnings went away and the files were
>>>> still not packaged when I used "/usr/src/backfire". Is this expected
>>>> behavior?
>>>>
>>>> I have pushed the new version to the same branch using
>>>> "/usr/src/backfire/*". Built,booted,and verified on qemux86 using
>>>> core-image-rt.
>>>
>>> FWIW with your original patch I'm seeing things being correctly packaged
>>> in the main package (rt-tests) which is consistent with the warning
>>> message being fixed.
>>
>> I see them appear in the rt-tests "package" dir, but not in the rootfs
>> unless I append the "/*".
> 
> If I view the contents of the rt-tests*.rpm in deploy/rpm/xxx/, I can
> see the files listed. This is with current master.
> 
> Please check what you tested ;-)
> 

I can't explain it, but the one in master (no "/*") is working correctly
per my last build.

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel



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

end of thread, other threads:[~2012-03-16 18:08 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-15 22:00 [PATCH 0/1] rt-tests: Add src/backfire to FILES_${PN} Darren Hart
2012-03-15 22:00 ` [PATCH 1/1] " Darren Hart
2012-03-16 15:33   ` Darren Hart
2012-03-16 15:55     ` Darren Hart
2012-03-16 16:26       ` Richard Purdie
2012-03-16 16:28         ` Darren Hart
2012-03-16 17:11           ` Richard Purdie
2012-03-16 17:59             ` Darren Hart
2012-03-16 16:06   ` Saul Wold
2012-03-16 16:16     ` Darren Hart
2012-03-16 16:24     ` Richard Purdie
2012-03-16  3:35 ` [PATCH 0/1] " Saul Wold
2012-03-16  5:32   ` Darren Hart

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox