pkgconfig deps: Warn when a static library isn't found

A hard error makes this feature useless in most cases since a static
library usually won't be found for every library, particularly system
libraries like -lm. Instead, warn so the user can provide the static
library if they wish.

This feature will be expanded and made more extensible and more usable
in the future.

Closes https://github.com/mesonbuild/meson/issues/2785

2 files changed, 21 insertions, 4 deletions

diff --git a/mesonbuild/compilers/c.py b/mesonbuild/compilers/c.py
index a59b7d3..1c9b9b4 100644
--- a/mesonbuild/compilers/c.py
+++ b/mesonbuild/compilers/c.py
@@ -780,9 +780,12 @@ class CCompiler(Compiler):
             args = ['-l' + libname]
             if self.links(code, env, extra_args=args):
                 return args
+        # Ensure that we won't modify the list that was passed to us
+        extra_dirs = extra_dirs[:]
+        # Search in the system libraries too
+        extra_dirs += self.get_library_dirs()
         # Not found or we want to use a specific libtype? Try to find the
         # library file itself.
-        extra_dirs += self.get_library_dirs()
         prefixes, suffixes = self.get_library_naming(env, libtype)
         # Triply-nested loop!
         for d in extra_dirs:
diff --git a/mesonbuild/dependencies/base.py b/mesonbuild/dependencies/base.py
index 66bc3b4..fefab3f 100644
--- a/mesonbuild/dependencies/base.py
+++ b/mesonbuild/dependencies/base.py
@@ -516,6 +516,7 @@ class PkgConfigDependency(ExternalDependency):
                                       (self.name, out))
         self.link_args = []
         libpaths = []
+        static_libs_notfound = []
         for lib in self._convert_mingw_paths(shlex.split(out)):
             # If we want to use only static libraries, we have to look for the
             # file ourselves instead of depending on the compiler to find it
@@ -524,13 +525,26 @@ class PkgConfigDependency(ExternalDependency):
             if self.static:
                 if lib.startswith('-L'):
                     libpaths.append(lib[2:])
+                    print(lib)
                     continue
+                # FIXME: try to handle .la files in static mode too?
                 elif lib.startswith('-l') and libpaths:
                     args = self.compiler.find_library(lib[2:], self.env, libpaths, libtype='static')
                     if not args or len(args) < 1:
-                        raise DependencyException('Static library not found for {!r}'
-                                                  ''.format(lib[2:]))
-                    lib = args[0]
+                        if lib in static_libs_notfound:
+                            continue
+                        mlog.warning('Static library {!r} not found for dependency {!r}, may '
+                                     'not be statically linked'.format(lib[2:], self.name))
+                        static_libs_notfound.append(lib)
+                        # Preserve the -l arg since we couldn't resolve it to
+                        # a static library. Also need all previous -L args now.
+                        for p in libpaths:
+                            lp = '-L' + p
+                            if lp not in self.link_args:
+                                self.link_args.append(lp)
+                    else:
+                        # Replace -l arg with full path to static library
+                        lib = args[0]
             elif lib.endswith(".la"):
                 shared_libname = self.extract_libtool_shlib(lib)
                 shared_lib = os.path.join(os.path.dirname(lib), shared_libname)