* [for denzil] kernel.bbclass: Copy bounds.h only if it exists, needed for 2.6.x.
@ 2012-07-31 20:35 sidebranch.openembedded
2012-07-31 22:00 ` Scott Garman
2012-08-01 6:55 ` Koen Kooi
0 siblings, 2 replies; 6+ messages in thread
From: sidebranch.openembedded @ 2012-07-31 20:35 UTC (permalink / raw)
To: yocto; +Cc: Leon Woestenberg
From: Leon Woestenberg <leon@sidebranch.com>
Linux 2.6.x kernels did not (all) have the bounds.h file, so copy
only iff exists.
(See OE-Core 02ac0d1b65389e1779d5f95047f761d7a82ef7a4)
Signed-off-by: Leon Woestenberg <leon@sidebranch.com>
---
meta/classes/kernel.bbclass | 5 +++++
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index c21ab96..5f6ff66 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -183,6 +183,11 @@ kernel_do_install() {
cp arch/powerpc/lib/crtsavres.o $kerneldir/arch/powerpc/lib/crtsavres.o
fi
+ # Necessary for building modules like compat-wireless.
+ if [ -f include/generated/bounds.h ]; then
+ cp include/generated/bounds.h $kerneldir/include/generated/bounds.h
+ fi
+
# Remove the following binaries which cause strip errors
# during do_package for cross-compiled platforms
bin_files="arch/powerpc/boot/addnote arch/powerpc/boot/hack-coff \
--
1.7.0.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [for denzil] kernel.bbclass: Copy bounds.h only if it exists, needed for 2.6.x.
2012-07-31 20:35 [for denzil] kernel.bbclass: Copy bounds.h only if it exists, needed for 2.6.x sidebranch.openembedded
@ 2012-07-31 22:00 ` Scott Garman
2012-07-31 22:08 ` Leon Woestenberg
2012-08-01 6:55 ` Koen Kooi
1 sibling, 1 reply; 6+ messages in thread
From: Scott Garman @ 2012-07-31 22:00 UTC (permalink / raw)
To: yocto
On 07/31/2012 01:35 PM, sidebranch.openembedded@gmail.com wrote:
> From: Leon Woestenberg <leon@sidebranch.com>
>
> Linux 2.6.x kernels did not (all) have the bounds.h file, so copy
> only iff exists.
>
> (See OE-Core 02ac0d1b65389e1779d5f95047f761d7a82ef7a4)
>
> Signed-off-by: Leon Woestenberg <leon@sidebranch.com>
> ---
> meta/classes/kernel.bbclass | 5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
> index c21ab96..5f6ff66 100644
> --- a/meta/classes/kernel.bbclass
> +++ b/meta/classes/kernel.bbclass
> @@ -183,6 +183,11 @@ kernel_do_install() {
> cp arch/powerpc/lib/crtsavres.o $kerneldir/arch/powerpc/lib/crtsavres.o
> fi
>
> + # Necessary for building modules like compat-wireless.
> + if [ -f include/generated/bounds.h ]; then
> + cp include/generated/bounds.h $kerneldir/include/generated/bounds.h
> + fi
> +
> # Remove the following binaries which cause strip errors
> # during do_package for cross-compiled platforms
> bin_files="arch/powerpc/boot/addnote arch/powerpc/boot/hack-coff \
>
Merged into my sgarman/denzil-next branches:
http://git.openembedded.org/openembedded-core-contrib/log/?h=sgarman/denzil-next
http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=sgarman/denzil-next
If this passes a round of autobuilder tests, I'll be submitting a pull
request to the official denzil branch shortly thereafter.
Thanks,
Scott
--
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [for denzil] kernel.bbclass: Copy bounds.h only if it exists, needed for 2.6.x.
2012-07-31 22:00 ` Scott Garman
@ 2012-07-31 22:08 ` Leon Woestenberg
0 siblings, 0 replies; 6+ messages in thread
From: Leon Woestenberg @ 2012-07-31 22:08 UTC (permalink / raw)
To: yocto; +Cc: Scott Garman
[-- Attachment #1: Type: text/plain, Size: 832 bytes --]
On Wed, Aug 1, 2012 at 12:00 AM, Scott Garman <scott.a.garman@intel.com>wrote:
> On 07/31/2012 01:35 PM, sidebranch.openembedded@gmail.com wrote:
>
>> From: Leon Woestenberg <leon@sidebranch.com>
>>
>> Linux 2.6.x kernels did not (all) have the bounds.h file, so copy
>> only iff exists.Merged into my sgarman/denzil-next branches:
>>
>
>
> http://git.openembedded.org/openembedded-core-contrib/log/?h=sgarman/denzil-next
>
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=sgarman/denzil-next
>
> If this passes a round of autobuilder tests, I'll be submitting a pull
> request to the official denzil branch shortly thereafter.
> Thanks,
> Scott
>
> Thanks Scott. Also thanks for the pointers to denzil-next. I will
consider adding those to my build rounds locally also.
Regards,
Leon.
[-- Attachment #2: Type: text/html, Size: 1722 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [for denzil] kernel.bbclass: Copy bounds.h only if it exists, needed for 2.6.x.
2012-07-31 20:35 [for denzil] kernel.bbclass: Copy bounds.h only if it exists, needed for 2.6.x sidebranch.openembedded
2012-07-31 22:00 ` Scott Garman
@ 2012-08-01 6:55 ` Koen Kooi
2012-08-01 9:08 ` Leon Woestenberg
1 sibling, 1 reply; 6+ messages in thread
From: Koen Kooi @ 2012-08-01 6:55 UTC (permalink / raw)
To: sidebranch.openembedded; +Cc: yocto, Leon Woestenberg
Shouldn't this be sent to the oe-core list?!?!?
Op 31 jul. 2012, om 22:35 heeft sidebranch.openembedded@gmail.com het volgende geschreven:
> From: Leon Woestenberg <leon@sidebranch.com>
>
> Linux 2.6.x kernels did not (all) have the bounds.h file, so copy
> only iff exists.
>
> (See OE-Core 02ac0d1b65389e1779d5f95047f761d7a82ef7a4)
>
> Signed-off-by: Leon Woestenberg <leon@sidebranch.com>
> ---
> meta/classes/kernel.bbclass | 5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
> index c21ab96..5f6ff66 100644
> --- a/meta/classes/kernel.bbclass
> +++ b/meta/classes/kernel.bbclass
> @@ -183,6 +183,11 @@ kernel_do_install() {
> cp arch/powerpc/lib/crtsavres.o $kerneldir/arch/powerpc/lib/crtsavres.o
> fi
>
> + # Necessary for building modules like compat-wireless.
> + if [ -f include/generated/bounds.h ]; then
> + cp include/generated/bounds.h $kerneldir/include/generated/bounds.h
> + fi
> +
> # Remove the following binaries which cause strip errors
> # during do_package for cross-compiled platforms
> bin_files="arch/powerpc/boot/addnote arch/powerpc/boot/hack-coff \
> --
> 1.7.0.4
>
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [for denzil] kernel.bbclass: Copy bounds.h only if it exists, needed for 2.6.x.
2012-08-01 6:55 ` Koen Kooi
@ 2012-08-01 9:08 ` Leon Woestenberg
2012-08-01 16:46 ` Scott Garman
0 siblings, 1 reply; 6+ messages in thread
From: Leon Woestenberg @ 2012-08-01 9:08 UTC (permalink / raw)
To: yocto
[-- Attachment #1: Type: text/plain, Size: 339 bytes --]
Hello Koen,
On 08/01/2012 08:55 AM, Koen Kooi wrote:
Shouldn't this be sent to the oe-core list?!?!?
It was intended for the denzil branch ([for denzil]) of Yocto, that's why I
sent it to yocto@yoctoproject.org.
If that was wrong reasoning, let me know, I would be glad to change
workflow if I know why.
Cheers,
Leon.
[-- Attachment #2: Type: text/html, Size: 533 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [for denzil] kernel.bbclass: Copy bounds.h only if it exists, needed for 2.6.x.
2012-08-01 9:08 ` Leon Woestenberg
@ 2012-08-01 16:46 ` Scott Garman
0 siblings, 0 replies; 6+ messages in thread
From: Scott Garman @ 2012-08-01 16:46 UTC (permalink / raw)
To: yocto
On 08/01/2012 02:08 AM, Leon Woestenberg wrote:
> Hello Koen,
>
> On 08/01/2012 08:55 AM, Koen Kooi wrote:
>> Shouldn't this be sent to the oe-core list?!?!?
>>
> It was intended for the denzil branch ([for denzil]) of Yocto, that's
> why I sent it to yocto@yoctoproject.org.
>
> If that was wrong reasoning, let me know, I would be glad to change
> workflow if I know why.
Hi Leon,
Since Yocto is built upon oe-core, changes to files that live in oe-core
should really be sent to the oe-core ML. Our workflow is then to pick
things up from oe-core and move them into the poky git repository.
Both oe-core and poky have denzil branches this patch applies to, so
you'd still make mention in your subject line that the patch is intended
for denzil.
The main reason to submit patches to the Yocto ML is if it's specific to
the meta-yocto layer you find in the Poky git repository, or to discuss
overall Yocto issues.
Just some things to keep in mind for the future.
Thanks,
Scott
--
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-08-01 16:46 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-31 20:35 [for denzil] kernel.bbclass: Copy bounds.h only if it exists, needed for 2.6.x sidebranch.openembedded
2012-07-31 22:00 ` Scott Garman
2012-07-31 22:08 ` Leon Woestenberg
2012-08-01 6:55 ` Koen Kooi
2012-08-01 9:08 ` Leon Woestenberg
2012-08-01 16:46 ` Scott Garman
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.