All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v4] linux-headers: allow use of headers from kernel "package" selected
Date: Mon, 18 Jan 2016 23:21:45 +0100	[thread overview]
Message-ID: <20160118232145.32bfbd40@free-electrons.com> (raw)
In-Reply-To: <20160118221343.GD3380@free.fr>

Yann,

On Mon, 18 Jan 2016 23:13:43 +0100, Yann E. MORIN wrote:

> IIRC, I think I already suggested that we set the version string from
> Kconfig, a bit like:
> 
>     choice
>         bool "headers version"
> 
>     [...]
> 
>     config BR2_PKG_HEADERS_AS_KERNEL
>         bool "like kernel"
>         depends on BR2_LINUX_KERNEL
> 
>     comment "blabla needs a kernel to be built"
>         depends on !BR2_LINUX_KERNEL
> 
>     [...]
> 
>     endchoice
> 
>     config BR2_DEFAULT_KERNEL_HEADERS
>         string
>         default BR2_LINUX_KERNEL_VERSION if BR2_PKG_HEADERS_AS_KERNEL

This is not sufficient. If you select "custom tarball", or "local",
then BR2_LINUX_KERNEL_VERSION just contains "custom". Or it will
contain just the Git tag/commit id, or just the Mercurial version. You
still lack the repo/tarball URL/local directory.

But if you have a proposal based on that, that takes into account those
issues, don't hesitate to submit a patch.

Alternatively, what we could do is simply duplicate some code from
linux.mk into linux-headers.mk and therefore have linux-header.mk
directly poke at the linux/Config.in variables. Not nice, but it solves
a long-standing request, and we don't change that often the Linux
download/patch logic.

I was trying to think of getting rid of the linux-headers package and
have the linux package provide the headers for the toolchain, but I
don't see a really feasible way of doing this.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2016-01-18 22:21 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-01 17:17 [Buildroot] [PATCH v4] linux-headers: allow use of headers from kernel "package" selected Karoly Kasza
2015-03-04 22:37 ` Thomas Petazzoni
2015-03-12 14:07   ` Алексей Бродкин
2016-01-09 17:29 ` Yann E. MORIN
2016-01-18  9:21   ` Peter Korsgaard
2016-01-18 21:47     ` Yann E. MORIN
2016-01-18 21:50       ` Peter Korsgaard
2016-01-18 22:03       ` Thomas Petazzoni
2016-01-18 22:13         ` Yann E. MORIN
2016-01-18 22:21           ` Thomas Petazzoni [this message]
2016-01-18 22:21         ` Peter Korsgaard
2016-01-18 22:24           ` Thomas Petazzoni
2016-01-19  8:21             ` Peter Korsgaard
2016-01-19 10:18               ` Thomas Petazzoni
2016-01-19 10:29                 ` Peter Korsgaard
2016-01-19 10:31                   ` Thomas Petazzoni
2016-01-19 10:37                     ` Peter Korsgaard
2016-01-19 17:22                       ` Yann E. MORIN
2016-01-18 17:42   ` Arnout Vandecappelle
2016-01-19  7:25     ` Алексей Бродкин
2016-01-19 20:10 ` Yann E. MORIN

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160118232145.32bfbd40@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.