Hello Stephen,
Am 16.06.2016 um 20:59 schrieb Stephen Warren:
From: Stephen Warren <swar...@nvidia.com>
CONFIG_VERSION_VARIABLE isn't always defined, so we can't simply look up
its value directly, or an exception will occur if it isn't defined.
Instead, we must use .get() to supply a default value if the variable
isn't defined.
Fixes: da37f006e7c5 ("tests: py: disable main_signon check for printenv cmd")
Cc: Heiko Schocher <h...@denx.de>
Signed-off-by: Stephen Warren <swar...@nvidia.com>
---
test/py/tests/test_env.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Thanks!
Acked-by: Heiko Schocher <h...@denx.de>
diff --git a/test/py/tests/test_env.py b/test/py/tests/test_env.py
index 22a22d1d538b..035dbf5cac4c 100644
--- a/test/py/tests/test_env.py
+++ b/test/py/tests/test_env.py
@@ -39,7 +39,8 @@ class StateTestEnv(object):
Nothing.
"""
- if self.u_boot_console.config.buildconfig['config_version_variable']
== 'y':
+ if self.u_boot_console.config.buildconfig.get(
+ 'config_version_variable', 'n') == 'y':
with self.u_boot_console.disable_check('main_signon'):
response = self.u_boot_console.run_command('printenv')
else:
Hmm... may we should also move VERSION_VARIABLE to Kconfig?
bye,
Heiko
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
