diff options
author | Sam Ravnborg <sam@ravnborg.org> | 2008-06-30 17:02:59 -0400 |
---|---|---|
committer | Sam Ravnborg <sam@ravnborg.org> | 2008-07-25 16:12:51 -0400 |
commit | 09748e178b6cb8d3b8a748d0159aa7ad8eadcbe1 (patch) | |
tree | 8e68a1922eb8affc95568d7c56b5442136515d31 | |
parent | cd9140e1e73a31fd45f1fd4585260643a2f9ab1d (diff) |
kconfig: make defconfig is no longer chatty
make defconfig generated a lot of output
then noone actually read.
Use conf_set_all_new_symbols() to generate the default
configuration and avoid the chatty output.
A typical run now looks like this:
$ make defconfig
*** Default configuration is based on 'i386_defconfig'
arch/x86/configs/i386_defconfig:13:warning: trying to assign nonexistent symbol SEMAPHORE_SLEEPERS
arch/x86/configs/i386_defconfig:176:warning: trying to assign nonexistent symbol PREEMPT_BKL
...
arch/x86/configs/i386_defconfig:1386:warning: trying to assign nonexistent symbol INSTRUMENTATION
$
As an added benefit we now clearly see the warnings generated
in the start of the process.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Roman Zippel <zippel@linux-m68k.org>
-rw-r--r-- | scripts/kconfig/conf.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c index 6cdaa0cc572f..9fba838c7069 100644 --- a/scripts/kconfig/conf.c +++ b/scripts/kconfig/conf.c | |||
@@ -102,9 +102,6 @@ static int conf_askvalue(struct symbol *sym, const char *def) | |||
102 | fflush(stdout); | 102 | fflush(stdout); |
103 | fgets(line, 128, stdin); | 103 | fgets(line, 128, stdin); |
104 | return 1; | 104 | return 1; |
105 | case set_default: | ||
106 | printf("%s\n", def); | ||
107 | return 1; | ||
108 | default: | 105 | default: |
109 | break; | 106 | break; |
110 | } | 107 | } |
@@ -318,10 +315,6 @@ static int conf_choice(struct menu *menu) | |||
318 | else | 315 | else |
319 | continue; | 316 | continue; |
320 | break; | 317 | break; |
321 | case set_default: | ||
322 | cnt = def; | ||
323 | printf("%d\n", cnt); | ||
324 | break; | ||
325 | default: | 318 | default: |
326 | break; | 319 | break; |
327 | } | 320 | } |
@@ -589,13 +582,15 @@ int main(int ac, char **av) | |||
589 | case set_random: | 582 | case set_random: |
590 | conf_set_all_new_symbols(def_random); | 583 | conf_set_all_new_symbols(def_random); |
591 | break; | 584 | break; |
585 | case set_default: | ||
586 | conf_set_all_new_symbols(def_default); | ||
587 | break; | ||
592 | case ask_silent: | 588 | case ask_silent: |
593 | case ask_new: | 589 | case ask_new: |
594 | if (conf_silent_update()) | 590 | if (conf_silent_update()) |
595 | exit(1); | 591 | exit(1); |
596 | break; | 592 | break; |
597 | case ask_all: | 593 | case ask_all: |
598 | case set_default: | ||
599 | if (conf_update()) | 594 | if (conf_update()) |
600 | exit(1); | 595 | exit(1); |
601 | break; | 596 | break; |