All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 0/13 v2] Add dependency on kernel headers
Date: Tue, 25 Feb 2014 18:24:37 +0100	[thread overview]
Message-ID: <530CD1D5.1010301@mind.be> (raw)
In-Reply-To: <cover.1393198603.git.yann.morin.1998@free.fr>

 Hi Yann,

On 24/02/14 00:40, Yann E. MORIN wrote:
> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> 
> Hello All!
> 
> This series introduces infrastructure for packages to depend on the
> required version of kernel headers.
> 
> It follows the proposal made on the list:
>     http://lists.busybox.net/pipermail/buildroot/2014-February/088924.html
> 
> The basic idea is to add a symbol for each kernel header version, and
> have toolchains select the appropriate symbol, and packages depend on the
> appropriate symbol.

 With all the additional symbols that are introduced by this series, some
of which the user has to select from a huge choice (at least, it will be
huge when we reach 3.45 five years from now), I wonder if it wouldn't be
easier to add support for int comparisons in Kconfig expressions. I have
taken a quick look and it doesn't seem impossible at first sight...


 Regards,
 Arnout

> 
> In this series:
> 
>   - such symbols are added for all kernel headers >= 3.0;
> 
>   - the internal toochain backend selects the appropriate symbol, without
>     user-visible changes for known kernel headers versions, and with a new
>     choice for manually-specified kernel headers version;
> 
>   - the predefined external toolchains select the appropriate symbols
>     without user-visible changes; those using kernel headers older than
>     3.0, select nothing, and a comment is added to state so;
> 
>   - for the custom external toolchain, a new choice is shown to the
>     user, to select the appropriate kernel headers version;
> 
>   - five packages are converted to take advantage of this.
> 
> For example:
> 
>   - BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R1 selects
>     BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_3
> 
>   - dvb-apps depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_3
> 
> 
> Changes v1 -> v2:
>   - typo and asciidoc syntax in the manual  (Samuel, Thomas DS)
>   - add choice for custom kernel headers  (Arnout)
>   - add check to strictly enforce selection  (Thomas, Arnout)
>   - add help to choice  (Thomas DS)
> 
> 
> Regards,
> Yann E. MORIN.
> 
> 
> ----------------------------------------------------------------
> Yann E. MORIN (13):
>       package/linux-headers: remove 2.6 snapshot
>       toolchain/common: add minimum kernel headers options
>       manual: document minimal kernel headers dependency
>       toolchain/external/predefined: add choice to select 'AT_LEAST_XXX' header version
>       toolchain/external/custom: add choice to select 'AT_LEAST_XXX' header version
>       package/linux-headers: select appropriate 'AT_LEAST_XXX' header version
>       toolchain/external: check kernel headers version for custom toolchain
>       toolchain/internal: check kernel headers version for manually-specified version
>       package/dvb-apps: requires kernel headers >= 3.3
>       package/w_scan: requires kernel headers >= 3.0
>       package/weston: requires kernel headers >= 3.0
>       package/tvheadend: requires kernel headers >= 3.2
>       package/mmc-utils: requires kernel headers >= 3.0
> 
>  docs/manual/adding-packages-directory.txt          |   6 ++
>  package/dvb-apps/Config.in                         |   5 +-
>  package/linux-headers/Config.in.host               |  86 +++++++++++++++-
>  package/linux-headers/linux-headers.mk             |   9 ++
>  package/mmc-utils/Config.in                        |   5 +-
>  package/tvheadend/Config.in                        |   5 +-
>  package/w_scan/Config.in                           |   4 +
>  package/weston/Config.in                           |   5 +-
>  support/scripts/check-kernel-headers.sh            |  22 ++++
>  toolchain/helpers.mk                               |  13 +++
>  toolchain/toolchain-common.in                      |  79 +++++++++++++++
>  toolchain/toolchain-external/Config.in             | 111 +++++++++++++++++++++
>  toolchain/toolchain-external/toolchain-external.mk |   7 +-
>  13 files changed, 344 insertions(+), 13 deletions(-)
>  create mode 100755 support/scripts/check-kernel-headers.sh
> 


-- 
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

      parent reply	other threads:[~2014-02-25 17:24 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-23 23:40 [Buildroot] [PATCH 0/13 v2] Add dependency on kernel headers Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 01/13] package/linux-headers: remove 2.6 snapshot Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 02/13] toolchain/common: add minimum kernel headers options Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 03/13] manual: document minimal kernel headers dependency Yann E. MORIN
2014-02-24 11:38   ` Thomas De Schampheleire
2014-02-24 19:18   ` Arnout Vandecappelle
2014-02-23 23:40 ` [Buildroot] [PATCH 04/13] toolchain/external/predefined: add choice to select 'AT_LEAST_XXX' header version Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 05/13] toolchain/external/custom: " Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 06/13] package/linux-headers: select appropriate " Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 07/13] toolchain/external: check kernel headers version for custom toolchain Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 08/13] toolchain/internal: check kernel headers version for manually-specified version Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 09/13] package/dvb-apps: requires kernel headers >= 3.3 Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 10/13] package/w_scan: requires kernel headers >= 3.0 Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 11/13] package/weston: " Yann E. MORIN
2014-02-23 23:40 ` [Buildroot] [PATCH 12/13] package/tvheadend: requires kernel headers >= 3.2 Yann E. MORIN
2014-02-23 23:41 ` [Buildroot] [PATCH 13/13] package/mmc-utils: requires kernel headers >= 3.0 Yann E. MORIN
2014-02-25 17:24 ` Arnout Vandecappelle [this message]

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=530CD1D5.1010301@mind.be \
    --to=arnout@mind.be \
    --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.