* [Buildroot] [PATCH] configs: introduce a minimalist defconfig
@ 2017-12-24 16:11 Yann E. MORIN
2017-12-27 22:20 ` Thomas Petazzoni
0 siblings, 1 reply; 3+ messages in thread
From: Yann E. MORIN @ 2017-12-24 16:11 UTC (permalink / raw)
To: buildroot
It is very often interesting to start from a minimalist configuration
with no package and no filesystem.
Re-use the existing minimalist configuration and promote it to a full
defconfig.
This thus contains options that default to 'y', like the paranoid
wrapper and busybox-show-others, as the defconfig is used by test-pkg
too.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Arnout Vandecappelle <arnout@mind.be>
---
support/config-fragments/minimal.config => configs/minimal_defconfig | 2 ++
utils/genrandconfig | 2 +-
utils/test-pkg | 2 +-
3 files changed, 4 insertions(+), 2 deletions(-)
rename support/config-fragments/minimal.config => configs/minimal_defconfig (82%)
diff --git a/support/config-fragments/minimal.config b/configs/minimal_defconfig
similarity index 82%
rename from support/config-fragments/minimal.config
rename to configs/minimal_defconfig
index 71344e2c69..88406fdf4d 100644
--- a/support/config-fragments/minimal.config
+++ b/configs/minimal_defconfig
@@ -4,5 +4,7 @@ BR2_INIT_NONE=y
BR2_SYSTEM_BIN_SH_NONE=y
# BR2_PACKAGE_BUSYBOX is not set
# BR2_TARGET_ROOTFS_TAR is not set
+
+# For test-pkg, we also want those options to be forcibly 'y'
BR2_COMPILER_PARANOID_UNSAFE_PATH=y
BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y
diff --git a/utils/genrandconfig b/utils/genrandconfig
index 883322552c..d193b99e8b 100755
--- a/utils/genrandconfig
+++ b/utils/genrandconfig
@@ -343,7 +343,7 @@ def gen_config(args):
# Combine with the minimal configuration
minimalconfigfile = os.path.join(args.buildrootdir,
- 'support/config-fragments/minimal.config')
+ 'configs/minimal_defconfig')
with open(minimalconfigfile) as minimalf:
configlines += minimalf.readlines()
diff --git a/utils/test-pkg b/utils/test-pkg
index 1b7046eac4..0e8b2f973b 100755
--- a/utils/test-pkg
+++ b/utils/test-pkg
@@ -99,7 +99,7 @@ build_one() {
mkdir -p "${dir}"
support/kconfig/merge_config.sh -O "${dir}" \
- "${toolchainconfig}" "support/config-fragments/minimal.config" "${cfg}" \
+ "${toolchainconfig}" "configs/minimal_defconfig" "${cfg}" \
> /dev/null
# We want all the options from the snippet to be present as-is (set
# or not set) in the actual .config; if one of them is not, it means
--
2.11.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] configs: introduce a minimalist defconfig
2017-12-24 16:11 [Buildroot] [PATCH] configs: introduce a minimalist defconfig Yann E. MORIN
@ 2017-12-27 22:20 ` Thomas Petazzoni
2017-12-28 7:09 ` Peter Korsgaard
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Petazzoni @ 2017-12-27 22:20 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 24 Dec 2017 17:11:23 +0100, Yann E. MORIN wrote:
> It is very often interesting to start from a minimalist configuration
> with no package and no filesystem.
>
> Re-use the existing minimalist configuration and promote it to a full
> defconfig.
>
> This thus contains options that default to 'y', like the paranoid
> wrapper and busybox-show-others, as the defconfig is used by test-pkg
> too.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Arnout Vandecappelle <arnout@mind.be>
I'm not too sure of the value of this. Our defconfigs are meant to
allow generating a working Linux system for a specific HW platform.
This minimalist_defconfig would heavily deviate from this principle,
and could confuse newcomers who want to build a "minimal" system.
I'd say this minimalist configuration is more of an internal thing, for
advanced Buildroot developers, and we should keep it "hidden" where it
is today.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] configs: introduce a minimalist defconfig
2017-12-27 22:20 ` Thomas Petazzoni
@ 2017-12-28 7:09 ` Peter Korsgaard
0 siblings, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2017-12-28 7:09 UTC (permalink / raw)
To: buildroot
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:
> Hello,
> On Sun, 24 Dec 2017 17:11:23 +0100, Yann E. MORIN wrote:
>> It is very often interesting to start from a minimalist configuration
>> with no package and no filesystem.
>>
>> Re-use the existing minimalist configuration and promote it to a full
>> defconfig.
>>
>> This thus contains options that default to 'y', like the paranoid
>> wrapper and busybox-show-others, as the defconfig is used by test-pkg
>> too.
>>
>> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
>> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
>> Cc: Arnout Vandecappelle <arnout@mind.be>
> I'm not too sure of the value of this. Our defconfigs are meant to
> allow generating a working Linux system for a specific HW platform.
> This minimalist_defconfig would heavily deviate from this principle,
> and could confuse newcomers who want to build a "minimal" system.
> I'd say this minimalist configuration is more of an internal thing, for
> advanced Buildroot developers, and we should keep it "hidden" where it
> is today.
I agree. It is handy for doing quick builds when developing on
Buildroot, but I also don't see how useful it is for users.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-12-28 7:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-12-24 16:11 [Buildroot] [PATCH] configs: introduce a minimalist defconfig Yann E. MORIN
2017-12-27 22:20 ` Thomas Petazzoni
2017-12-28 7:09 ` Peter Korsgaard
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.