On Thu, Jul 29, 2010 at 11:17:46AM +0200, Michal Marek wrote: > On 28.7.2010 22:36, Sam Ravnborg wrote: > > Hi Michal. > > > > I cooked up the following to introduce long options in conf. > > But in the process I dropped the short options. Is this OK? > > Nice! I would probably name the option and the enum value for randconfig > simply --randconfig to not introduce new names (the scripts/kconfig/conf > call is displayed to the user). I had in mind to deprecate randconfig in favour of allrandconfig to get more consitency in the names. But I failed to document this in the changelog. I can submit the following as a real patch if you agree. Sam diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile index 1290ac3..9670323 100644 --- a/scripts/kconfig/Makefile +++ b/scripts/kconfig/Makefile @@ -101,12 +101,12 @@ update-po-config: $(obj)/kxgettext $(obj)/gconf.glade.h $(Q)rm -f arch/um/Kconfig.arch $(Q)rm -f $(obj)/config.pot -PHONY += randconfig allyesconfig allnoconfig allmodconfig defconfig +PHONY += allnoconfig allyesconfig allmodconfig allrandconfig randconfig defconfig -randconfig: $(obj)/conf - $< --allrandconfig $(Kconfig) +# Support deprecated randconfig target +randconfig: allrandconfig -allnoconfig allyesconfig allmodconfig: $(obj)/conf +allnoconfig allyesconfig allmodconfig allrandconfig: $(obj)/conf $< --$@ $(Kconfig) defconfig: $(obj)/conf @@ -131,7 +131,7 @@ help: @echo ' localmodconfig - Update current config disabling modules not loaded' @echo ' localyesconfig - Update current config converting local mods to core' @echo ' silentoldconfig - Same as oldconfig, but quietly, additionally update deps' - @echo ' randconfig - New config with random answer to all options' + @echo ' allrandconfig - New config with random answer to all options' @echo ' defconfig - New config with default answer to all options' @echo ' allmodconfig - New config selecting modules when possible' @echo ' allyesconfig - New config where all options are accepted with yes' -- 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
