msetup: not-found subprojects do not have known options

If a subproject had its SubprojectHolder created, but was disabled, its options were not
parsed; therefore the subproject should not be checked for unknown options.

Fixes: #14969
Signed-off-by: Paolo Bonzini <[email protected]>

Author: bonzini

mesonbuild/msetup.py

@@ -16,6 +16,7 @@
 if T.TYPE_CHECKING:
     from typing_extensions import Protocol
     from .coredata import SharedCMDOptions
+    from .interpreter import SubprojectHolder
 
     class CMDOptions(SharedCMDOptions, Protocol):
 
@@ -192,9 +193,9 @@ def generate(self, capture: bool = False, vslite_ctx: T.Optional[dict] = None) -
                                     'Some other Meson process is already using this build directory. Exiting.'):
             return self._generate(env, capture, vslite_ctx)
 
-    def check_unused_options(self, coredata: 'coredata.CoreData', cmd_line_options: T.Dict[OptionKey, str], all_subprojects: T.Mapping[str, object]) -> None:
+    def check_unused_options(self, coredata: 'coredata.CoreData', cmd_line_options: T.Dict[OptionKey, str], all_subprojects: T.Mapping[str, SubprojectHolder]) -> None:
         errlist: T.List[str] = []
-        known_subprojects = all_subprojects.keys()
+        known_subprojects = [name for name, obj in all_subprojects.items() if obj.found()]
         for opt in cmd_line_options:
             # Accept options that exist or could appear in subsequent reconfigurations,
             # including options for subprojects that were not used

test cases/common/42 subproject/meson.build

@@ -8,6 +8,7 @@ project('subproj user', 'c',
 assert(meson.project_name() == 'subproj user', 'Incorrect project name')
 
 sub = subproject('sublib', version : '1.0.0')
+assert(sub.found())
 
 if meson.project_version() != '2.3.4'
   error('Incorrect master project version string:' + meson.project_version())
@@ -29,3 +30,6 @@ unknown_var = sub.get_variable('does-not-exist', [])
 if unknown_var != []
   error ('unexpected fallback value for subproject.get_variable()')
 endif
+
+unusedsub = subproject('subunused', required: get_option('disabled_feature'))
+assert(not unusedsub.found())

test cases/common/42 subproject/meson.options

@@ -0,0 +1 @@
+option('disabled_feature', type: 'feature', value: 'disabled')

test cases/common/42 subproject/subprojects/subunused/meson.build

@@ -0,0 +1 @@
+syntax error because the subproject is unused

test cases/common/42 subproject/subprojects/subunused/meson.options

@@ -0,0 +1 @@
+syntax error because the subproject is unused

test cases/common/42 subproject/test.json

@@ -7,5 +7,12 @@
     {"type": "file", "file": "usr/share/sublib/COPYING"},
     {"type": "file", "file": "usr/share/sublib/subprojects/sublib/sublicense1.txt"},
     {"type": "file", "file": "usr/share/sublib/subprojects/sublib/sublicense2.txt"}
-  ]
+  ],
+  "matrix": {
+    "options": {
+      "subunused:opt1": [
+	{ "val": "42" }
+      ]
+    }
+  }
 }