Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/linux-headers: remove 2.6 snapshot
@ 2014-02-23 20:34 Yann E. MORIN
  2014-02-24  7:07 ` Arnout Vandecappelle
  0 siblings, 1 reply; 5+ messages in thread
From: Yann E. MORIN @ 2014-02-23 20:34 UTC (permalink / raw)
  To: buildroot

From: "Yann E. MORIN" <yann.morin.1998@free.fr>

As Thomas states:
    It is completely weird to ask the tarball to be named exactly
    linux-2.6.tar.bz2. IMO, we should simply get rid of this feature.

Make it so! ;-)

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/linux-headers/Config.in.host | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/package/linux-headers/Config.in.host b/package/linux-headers/Config.in.host
index 134e338..fcecd62 100644
--- a/package/linux-headers/Config.in.host
+++ b/package/linux-headers/Config.in.host
@@ -56,9 +56,6 @@ choice
 
 	config BR2_KERNEL_HEADERS_VERSION
 		bool "Manually specified Linux version"
-
-	config BR2_KERNEL_HEADERS_SNAP
-		bool "Local Linux snapshot (linux-2.6.tar.bz2)"
 endchoice
 
 config BR2_DEFAULT_KERNEL_VERSION
@@ -81,5 +78,4 @@ config BR2_DEFAULT_KERNEL_HEADERS
 	default "3.11.10"	if BR2_KERNEL_HEADERS_3_11
 	default "3.12.13"	if BR2_KERNEL_HEADERS_3_12
 	default "3.13.5"	if BR2_KERNEL_HEADERS_3_13
-	default "2.6"		if BR2_KERNEL_HEADERS_SNAP
 	default BR2_DEFAULT_KERNEL_VERSION if BR2_KERNEL_HEADERS_VERSION
-- 
1.8.3.2

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] package/linux-headers: remove 2.6 snapshot
  2014-02-23 20:34 [Buildroot] [PATCH] package/linux-headers: remove 2.6 snapshot Yann E. MORIN
@ 2014-02-24  7:07 ` Arnout Vandecappelle
  2014-02-24 17:19   ` Yann E. MORIN
  2014-02-24 17:59   ` Yann E. MORIN
  0 siblings, 2 replies; 5+ messages in thread
From: Arnout Vandecappelle @ 2014-02-24  7:07 UTC (permalink / raw)
  To: buildroot

On 02/23/14 21:34, Yann E. MORIN wrote:
> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> 
> As Thomas states:
>     It is completely weird to ask the tarball to be named exactly
>     linux-2.6.tar.bz2. IMO, we should simply get rid of this feature.
> 
> Make it so! ;-)
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  package/linux-headers/Config.in.host | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/package/linux-headers/Config.in.host b/package/linux-headers/Config.in.host
> index 134e338..fcecd62 100644
> --- a/package/linux-headers/Config.in.host
> +++ b/package/linux-headers/Config.in.host
> @@ -56,9 +56,6 @@ choice
>  
>  	config BR2_KERNEL_HEADERS_VERSION
>  		bool "Manually specified Linux version"
> -
> -	config BR2_KERNEL_HEADERS_SNAP
> -		bool "Local Linux snapshot (linux-2.6.tar.bz2)"
>  endchoice
>  
>  config BR2_DEFAULT_KERNEL_VERSION
> @@ -81,5 +78,4 @@ config BR2_DEFAULT_KERNEL_HEADERS
>  	default "3.11.10"	if BR2_KERNEL_HEADERS_3_11
>  	default "3.12.13"	if BR2_KERNEL_HEADERS_3_12
>  	default "3.13.5"	if BR2_KERNEL_HEADERS_3_13
> -	default "2.6"		if BR2_KERNEL_HEADERS_SNAP
>  	default BR2_DEFAULT_KERNEL_VERSION if BR2_KERNEL_HEADERS_VERSION
> 

 This is missing Config.in.legacy support. The help text of the legacy
support should mention that the same can be achieved by using
BR2_LINUX_KERNEL_CUSTOM_TARBALL.

 Regards,
 Arnout

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] package/linux-headers: remove 2.6 snapshot
  2014-02-24  7:07 ` Arnout Vandecappelle
@ 2014-02-24 17:19   ` Yann E. MORIN
  2014-02-24 17:59   ` Yann E. MORIN
  1 sibling, 0 replies; 5+ messages in thread
From: Yann E. MORIN @ 2014-02-24 17:19 UTC (permalink / raw)
  To: buildroot

Arnout, All,

On 2014-02-24 08:07 +0100, Arnout Vandecappelle spake thusly:
> On 02/23/14 21:34, Yann E. MORIN wrote:
> > From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > 
> > As Thomas states:
> >     It is completely weird to ask the tarball to be named exactly
> >     linux-2.6.tar.bz2. IMO, we should simply get rid of this feature.
> > 
> > Make it so! ;-)
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > ---
> >  package/linux-headers/Config.in.host | 4 ----
> >  1 file changed, 4 deletions(-)
> > 
> > diff --git a/package/linux-headers/Config.in.host b/package/linux-headers/Config.in.host
> > index 134e338..fcecd62 100644
> > --- a/package/linux-headers/Config.in.host
> > +++ b/package/linux-headers/Config.in.host
> > @@ -56,9 +56,6 @@ choice
> >  
> >  	config BR2_KERNEL_HEADERS_VERSION
> >  		bool "Manually specified Linux version"
> > -
> > -	config BR2_KERNEL_HEADERS_SNAP
> > -		bool "Local Linux snapshot (linux-2.6.tar.bz2)"
> >  endchoice
> >  
> >  config BR2_DEFAULT_KERNEL_VERSION
> > @@ -81,5 +78,4 @@ config BR2_DEFAULT_KERNEL_HEADERS
> >  	default "3.11.10"	if BR2_KERNEL_HEADERS_3_11
> >  	default "3.12.13"	if BR2_KERNEL_HEADERS_3_12
> >  	default "3.13.5"	if BR2_KERNEL_HEADERS_3_13
> > -	default "2.6"		if BR2_KERNEL_HEADERS_SNAP
> >  	default BR2_DEFAULT_KERNEL_VERSION if BR2_KERNEL_HEADERS_VERSION
> > 
> 
>  This is missing Config.in.legacy support. The help text of the legacy
> support should mention that the same can be achieved by using
> BR2_LINUX_KERNEL_CUSTOM_TARBALL.

Right, I forgot that. Thanks for spotting it! :-)

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] package/linux-headers: remove 2.6 snapshot
  2014-02-24  7:07 ` Arnout Vandecappelle
  2014-02-24 17:19   ` Yann E. MORIN
@ 2014-02-24 17:59   ` Yann E. MORIN
  2014-02-24 20:28     ` Peter Korsgaard
  1 sibling, 1 reply; 5+ messages in thread
From: Yann E. MORIN @ 2014-02-24 17:59 UTC (permalink / raw)
  To: buildroot

Arnout, All,

On 2014-02-24 08:07 +0100, Arnout Vandecappelle spake thusly:
> On 02/23/14 21:34, Yann E. MORIN wrote:
> > From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > 
> > As Thomas states:
> >     It is completely weird to ask the tarball to be named exactly
> >     linux-2.6.tar.bz2. IMO, we should simply get rid of this feature.
> > 
> > Make it so! ;-)
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > ---
> >  package/linux-headers/Config.in.host | 4 ----
> >  1 file changed, 4 deletions(-)
> > 
> > diff --git a/package/linux-headers/Config.in.host b/package/linux-headers/Config.in.host
> > index 134e338..fcecd62 100644
> > --- a/package/linux-headers/Config.in.host
> > +++ b/package/linux-headers/Config.in.host
> > @@ -56,9 +56,6 @@ choice
> >  
> >  	config BR2_KERNEL_HEADERS_VERSION
> >  		bool "Manually specified Linux version"
> > -
> > -	config BR2_KERNEL_HEADERS_SNAP
> > -		bool "Local Linux snapshot (linux-2.6.tar.bz2)"
> >  endchoice
> >  
> >  config BR2_DEFAULT_KERNEL_VERSION
> > @@ -81,5 +78,4 @@ config BR2_DEFAULT_KERNEL_HEADERS
> >  	default "3.11.10"	if BR2_KERNEL_HEADERS_3_11
> >  	default "3.12.13"	if BR2_KERNEL_HEADERS_3_12
> >  	default "3.13.5"	if BR2_KERNEL_HEADERS_3_13
> > -	default "2.6"		if BR2_KERNEL_HEADERS_SNAP
> >  	default BR2_DEFAULT_KERNEL_VERSION if BR2_KERNEL_HEADERS_VERSION
> > 
> 
>  This is missing Config.in.legacy support. The help text of the legacy
> support should mention that the same can be achieved by using
> BR2_LINUX_KERNEL_CUSTOM_TARBALL.

In retrospect, BR2_LINUX_KERNEL_CUSTOM_TARBALL is for building the
kernel. It is not used to install the headers, which is what we are
doing here.

We still need to add a legacy option, though.

Regarrds,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] package/linux-headers: remove 2.6 snapshot
  2014-02-24 17:59   ` Yann E. MORIN
@ 2014-02-24 20:28     ` Peter Korsgaard
  0 siblings, 0 replies; 5+ messages in thread
From: Peter Korsgaard @ 2014-02-24 20:28 UTC (permalink / raw)
  To: buildroot

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 >> This is missing Config.in.legacy support. The help text of the legacy
 >> support should mention that the same can be achieved by using
 >> BR2_LINUX_KERNEL_CUSTOM_TARBALL.

 > In retrospect, BR2_LINUX_KERNEL_CUSTOM_TARBALL is for building the
 > kernel. It is not used to install the headers, which is what we are
 > doing here.

True. A custom version (but still mainline kernel) can be handled
through BR2_KERNEL_HEADERS_VERSION, but it isn't exactly the same as the
custom tarball.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-02-24 20:28 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-23 20:34 [Buildroot] [PATCH] package/linux-headers: remove 2.6 snapshot Yann E. MORIN
2014-02-24  7:07 ` Arnout Vandecappelle
2014-02-24 17:19   ` Yann E. MORIN
2014-02-24 17:59   ` Yann E. MORIN
2014-02-24 20:28     ` Peter Korsgaard

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