On Wed, Jul 30, 2008 at 10:26 PM, Sam Ravnborg <sam@xxxxxxxxxxxx> wrote: > On Sat, Jul 26, 2008 at 11:34:14AM +0200, John Kacur wrote: >> On Wed, Jul 16, 2008 at 12:31 AM, jkacur <jkacur@xxxxxxxxx> wrote: >> > Change the Kbuild "If unsure" message to match the default. >> > >> > Signed-off-by: John Kacur <jkacur at gmail dot com> >> > >> > Index: linux-2.6.26/init/Kconfig >> > =================================================================== >> > --- linux-2.6.26.orig/init/Kconfig >> > +++ linux-2.6.26/init/Kconfig >> > @@ -522,7 +522,7 @@ config CC_OPTIMIZE_FOR_SIZE >> > Enabling this option will pass "-Os" instead of "-O2" to gcc >> > resulting in a smaller kernel. >> > >> > - If unsure, say N. >> > + If unsure, say Y. >> > >> > config SYSCTL >> > bool >> > >> >> This seems like a no-brainer to me that the default and the "If >> unsure" message should match, any reason you haven't picked-it up? > > Two reasons: > > 1) I have not started to look at new patches until now > > And this is due to several reasons. > First off I have been without internet for a while. > And secondly I do try to avoid new patches during the > merge window which does not fix merge issues. Ok, no problem! > > 2) I am not convinced that "default y" is really what we want. > What does popular distroes use (aka what gets wide testing?). > Hmnn, good point, in any case there is an inconsistency here. I can only speak for my laptop here running openSUSE 11.0 (X86-64) gunzip -c /proc/config.gz | grep CC_OPTIMIZE_FOR_SIZE CONFIG_CC_OPTIMIZE_FOR_SIZE=y uname -a Linux linux-ipxk 2.6.25.11-0.1-default #1 SMP 2008-07-13 20:48:28 +0200 x86_64 x86_64 x86_64 GNU/Linux -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html