diff options
| author | Stefan Reinauer <reinauer@chromium.org> | 2015-08-20 11:19:34 -0700 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2015-09-30 06:58:57 +0000 |
| commit | 57a31317a26ff5f55638c4f3dd8d06b8a69b78f2 (patch) | |
| tree | c8ac4862f908bf51f2bab03b80f6c34e8e0874f9 /util/kconfig/mconf.c | |
| parent | 588ad7b5db26a88dad36e2dbecf5a4242d8410be (diff) | |
kconfig: Allow KCONFIG_STRICT outside of confdata.c
To catch dependency errors in symbol.c (such as the ones
fixed by I51b4ee326f082c6a656a813ee5772e9c34f5c343) we need
to check for global kconfig warnings before saving config
files.
This patch will produce errors for wrong dependencies and
add catching of errors to conf, nconf and mconf. Sorry,
gconf users, you will have to wait.
Change-Id: Idf7ee406ce3869941af319219aea16fab826df84
Signed-off-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Reviewed-on: http://review.coreboot.org/11291
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'util/kconfig/mconf.c')
| -rw-r--r-- | util/kconfig/mconf.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/util/kconfig/mconf.c b/util/kconfig/mconf.c index 1d651c1382..b4a08440af 100644 --- a/util/kconfig/mconf.c +++ b/util/kconfig/mconf.c @@ -952,6 +952,7 @@ static void conf_save(void) static int handle_exit(void) { int res; + char *env; save_and_exit = 1; reset_subtitle(); @@ -966,6 +967,13 @@ static int handle_exit(void) end_dialog(saved_x, saved_y); + env = getenv("KCONFIG_STRICT"); + if (env && *env && kconfig_warnings) { + fprintf(stderr, _("\n*** ERROR: %d warnings encountered, and " + "warnings are errors.\n\n"), kconfig_warnings); + res = 2; + } + switch (res) { case 0: if (conf_write(filename)) { |