rust: fix -C prefer-dynamic behavior

I noticed when building a project that uses a proc macro that Meson
passed -C prefer-dynamic for the executable, and not the proc macro,
while cargo passed -C prefer-dynamic for the proc macro, but not for
the executable.  Meson's behavior broke setting -C panic=abort on the
executable.

As far as we can tell, because we explicitly pass each library path to
rustc, the only thing -C prefer-dynamic affects in Meson is how the
standard libraries are linked.  Generally, one does not want the
standard libraries to be dynamically linked, because if the Rust
compiler is ever updated, anything linked against the old standard
libraries will likely break, due to the lack of a stable Rust ABI.
Therefore, I've reorganised Meson's behavior around the principle that
the standard libraries should only be dynamically linked when Rust
dynamic linking has already been opted into in some other way.  The
details of how this manifests are now explained in the documentation.

1 files changed, 10 insertions, 9 deletions

diff --git a/mesonbuild/backend/ninjabackend.py b/mesonbuild/backend/ninjabackend.py
index 348d0c9..c60143b 100644
--- a/mesonbuild/backend/ninjabackend.py
+++ b/mesonbuild/backend/ninjabackend.py
@@ -2043,16 +2043,17 @@ class NinjaBackend(backends.Backend):
             args += ['-L', d]
         target_deps = target.get_dependencies()
         has_shared_deps = any(isinstance(dep, build.SharedLibrary) for dep in target_deps)
-        if isinstance(target, build.SharedLibrary) or has_shared_deps:
-            has_rust_shared_deps = any(isinstance(dep, build.SharedLibrary) and dep.uses_rust()
-                                       and dep.rust_crate_type not in {'cdylib', 'proc-macro'}
-                                       for dep in target_deps)
-            if cratetype not in {'cdylib', 'proc-macro'} or has_rust_shared_deps:
-                # add prefer-dynamic if any of the Rust libraries we link
-                # against are dynamic or this is a dynamic library itself,
-                # otherwise we'll end up with multiple implementations of crates
-                args += ['-C', 'prefer-dynamic']
+        has_rust_shared_deps = any(dep.uses_rust()
+                                   and dep.rust_crate_type == 'dylib'
+                                   for dep in target_deps)
+
+        if cratetype in {'dylib', 'proc-macro'} or has_rust_shared_deps:
+            # add prefer-dynamic if any of the Rust libraries we link
+            # against are dynamic or this is a dynamic library itself,
+            # otherwise we'll end up with multiple implementations of libstd.
+            args += ['-C', 'prefer-dynamic']
 
+        if isinstance(target, build.SharedLibrary) or has_shared_deps:
             # build the usual rpath arguments as well...
 
             # Set runtime-paths so we can run executables without needing to set