diff options
| author | Xavier Claessens <xavier.claessens@collabora.com> | 2018-06-19 10:50:13 -0400 |
|---|---|---|
| committer | Nirbheek Chauhan <nirbheek.chauhan@gmail.com> | 2018-06-19 20:29:57 +0000 |
| commit | 99dd5f34dece95e463a85c54034f685b9cb391e3 (patch) | |
| tree | e11f3c91dada47577b883fa072dcf25827034c84 | |
| parent | 737f61792cc33dd5f234c961bc2ab850be3f9959 (diff) | |
| download | meson-99dd5f34dece95e463a85c54034f685b9cb391e3.tar.gz | |
Downgrade unknown option from exception to warning
It used to be non-fatal warnings but recent command line refactor made
it fatal. It looks like GNOME continuous would break with this change.
To avoid delaying upcoming 0.47.0 release adoption, let's downgrade this
back to warning for now and reconsider after the release.
| -rw-r--r-- | mesonbuild/coredata.py | 2 | ||||
| -rwxr-xr-x | run_unittests.py | 8 |
2 files changed, 4 insertions, 6 deletions
diff --git a/mesonbuild/coredata.py b/mesonbuild/coredata.py index 557aabc51..f075e75e0 100644 --- a/mesonbuild/coredata.py +++ b/mesonbuild/coredata.py @@ -407,7 +407,7 @@ class CoreData: if unknown_options: unknown_options = ', '.join(sorted(unknown_options)) sub = 'In subproject {}: '.format(subproject) if subproject else '' - raise MesonException('{}Unknown options: "{}"'.format(sub, unknown_options)) + mlog.warning('{}Unknown options: "{}"'.format(sub, unknown_options)) def load(build_dir): filename = os.path.join(build_dir, 'meson-private', 'coredata.dat') diff --git a/run_unittests.py b/run_unittests.py index 11deb133a..582d6f5f1 100755 --- a/run_unittests.py +++ b/run_unittests.py @@ -2142,11 +2142,9 @@ recommended as it is not supported on some platforms''') self.assertEqual(obj.builtins['default_library'].value, 'shared') self.wipe() - # Should fail on unknown options - with self.assertRaises(subprocess.CalledProcessError) as cm: - self.init(testdir, extra_args=['-Dbad=1', '-Dfoo=2', '-Dwrong_link_args=foo']) - self.assertNotEqual(0, cm.exception.returncode) - self.assertIn('Unknown options: "bad, foo, wrong_link_args"', cm.exception.output) + # Should warn on unknown options + out = self.init(testdir, extra_args=['-Dbad=1', '-Dfoo=2', '-Dwrong_link_args=foo']) + self.assertIn('Unknown options: "bad, foo, wrong_link_args"', out) self.wipe() # Should fail on malformed option |
