Fix unknown base options not detected in commandline arguments

pull/12910/head
Charles Brunet 11 months ago committed by Dylan Baker
parent 76f6874e48
commit cc4cfbcad9
  1. 7
      mesonbuild/coredata.py
  2. 2
      unittests/allplatformstests.py
  3. 7
      unittests/platformagnostictests.py

@ -974,6 +974,8 @@ class CoreData:
return dirty
def set_default_options(self, default_options: T.MutableMapping[OptionKey, str], subproject: str, env: 'Environment') -> None:
from .compilers import base_options
# Main project can set default options on subprojects, but subprojects
# can only set default options on themselves.
# Preserve order: if env.options has 'buildtype' it must come after
@ -1005,7 +1007,10 @@ class CoreData:
continue
# Skip base, compiler, and backend options, they are handled when
# adding languages and setting backend.
if k.type in {OptionType.COMPILER, OptionType.BACKEND, OptionType.BASE}:
if k.type in {OptionType.COMPILER, OptionType.BACKEND}:
continue
if k.type == OptionType.BASE and k in base_options:
# set_options will report unknown base options
continue
options[k] = v

@ -2694,7 +2694,7 @@ class AllPlatformTests(BasePlatformTests):
# It is not an error to set wrong option for unknown subprojects or
# language because we don't have control on which one will be selected.
self.init(testdir, extra_args=['-Dc_wrong=1', '-Dwrong:bad=1', '-Db_wrong=1'])
self.init(testdir, extra_args=['-Dc_wrong=1', '-Dwrong:bad=1'])
self.wipe()
# Test we can set subproject option

@ -285,3 +285,10 @@ class PlatformAgnosticTests(BasePlatformTests):
out = self.init(testdir, extra_args=['--reconfigure', '-Db_ndebug=if-release', '-Dc_std=c99'])
self.assertIn('\nMessage: b_ndebug: if-release\n', out)
self.assertIn('\nMessage: c_std: c99\n', out)
def test_setup_with_unknown_option(self):
testdir = os.path.join(self.common_test_dir, '1 trivial')
for option in ('not_an_option', 'b_not_an_option'):
out = self.init(testdir, extra_args=['--wipe', f'-D{option}=1'], allow_fail=True)
self.assertIn(f'ERROR: Unknown options: "{option}"', out)

Loading…
Cancel
Save