aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSergey Ryazanov <ryazanov.s.a@gmail.com>2017-05-07 20:19:13 +0300
committerJo-Philipp Wich <jo@mein.io>2017-05-11 00:53:05 +0200
commite06d8f0f6f041c8ac1acba810eea96c32dc983e4 (patch)
tree6615c670582e18a8a8f655b10683b599b2806dbb
parent44da45a881a7aa13a237c5c8843dca914d40d7ae (diff)
downloadupstream-e06d8f0f6f041c8ac1acba810eea96c32dc983e4.tar.gz
upstream-e06d8f0f6f041c8ac1acba810eea96c32dc983e4.tar.bz2
upstream-e06d8f0f6f041c8ac1acba810eea96c32dc983e4.zip
build: new fixes for symlinked .config handling
When running "make {config|defconfig|oldconfig}" with symlinked .config (e.g. to env/.config) it renames symlink to .config.old, creates new .config file, and writes the updated configuration into it. This breaks the desired workflow when changes in the configuration can be checked using "scripts/env diff" and commited using "scripts/env save". Since the env/.config file is not updated. The things become even worse when working with feeds, since feeds script quite often silently invokes "make {oldconfig|defconfig}" and breaks the symlink. 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. This change uses the same behaviour as "make menucofig", which has already been fixed in commit 5bf98b1acc3b6b178f8954c5075a58e1e6a99d6a. Also make a tiny cosmetic update to the "make menuconfig" target code layout to make it look like other config handling targets. Signed-off-by: Sergey Ryazanov <ryazanov.s.a@gmail.com>
-rw-r--r--include/toplevel.mk12
1 files changed, 8 insertions, 4 deletions
diff --git a/include/toplevel.mk b/include/toplevel.mk
index d8ea6cff60..09aec791ba 100644
--- a/include/toplevel.mk
+++ b/include/toplevel.mk
@@ -107,7 +107,8 @@ scripts/config/conf:
@$(_SINGLE)$(SUBMAKE) -s -C scripts/config conf CC="$(HOSTCC_WRAPPER)"
config: scripts/config/conf prepare-tmpinfo FORCE
- $< Config.in
+ [ -L .config ] && export KCONFIG_OVERWRITECONFIG=1; \
+ $< Config.in
config-clean: FORCE
$(_SINGLE)$(NO_TRACE_MAKE) -C scripts/config clean
@@ -115,7 +116,8 @@ config-clean: FORCE
defconfig: scripts/config/conf prepare-tmpinfo FORCE
touch .config
@if [ ! -s .config -a -e $(HOME)/.openwrt/defconfig ]; then cp $(HOME)/.openwrt/defconfig .config; fi
- $< --defconfig=.config Config.in
+ [ -L .config ] && export KCONFIG_OVERWRITECONFIG=1; \
+ $< --defconfig=.config Config.in
confdefault-y=allyes
confdefault-m=allmod
@@ -123,13 +125,15 @@ confdefault-n=allno
confdefault:=$(confdefault-$(CONFDEFAULT))
oldconfig: scripts/config/conf prepare-tmpinfo FORCE
- $< --$(if $(confdefault),$(confdefault),old)config Config.in
+ [ -L .config ] && export KCONFIG_OVERWRITECONFIG=1; \
+ $< --$(if $(confdefault),$(confdefault),old)config Config.in
menuconfig: scripts/config/mconf prepare-tmpinfo FORCE
if [ \! -e .config -a -e $(HOME)/.openwrt/defconfig ]; then \
cp $(HOME)/.openwrt/defconfig .config; \
fi
- [ -L .config ] && export KCONFIG_OVERWRITECONFIG=1; $< Config.in
+ [ -L .config ] && export KCONFIG_OVERWRITECONFIG=1; \
+ $< Config.in
prepare_kernel_conf: .config FORCE