From 5d693b2e66451aedfd694ffde4dba7d4b74afa46 Mon Sep 17 00:00:00 2001
From: Ulf Magnusson <ulfalizer@gmail.com>
Date: Fri, 5 Jan 2018 20:37:00 +0100
Subject: Don't write out 'option env' symbols to C header

Oversight. SYMBOL_AUTO (env_var) being set indirectly clears
SYMBOL_WRITE (_write_to_conf) in sym_calc_value(). The .config case was
already fine due to an explicit env_var check.

Even non-visible env. symbols ended up in the header, due to
'option env' internally adding a default.

Disallow user values altogether on 'option env' symbols, even if
specified manually. This matches the C implementation. Add a warning
too.
---
 tests/Kmisc | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

(limited to 'tests/Kmisc')

diff --git a/tests/Kmisc b/tests/Kmisc
index 44fdfdc..9da57a4 100644
--- a/tests/Kmisc
+++ b/tests/Kmisc
@@ -61,12 +61,18 @@ menu "menu"
 endmenu
 
 config FROM_ENV
-    string
+    string "from env"
     option env="ENV_VAR"
 
 config FROM_ENV_MISSING
-    string
+    string "from env missing"
     option env="MISSING_ENV_VAR"
+    default "missing"
+
+config FROM_ENV_WEIRD
+    string
+    default "weird"
+    option env="ENV_VAR"
 
 config NOT_ALLNOCONFIG_Y
     bool "not allnoconfig_y"
-- 
cgit v1.2.3