From 29758ae0652799488a567df682a7ef391c68aecb Mon Sep 17 00:00:00 2001 From: Ulf Magnusson Date: Tue, 2 Jul 2019 17:48:19 +0200 Subject: setconfig: Use 'orig_type' instead of 'type' in error message It's confusing to have tristate symbols show up as bool in the bad-value error message when modules are disabled. set_value() validates the value against 'orig_type' rather than 'type' as well. Piggyback some quote consistency and make a comment in set_value() more informative. --- kconfiglib.py | 2 +- setconfig.py | 11 ++++++----- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/kconfiglib.py b/kconfiglib.py index aa036ec..1faa603 100644 --- a/kconfiglib.py +++ b/kconfiglib.py @@ -4473,7 +4473,7 @@ class Symbol(object): value = STR_TO_TRI[value] # If the new user value matches the old, nothing changes, and we can - # save some work. + # avoid invalidating cached values. # # This optimization is skipped for choice symbols: Setting a choice # symbol's user value to y might change the state of the choice, so it diff --git a/setconfig.py b/setconfig.py index 2a14310..b689754 100755 --- a/setconfig.py +++ b/setconfig.py @@ -38,15 +38,15 @@ def main(): parser.add_argument( "--no-check-exists", - dest='check_exists', - action='store_false', + dest="check_exists", + action="store_false", help="Ignore assignments to non-existent symbols instead of erroring " "out") parser.add_argument( "--no-check-value", - dest='check_value', - action='store_false', + dest="check_value", + action="store_false", help="Ignore assignments that didn't \"take\" (where the symbol got a " "different value, e.g. due to unsatisfied dependencies) instead " "of erroring out") @@ -76,7 +76,8 @@ def main(): if not sym.set_value(value): sys.exit("error: '{}' is an invalid value for the {} symbol {}" - .format(value, kconfiglib.TYPE_TO_STR[sym.type], name)) + .format(value, kconfiglib.TYPE_TO_STR[sym.orig_type], + name)) if args.check_value and sym.str_value != value: sys.exit("error: {} was assigned the value '{}', but got the " -- cgit v1.2.3