Honor dependency `fallback` argument even if the dependency is not required (#735)

You can potentially have a fallback subproject and if that subproject
fails, you can continue without that dependency

mesonbuild/interpreter.py

@@ -1717,14 +1717,23 @@ class Interpreter():
             dep = cached_dep
         else:
             # We need to actually search for this dep
+            exception = None
+            dep = None
             try:
                 dep = dependencies.find_external_dependency(name, self.environment, kwargs)
-            except dependencies.DependencyException:
+            except dependencies.DependencyException as e:
+                exception = e
+                pass
+
+            if not dep or not dep.found():
                 if 'fallback' in kwargs:
-                    dep = self.dependency_fallback(name, kwargs)
+                    fallback_dep = self.dependency_fallback(name, kwargs)
-                    self.coredata.deps[identifier] = dep.held_object
+                    if fallback_dep:
-                    return dep
+                        return fallback_dep
-                raise
+
+                if not dep:
+                    raise exception
+
         self.coredata.deps[identifier] = dep
         return DependencyHolder(dep)
 
@@ -1740,7 +1749,10 @@ class Interpreter():
             mlog.log('Also couldn\'t find a fallback subproject in',
                     mlog.bold(os.path.join(self.subproject_dir, dirname)),
                     'for the dependency', mlog.bold(name))
+            if kwargs.get('required', True):
                 raise
+            else:
+                return None
         dep = self.subprojects[dirname].get_variable_method([varname], {})
         if not isinstance(dep, (DependencyHolder, InternalDependencyHolder)):
             raise InterpreterException('Fallback variable is not a dependency object.')

test cases/common/95 dep fallback/meson.build

@@ -1,6 +1,10 @@
 project('dep fallback', 'c')
 
-bob = dependency('boblib', fallback : ['boblib', 'bob_dep'])
+bob = dependency('boblib', fallback : ['boblib', 'bob_dep'], required: false)
+if not bob.found()
+    error('Bob is actually needed')
+endif
+jimmy = dependency('jimmylib', fallback : ['jimmylib', 'jimmy_dep'], required: false)
 
 exe = executable('bobtester', 'tester.c', dependencies : bob)
 test('bobtester', exe)