All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.