* [PATCH] linux-kernel-base: linux/version.h path update for 3.7-rc1
@ 2013-03-20 13:22 anders.roxell
2013-03-20 13:42 ` Bruce Ashfield
0 siblings, 1 reply; 6+ messages in thread
From: anders.roxell @ 2013-03-20 13:22 UTC (permalink / raw)
To: openembedded-core; +Cc: Anders Roxell
From: Anders Roxell <anders.roxell@enea.com>
In mainline kernel 3.7-rc1 (hash:abbf1590de22a6d22)
The path include/linux/version.h was updated to
include/generated/uapi/linux/version.h
Signed-off-by: Anders Roxell <anders.roxell@enea.com>
---
meta/classes/linux-kernel-base.bbclass | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/meta/classes/linux-kernel-base.bbclass b/meta/classes/linux-kernel-base.bbclass
index 4f2b0a4..b7d0ffa 100644
--- a/meta/classes/linux-kernel-base.bbclass
+++ b/meta/classes/linux-kernel-base.bbclass
@@ -8,6 +8,9 @@ def get_kernelversion(p):
fn = p + '/include/generated/utsrelease.h'
if not os.path.isfile(fn):
fn = p + '/include/linux/version.h'
+ if not os.path.isfile(fn):
+ # after 3.7-rc1
+ fn = p + '/include/generated/uapi/linux/version.h'
import re
try:
--
1.7.5.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] linux-kernel-base: linux/version.h path update for 3.7-rc1
2013-03-20 13:22 [PATCH] linux-kernel-base: linux/version.h path update for 3.7-rc1 anders.roxell
@ 2013-03-20 13:42 ` Bruce Ashfield
2013-03-20 14:55 ` Anders Roxell
0 siblings, 1 reply; 6+ messages in thread
From: Bruce Ashfield @ 2013-03-20 13:42 UTC (permalink / raw)
To: anders.roxell
Cc: Anders Roxell, Patches and discussions about the oe-core layer
On Wed, Mar 20, 2013 at 9:22 AM, <anders.roxell@gmail.com> wrote:
> From: Anders Roxell <anders.roxell@enea.com>
>
> In mainline kernel 3.7-rc1 (hash:abbf1590de22a6d22)
> The path include/linux/version.h was updated to
> include/generated/uapi/linux/version.h
It's true that uapi now holds the generated version.h, but what build
error were you seeing without this change in place ? utsrelease.h is
still created where it has been for some time, and should be what is
typically used for any version checks.
Cheers,
Bruce
>
> Signed-off-by: Anders Roxell <anders.roxell@enea.com>
> ---
> meta/classes/linux-kernel-base.bbclass | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/meta/classes/linux-kernel-base.bbclass b/meta/classes/linux-kernel-base.bbclass
> index 4f2b0a4..b7d0ffa 100644
> --- a/meta/classes/linux-kernel-base.bbclass
> +++ b/meta/classes/linux-kernel-base.bbclass
> @@ -8,6 +8,9 @@ def get_kernelversion(p):
> fn = p + '/include/generated/utsrelease.h'
> if not os.path.isfile(fn):
> fn = p + '/include/linux/version.h'
> + if not os.path.isfile(fn):
> + # after 3.7-rc1
> + fn = p + '/include/generated/uapi/linux/version.h'
>
> import re
> try:
> --
> 1.7.5.4
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
--
"Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end"
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] linux-kernel-base: linux/version.h path update for 3.7-rc1
2013-03-20 13:42 ` Bruce Ashfield
@ 2013-03-20 14:55 ` Anders Roxell
2013-03-20 15:30 ` Bruce Ashfield
0 siblings, 1 reply; 6+ messages in thread
From: Anders Roxell @ 2013-03-20 14:55 UTC (permalink / raw)
To: Bruce Ashfield
Cc: Anders Roxell, Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 1990 bytes --]
On Wed, Mar 20, 2013 at 2:42 PM, Bruce Ashfield <bruce.ashfield@gmail.com>wrote:
> On Wed, Mar 20, 2013 at 9:22 AM, <anders.roxell@gmail.com> wrote:
> > From: Anders Roxell <anders.roxell@enea.com>
> >
> > In mainline kernel 3.7-rc1 (hash:abbf1590de22a6d22)
> > The path include/linux/version.h was updated to
> > include/generated/uapi/linux/version.h
>
> It's true that uapi now holds the generated version.h, but what build
> error were you seeing without this change in place ?
We were trying to build linx (
http://linx.sourceforge.net/linxdoc/doc/index.html)
Please let me know if it is better to rewrite linx or include this patch?
Regards,
Anders
> utsrelease.h is
> still created where it has been for some time, and should be what is
> typically used for any version checks.
>
> Cheers,
>
> Bruce
>
>
>
> >
> > Signed-off-by: Anders Roxell <anders.roxell@enea.com>
> > ---
> > meta/classes/linux-kernel-base.bbclass | 3 +++
> > 1 files changed, 3 insertions(+), 0 deletions(-)
> >
> > diff --git a/meta/classes/linux-kernel-base.bbclass
> b/meta/classes/linux-kernel-base.bbclass
> > index 4f2b0a4..b7d0ffa 100644
> > --- a/meta/classes/linux-kernel-base.bbclass
> > +++ b/meta/classes/linux-kernel-base.bbclass
> > @@ -8,6 +8,9 @@ def get_kernelversion(p):
> > fn = p + '/include/generated/utsrelease.h'
> > if not os.path.isfile(fn):
> > fn = p + '/include/linux/version.h'
> > + if not os.path.isfile(fn):
> > + # after 3.7-rc1
> > + fn = p + '/include/generated/uapi/linux/version.h'
> >
> > import re
> > try:
> > --
> > 1.7.5.4
> >
> >
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>
>
> --
> "Thou shalt not follow the NULL pointer, for chaos and madness await
> thee at its end"
>
[-- Attachment #2: Type: text/html, Size: 3291 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] linux-kernel-base: linux/version.h path update for 3.7-rc1
2013-03-20 14:55 ` Anders Roxell
@ 2013-03-20 15:30 ` Bruce Ashfield
2013-03-27 9:25 ` Anders Roxell
0 siblings, 1 reply; 6+ messages in thread
From: Bruce Ashfield @ 2013-03-20 15:30 UTC (permalink / raw)
To: Anders Roxell
Cc: Anders Roxell, Patches and discussions about the oe-core layer
On Wed, Mar 20, 2013 at 10:55 AM, Anders Roxell <anders.roxell@gmail.com> wrote:
>
>
>
> On Wed, Mar 20, 2013 at 2:42 PM, Bruce Ashfield <bruce.ashfield@gmail.com>
> wrote:
>>
>> On Wed, Mar 20, 2013 at 9:22 AM, <anders.roxell@gmail.com> wrote:
>> > From: Anders Roxell <anders.roxell@enea.com>
>> >
>> > In mainline kernel 3.7-rc1 (hash:abbf1590de22a6d22)
>> > The path include/linux/version.h was updated to
>> > include/generated/uapi/linux/version.h
>>
>> It's true that uapi now holds the generated version.h, but what build
>> error were you seeing without this change in place ?
>
>
> We were trying to build linx
> (http://linx.sourceforge.net/linxdoc/doc/index.html)
> Please let me know if it is better to rewrite linx or include this patch?
My concern is that as a list of alternatives builds up over time, that don't
necessarily use the same format of the linux version, you'll end up with
applications that are making the wrong decisions if a fallback is used. Having
a clear public / exported file, and throwing errors if it doesn't exist would
ensure consistent behaviour (granted annoying if you get the error when you
think you shouldn't).
If you look at the code in get_kernelversion() it shouldn't even work if
version.h is used versus utsrelease.h. I'd be inclined to suggest the
removal of the existing version.h fallback, versus adding another.
You might not even need to modify the application code to fix this. What
are the dependencies of your recipe that would allow for utsrelease.h
to not be generated when the build calls get_kernelversion ?
Cheers,
Bruce
>
> Regards,
> Anders
>
>>
>> utsrelease.h is
>> still created where it has been for some time, and should be what is
>> typically used for any version checks.
>>
>> Cheers,
>>
>> Bruce
>>
>>
>>
>> >
>> > Signed-off-by: Anders Roxell <anders.roxell@enea.com>
>> > ---
>> > meta/classes/linux-kernel-base.bbclass | 3 +++
>> > 1 files changed, 3 insertions(+), 0 deletions(-)
>> >
>> > diff --git a/meta/classes/linux-kernel-base.bbclass
>> > b/meta/classes/linux-kernel-base.bbclass
>> > index 4f2b0a4..b7d0ffa 100644
>> > --- a/meta/classes/linux-kernel-base.bbclass
>> > +++ b/meta/classes/linux-kernel-base.bbclass
>> > @@ -8,6 +8,9 @@ def get_kernelversion(p):
>> > fn = p + '/include/generated/utsrelease.h'
>> > if not os.path.isfile(fn):
>> > fn = p + '/include/linux/version.h'
>> > + if not os.path.isfile(fn):
>> > + # after 3.7-rc1
>> > + fn = p + '/include/generated/uapi/linux/version.h'
>> >
>> > import re
>> > try:
>> > --
>> > 1.7.5.4
>> >
>> >
>> > _______________________________________________
>> > Openembedded-core mailing list
>> > Openembedded-core@lists.openembedded.org
>> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>>
>>
>> --
>> "Thou shalt not follow the NULL pointer, for chaos and madness await
>> thee at its end"
>
>
--
"Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end"
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] linux-kernel-base: linux/version.h path update for 3.7-rc1
2013-03-20 15:30 ` Bruce Ashfield
@ 2013-03-27 9:25 ` Anders Roxell
2013-03-27 12:42 ` Bruce Ashfield
0 siblings, 1 reply; 6+ messages in thread
From: Anders Roxell @ 2013-03-27 9:25 UTC (permalink / raw)
To: Bruce Ashfield
Cc: Anders Roxell, Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 3680 bytes --]
On Wed, Mar 20, 2013 at 4:30 PM, Bruce Ashfield <bruce.ashfield@gmail.com>wrote:
> On Wed, Mar 20, 2013 at 10:55 AM, Anders Roxell <anders.roxell@gmail.com>
> wrote:
> >
> >
> >
> > On Wed, Mar 20, 2013 at 2:42 PM, Bruce Ashfield <
> bruce.ashfield@gmail.com>
> > wrote:
> >>
> >> On Wed, Mar 20, 2013 at 9:22 AM, <anders.roxell@gmail.com> wrote:
> >> > From: Anders Roxell <anders.roxell@enea.com>
> >> >
> >> > In mainline kernel 3.7-rc1 (hash:abbf1590de22a6d22)
> >> > The path include/linux/version.h was updated to
> >> > include/generated/uapi/linux/version.h
> >>
> >> It's true that uapi now holds the generated version.h, but what build
> >> error were you seeing without this change in place ?
> >
> >
> > We were trying to build linx
> > (http://linx.sourceforge.net/linxdoc/doc/index.html)
> > Please let me know if it is better to rewrite linx or include this patch?
>
> My concern is that as a list of alternatives builds up over time, that
> don't
> necessarily use the same format of the linux version, you'll end up with
> applications that are making the wrong decisions if a fallback is used.
> Having
> a clear public / exported file, and throwing errors if it doesn't exist
> would
> ensure consistent behaviour (granted annoying if you get the error when you
> think you shouldn't).
>
> If you look at the code in get_kernelversion() it shouldn't even work if
> version.h is used versus utsrelease.h. I'd be inclined to suggest the
> removal of the existing version.h fallback, versus adding another.
>
> You might not even need to modify the application code to fix this. What
> are the dependencies of your recipe that would allow for utsrelease.h
> to not be generated when the build calls get_kernelversion ?
>
>
Sorry for the late response!
I agree with you that its better to use utsrelease.h rather than depending
on version.h.
We have added a patch for the linx source to solve this.
Thank you very much for the help and support!
With kind regards,
Anders
> Cheers,
>
> Bruce
>
> >
> > Regards,
> > Anders
> >
> >>
> >> utsrelease.h is
> >> still created where it has been for some time, and should be what is
> >> typically used for any version checks.
> >>
> >> Cheers,
> >>
> >> Bruce
> >>
> >>
> >>
> >> >
> >> > Signed-off-by: Anders Roxell <anders.roxell@enea.com>
> >> > ---
> >> > meta/classes/linux-kernel-base.bbclass | 3 +++
> >> > 1 files changed, 3 insertions(+), 0 deletions(-)
> >> >
> >> > diff --git a/meta/classes/linux-kernel-base.bbclass
> >> > b/meta/classes/linux-kernel-base.bbclass
> >> > index 4f2b0a4..b7d0ffa 100644
> >> > --- a/meta/classes/linux-kernel-base.bbclass
> >> > +++ b/meta/classes/linux-kernel-base.bbclass
> >> > @@ -8,6 +8,9 @@ def get_kernelversion(p):
> >> > fn = p + '/include/generated/utsrelease.h'
> >> > if not os.path.isfile(fn):
> >> > fn = p + '/include/linux/version.h'
> >> > + if not os.path.isfile(fn):
> >> > + # after 3.7-rc1
> >> > + fn = p + '/include/generated/uapi/linux/version.h'
> >> >
> >> > import re
> >> > try:
> >> > --
> >> > 1.7.5.4
> >> >
> >> >
> >> > _______________________________________________
> >> > Openembedded-core mailing list
> >> > Openembedded-core@lists.openembedded.org
> >> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
> >>
> >>
> >>
> >> --
> >> "Thou shalt not follow the NULL pointer, for chaos and madness await
> >> thee at its end"
> >
> >
>
>
>
> --
> "Thou shalt not follow the NULL pointer, for chaos and madness await
> thee at its end"
>
[-- Attachment #2: Type: text/html, Size: 5598 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] linux-kernel-base: linux/version.h path update for 3.7-rc1
2013-03-27 9:25 ` Anders Roxell
@ 2013-03-27 12:42 ` Bruce Ashfield
0 siblings, 0 replies; 6+ messages in thread
From: Bruce Ashfield @ 2013-03-27 12:42 UTC (permalink / raw)
To: Anders Roxell
Cc: Anders Roxell, Patches and discussions about the oe-core layer
On Wed, Mar 27, 2013 at 5:25 AM, Anders Roxell <anders.roxell@gmail.com> wrote:
>
> On Wed, Mar 20, 2013 at 4:30 PM, Bruce Ashfield <bruce.ashfield@gmail.com>
> wrote:
>>
>> On Wed, Mar 20, 2013 at 10:55 AM, Anders Roxell <anders.roxell@gmail.com>
>> wrote:
>> >
>> >
>> >
>> > On Wed, Mar 20, 2013 at 2:42 PM, Bruce Ashfield
>> > <bruce.ashfield@gmail.com>
>> > wrote:
>> >>
>> >> On Wed, Mar 20, 2013 at 9:22 AM, <anders.roxell@gmail.com> wrote:
>> >> > From: Anders Roxell <anders.roxell@enea.com>
>> >> >
>> >> > In mainline kernel 3.7-rc1 (hash:abbf1590de22a6d22)
>> >> > The path include/linux/version.h was updated to
>> >> > include/generated/uapi/linux/version.h
>> >>
>> >> It's true that uapi now holds the generated version.h, but what build
>> >> error were you seeing without this change in place ?
>> >
>> >
>> > We were trying to build linx
>> > (http://linx.sourceforge.net/linxdoc/doc/index.html)
>> > Please let me know if it is better to rewrite linx or include this
>> > patch?
>>
>> My concern is that as a list of alternatives builds up over time, that
>> don't
>> necessarily use the same format of the linux version, you'll end up with
>> applications that are making the wrong decisions if a fallback is used.
>> Having
>> a clear public / exported file, and throwing errors if it doesn't exist
>> would
>> ensure consistent behaviour (granted annoying if you get the error when
>> you
>> think you shouldn't).
>>
>> If you look at the code in get_kernelversion() it shouldn't even work if
>> version.h is used versus utsrelease.h. I'd be inclined to suggest the
>> removal of the existing version.h fallback, versus adding another.
>>
>> You might not even need to modify the application code to fix this. What
>> are the dependencies of your recipe that would allow for utsrelease.h
>> to not be generated when the build calls get_kernelversion ?
>>
>
> Sorry for the late response!
> I agree with you that its better to use utsrelease.h rather than depending
> on version.h.
>
> We have added a patch for the linx source to solve this.
>
> Thank you very much for the help and support!
I was wondering about how this went a day or so ago .. and now I know.
Glad to hear it worked out!
Cheers,
Bruce
>
> With kind regards,
> Anders
>
>>
>> Cheers,
>>
>> Bruce
>>
>> >
>> > Regards,
>> > Anders
>> >
>> >>
>> >> utsrelease.h is
>> >> still created where it has been for some time, and should be what is
>> >> typically used for any version checks.
>> >>
>> >> Cheers,
>> >>
>> >> Bruce
>> >>
>> >>
>> >>
>> >> >
>> >> > Signed-off-by: Anders Roxell <anders.roxell@enea.com>
>> >> > ---
>> >> > meta/classes/linux-kernel-base.bbclass | 3 +++
>> >> > 1 files changed, 3 insertions(+), 0 deletions(-)
>> >> >
>> >> > diff --git a/meta/classes/linux-kernel-base.bbclass
>> >> > b/meta/classes/linux-kernel-base.bbclass
>> >> > index 4f2b0a4..b7d0ffa 100644
>> >> > --- a/meta/classes/linux-kernel-base.bbclass
>> >> > +++ b/meta/classes/linux-kernel-base.bbclass
>> >> > @@ -8,6 +8,9 @@ def get_kernelversion(p):
>> >> > fn = p + '/include/generated/utsrelease.h'
>> >> > if not os.path.isfile(fn):
>> >> > fn = p + '/include/linux/version.h'
>> >> > + if not os.path.isfile(fn):
>> >> > + # after 3.7-rc1
>> >> > + fn = p + '/include/generated/uapi/linux/version.h'
>> >> >
>> >> > import re
>> >> > try:
>> >> > --
>> >> > 1.7.5.4
>> >> >
>> >> >
>> >> > _______________________________________________
>> >> > Openembedded-core mailing list
>> >> > Openembedded-core@lists.openembedded.org
>> >> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>> >>
>> >>
>> >>
>> >> --
>> >> "Thou shalt not follow the NULL pointer, for chaos and madness await
>> >> thee at its end"
>> >
>> >
>>
>>
>>
>> --
>> "Thou shalt not follow the NULL pointer, for chaos and madness await
>> thee at its end"
>
>
--
"Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end"
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-03-27 12:59 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-20 13:22 [PATCH] linux-kernel-base: linux/version.h path update for 3.7-rc1 anders.roxell
2013-03-20 13:42 ` Bruce Ashfield
2013-03-20 14:55 ` Anders Roxell
2013-03-20 15:30 ` Bruce Ashfield
2013-03-27 9:25 ` Anders Roxell
2013-03-27 12:42 ` Bruce Ashfield
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.