Re: [PATCH] busybox: import the configuration from openembedded.dev

[Phil Blundell <philb@gnu.org>]

On Mon, 2011-09-05 at 17:51 +0200, Denis Carikli wrote:
> Without that patch many applets are not build, like for instance
>   mdev, which could be necessary for building certain images.

If your distro needs mdev then you should probably supply your own
busybox config which turns it on.  For modern kernels mdev is probably
redundant since you will usually be better off with devtmpfs.

Or, conversely, if you feel that a particular change is important enough
that it really needs to go into the global busybox config for oe-core,
please send a patch which makes just that one change.  The patch you
sent flips a large set of of seemingly-random switches, for example:

@@ -85,14 +82,12 @@ CONFIG_FEATURE_FAST_TOP=y
 CONFIG_FEATURE_EDITING=y
 CONFIG_FEATURE_EDITING_MAX_LEN=1024
 # CONFIG_FEATURE_EDITING_VI is not set
-CONFIG_FEATURE_EDITING_HISTORY=15
+CONFIG_FEATURE_EDITING_HISTORY=64
 CONFIG_FEATURE_EDITING_SAVEHISTORY=y
 CONFIG_FEATURE_TAB_COMPLETION=y
 CONFIG_FEATURE_USERNAME_COMPLETION=y
 CONFIG_FEATURE_EDITING_FANCY_PROMPT=y
-# CONFIG_FEATURE_EDITING_ASK_TERMINAL is not set
-CONFIG_FEATURE_NON_POSIX_CP=y
-# CONFIG_FEATURE_VERBOSE_CP_MESSAGE is not set
+CONFIG_FEATURE_VERBOSE_CP_MESSAGE=y
 CONFIG_FEATURE_COPYBUF_KB=4
 CONFIG_MONOTONIC_SYSCALL=y
 CONFIG_IOCTL_HEX2STR_ERROR=y
@@ -105,7 +100,7 @@ CONFIG_FEATURE_HWIB=y
 #
 # Archival Utilities
 #
-CONFIG_FEATURE_SEAMLESS_LZMA=y
+# CONFIG_FEATURE_SEAMLESS_LZMA is not set
 CONFIG_FEATURE_SEAMLESS_BZ2=y
 CONFIG_FEATURE_SEAMLESS_GZ=y
 CONFIG_FEATURE_SEAMLESS_Z=y

... which is impossible to review sensibly, and it isn't at all obvious
why this new configuration would necessarily be an improvement.

p.