From: Konrad Eisele <konrad@gaisler.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 0/2] intro
Date: Fri, 18 Nov 2011 11:46:33 +0100 [thread overview]
Message-ID: <4EC63789.5040704@gaisler.com> (raw)
In-Reply-To: <CAAXf6LXj3OgU-3w4H9UYfjor_sm+W0gkBs-T8KGxfBrfjoqsxg@mail.gmail.com>
Thomas De Schampheleire wrote:
> On Fri, Nov 18, 2011 at 12:36 AM, Arnout Vandecappelle <arnout@mind.be> wrote:
>> Hoi,
>>
>> I'm not very enthousiastic about either of these two features.
>>
>> On Thursday 17 November 2011 14:18:07 Konrad Eisele wrote:
>>> The aim of patch-1 is to make it possible to have configuration subtrees.
>>> This makes it possible to have a structure like this:
>>>
>>> buildroot-kconfigs
>>> + linux-kconfigs
>>> + busybox-kconfigs
>>> + uclibc-kconfigs
>>> + crosstools-kconfigs
>>>
>>> Where all configuration appear in one xconfig screen. Currently I have focues on
>>> qconfig only, I think however adding support for gconfig and mconfig is possible
>>> easily. The subtree feature is enabled with the -s option to qconfig: "qconfig -s <kconf>"
>>
>> - The subtree that has to be included depends on your buildroot configuration.
>> So you have to include all possible linux, busybox, uclibc, ... configs and
>> protect them with IFs. I can hardly imagine that Kconfig can deal with such
>> huge configurations.
>>
>> - I don't like the size explosion of the buildroot tree that we would see
>> if all these configs are included.
>>
>> - The packages which have kconfigs are the ones that are most likely to need
>> board-specific modifications, which may define additional config options. This
>> means that copying the config tree into buildroot isn't going to cut it.
>>
>> - Running configs for these things is a bit of an expert step. In particular
>> because the configs have to be post-processed by buildroot and because
>> you have to save them explicitly afterwards in a place different from the
>> output directory. I think that part should be smoothed out first. Until
>> then, I consider it a good thing that the normal user runs 'make xconfig'
>> while the expert user runs 'make {,linux-,busybox-}menuconfig'.
>>
>> - I don't know what it will look like visually because the patch failed to
>> compile for me (current_conf_level is undefined), but I wonder if there is a
>> significant advantage compared to just menus. At least in menuconfig
>> it wouldn't really make a difference.
>>
>>
>>> The other feature that patch-1 adds is a config-entry type "execute: It is
>>> like a string, however when doubleclicking (trying to edit) in qconfig
>>> (only in qconfig currently) then the string is executed using "system(<str>)".
>>> The goal is to be able to execute "make" from inside the gui, without having
>>> to exit.
>>
>> Here I simply don't see the benefit. Whatever needs to be executed there
>> can just be done with the normal make after the config finishes. If people
>> want to push a button to run make, give them Eclipse with a buildroot
>> plugin :-)
>>
>
> In fact, similar features (e.g. including one menuconfig into another)
> were discussed in the context of crosstool-ng integration into
> buildroot. See the thread "Report from the Buildroot Developer Day"
> and Yann's comments on it.
>
I read the thread and:
>There are two ways to do a better integration:
>
>1) Include the crosstool-NG menuconfig entries into the buildroot
> menuconfig. This will be difficule for two reasons:
> - namespace clashing [1]
with "subsource" there is not namespace clashing.
> - maintenance when a newer crosstool-NG version gets integrated
The newest crosstools kconfig gets added always. As written in a previous mal
you can make a "subsource" to the kconfig _after_ download and install. It not
present the "subsource" is ignored.
>
> Best regards,
> Thomas
>
>
prev parent reply other threads:[~2011-11-18 10:46 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-17 14:18 [Buildroot] [PATCH 0/2] intro Konrad Eisele
2011-11-17 14:18 ` [Buildroot] [PATCH 1/2] Add a configuration subtree and an execution command to kconfig: Konrad Eisele
2011-11-17 14:18 ` [Buildroot] [PATCH 2/2] Example of how to define a subtree using kconfig tag "subsource": package/busybox/busybox-1.19.x.in is a flattend version of busybox-1.19.3's kconfig scripts (all-in-one). package/busybox/Config.in adds this Kconfig as a subtree to the main configuration tree: subsource "package/busybox/busybox-1.19.x.in" "package/busybox/" "package/busybox/busybox-1.19.x.config" "Busybox 1.19.x configuration" BUSYBOX_ CONFIG_ package/busybox/busybox-1.19.x.config is used as the .config for this subtree. config BR2_BUSYBOX_VERSION_1_19_X has to be selected for the subtree to show up Konrad Eisele
2011-11-17 14:20 ` [Buildroot] [PATCH 1/2] Add a configuration subtree and an execution command to kconfig: Konrad Eisele
2011-11-17 20:09 ` Thomas De Schampheleire
2011-11-18 7:42 ` [Buildroot] [PATCH 1/1] kconfig: Add a configuration subtree command to kconfig Konrad Eisele
2011-11-18 7:49 ` Konrad Eisele
2011-11-18 9:04 ` [Buildroot] [PATCH 1/1] kconfig: Add "execute" config-type Konrad Eisele
2011-11-17 23:36 ` [Buildroot] [PATCH 0/2] intro Arnout Vandecappelle
2011-11-18 9:30 ` Konrad Eisele
2011-11-18 9:38 ` Thomas De Schampheleire
2011-11-18 10:39 ` Konrad Eisele
2011-11-18 10:46 ` Konrad Eisele [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=4EC63789.5040704@gaisler.com \
--to=konrad@gaisler.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox