tree 7134c95f300db64925efef7f8740e69af81b8445
parent 24e2f7df92641de0351a96096fb2c490b2436bb8
author Eli Lipsitz <elipsitz@fb.com> 1626384375 -0700
committer Commit Bot <commit-bot@chromium.org> 1626472695 +0000

Fix linking dynamic libraries in Rust binaries

An earlier commit, 646a62e, changed how Rust links against native
dependencies. Rather than a `-l...` argument to instruct rustc to
link against the specified library, GN uses `-Clink-arg=` to bypass
rustc and directly instruct the linker to link the native library.

This has a few implications: normally, Rust uses the `-Bstatic`
and `-Bdynamic` linker arguments to tell the linker whether or not
to link dynamic binaries. These flags take effect for the rest
of the arguments specified (or until another one of the flags changes
the mode). rustc has logic to ensure that these flags are emitted
correctly, switching the mode as necessary when it outputs linker
arguments to link objects of each type.

Bypassing rustc via `-Clink-arg` prevents this logic from being used,
and so it may be the case that, when rustc is building up the linker
arguments, it's already emitted a `-Bstatic` flag, causing the linker
arguments to be in the wrong mode when we emit the arguments to
link the native libraries. Specicially, we cannot pass a dynamic object
when the linker is in static-only mode.

This commit addresses this issue by emitting a `-Bdynamic` linker
argument before emitting the `-Clink-arg` arguments for the native
libraries, ensuring that the linker is always in a mode that allows
linking dynamic libraries.

Change-Id: I79c1f285e661dc7e4638b1374b718fbbc4f31049
Reviewed-on: https://gn-review.googlesource.com/c/gn/+/12000
Reviewed-by: Tyler Mandry <tmandry@google.com>
Commit-Queue: Tyler Mandry <tmandry@google.com>