]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
kconfig: fix file name in warnings when loading KCONFIG_DEFCONFIG_LIST
authorMasahiro Yamada <masahiroy@kernel.org>
Mon, 20 Jan 2025 07:59:14 +0000 (16:59 +0900)
committerMasahiro Yamada <masahiroy@kernel.org>
Wed, 29 Jan 2025 23:40:17 +0000 (08:40 +0900)
Most 'make *config' commands use .config as the base configuration file.

When .config does not exist, Kconfig tries to load a file listed in
KCONFIG_DEFCONFIG_LIST instead.

However, since commit b75b0a819af9 ("kconfig: change defconfig_list
option to environment variable"), warning messages have displayed an
incorrect file name in such cases.

Below is a demonstration using Debian Trixie. While loading
/boot/config-6.12.9-amd64, the warning messages incorrectly show .config
as the file name.

With this commit, the correct file name is displayed in warnings.

[Before]

  $ rm -f .config
  $ make config
  #
  # using defaults found in /boot/config-6.12.9-amd64
  #
  .config:6804:warning: symbol value 'm' invalid for FB_BACKLIGHT
  .config:9895:warning: symbol value 'm' invalid for ANDROID_BINDER_IPC

[After]

  $ rm -f .config
  $ make config
  #
  # using defaults found in /boot/config-6.12.9-amd64
  #
  /boot/config-6.12.9-amd64:6804:warning: symbol value 'm' invalid for FB_BACKLIGHT
  /boot/config-6.12.9-amd64:9895:warning: symbol value 'm' invalid for ANDROID_BINDER_IPC

Fixes: b75b0a819af9 ("kconfig: change defconfig_list option to environment variable")
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
scripts/kconfig/confdata.c

index 4286d5e7f95dc14057d550049f047a099ca21535..3b55e7a4131d9a4fb87499617158dfe906f79164 100644 (file)
@@ -360,10 +360,12 @@ int conf_read_simple(const char *name, int def)
 
                        *p = '\0';
 
-                       in = zconf_fopen(env);
+                       name = env;
+
+                       in = zconf_fopen(name);
                        if (in) {
                                conf_message("using defaults found in %s",
-                                            env);
+                                            name);
                                goto load;
                        }