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 /scripts | |
| 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>
Diffstat (limited to 'scripts')
| -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; |
