build: fix symlinked .config handling
authorSergey Ryazanov <ryazanov.s.a@gmail.com>
Thu, 20 Apr 2017 02:10:45 +0000 (05:10 +0300)
committerFelix Fietkau <nbd@nbd.name>
Wed, 26 Apr 2017 09:34:41 +0000 (11:34 +0200)
When running "make menuconfig" with symlinked .config (e.g. to
env/.config) it renames symlink to .config.old, creates new .config file
and writes updated configuration here.

This breaks the desired workflow when changes in the configuration could
be checked using "scripts/env diff" and commited with
"scripts/env save". Since the env/.config file is not updated.

Fix this issue by exporting KCONFIG_OVERWRITECONFIG=1, which forces
mconf to overwrite the .config content, instead of renaming it and
creating a new file. This variable is set only if .config is a symlink,
otherwise the variable is not exported and the old behaviour is
preserved.

Signed-off-by: Sergey Ryazanov <ryazanov.s.a@gmail.com>
include/toplevel.mk

index ceafffb3a7ba088e5fc5486e666c92bad9831142..4b63e8ab47a570c35d256ca011487f3d305b2799 100644 (file)
@@ -129,7 +129,7 @@ menuconfig: scripts/config/mconf prepare-tmpinfo FORCE
        if [ \! -e .config -a -e $(HOME)/.openwrt/defconfig ]; then \
                cp $(HOME)/.openwrt/defconfig .config; \
        fi
-       $< Config.in
+       [ -L .config ] && export KCONFIG_OVERWRITECONFIG=1; $< Config.in
 
 prepare_kernel_conf: .config FORCE