diff --git a/mesonbuild/interpreter.py b/mesonbuild/interpreter.py index 3e39720ef..d554cce4d 100644 --- a/mesonbuild/interpreter.py +++ b/mesonbuild/interpreter.py @@ -2447,7 +2447,12 @@ class Interpreter(InterpreterBase): def get_non_matching_default_options(self) -> T.Iterator[T.Tuple[str, str, coredata.UserOption]]: for def_opt_name, def_opt_value in self.project_default_options.items(): cur_opt_value = self.coredata.options.get(def_opt_name) - if cur_opt_value is not None and def_opt_value != cur_opt_value.value: + try: + if cur_opt_value is not None and cur_opt_value.validate_value(def_opt_value) != cur_opt_value.value: + yield (str(def_opt_name), def_opt_value, cur_opt_value) + except mesonlib.MesonException: + # Since the default value does not validate, it cannot be in use + # Report the user-specified value as non-matching yield (str(def_opt_name), def_opt_value, cur_opt_value) def build_func_dict(self): diff --git a/run_unittests.py b/run_unittests.py index dd59ceb00..72b497a66 100755 --- a/run_unittests.py +++ b/run_unittests.py @@ -4002,7 +4002,8 @@ class AllPlatformTests(BasePlatformTests): # Verify default values when passing no args that affect the # configuration, and as a bonus, test that --profile-self works. - self.init(testdir, extra_args=['--profile-self', '--fatal-meson-warnings']) + out = self.init(testdir, extra_args=['--profile-self', '--fatal-meson-warnings']) + self.assertNotIn('[default: true]', out) obj = mesonbuild.coredata.load(self.builddir) self.assertEqual(obj.options[OptionKey('default_library')].value, 'static') self.assertEqual(obj.options[OptionKey('warning_level')].value, '1')