diff --git a/RELEASES.md b/RELEASES.md index b07618b7..54888ec9 100644 --- a/RELEASES.md +++ b/RELEASES.md @@ -10,6 +10,9 @@ ### New features - Support using the `$` generator expression in `OUTPUT_DIRECTORY`. [#459] +- If `corrosion_link_libraries()` is called on a Rust static library target, then + `target_link_libraries()` is called to propogate the dependencies to C/C++ consumers. + Previously a warning was emitted in this case and the arguments ignored. ### Fixes diff --git a/cmake/Corrosion.cmake b/cmake/Corrosion.cmake index 14b7bd2d..e16c4900 100644 --- a/cmake/Corrosion.cmake +++ b/cmake/Corrosion.cmake @@ -1071,11 +1071,15 @@ function(corrosion_set_features target_name) endfunction() function(corrosion_link_libraries target_name) - if(TARGET "${target_name}-static" AND NOT TARGET "${target_name}-shared") - message(WARNING "The target ${target_name} builds a static library." - "The linker is never invoked for a static libraries to link has effect " - " aside from establishing a build dependency." - ) + if(TARGET "${target_name}-static") + message(DEBUG "The target ${target_name} builds a static Rust library." + "Calling `target_link_libraries()` instead." + ) + target_link_libraries("${target_name}-static" INTERFACE ${ARGN}) + if(NOT TARGET "${target_name}-shared") + # Early return, since Rust won't invoke the linker for static libraries + return() + endif() endif() add_dependencies(_cargo-build_${target_name} ${ARGN}) foreach(library ${ARGN}) @@ -1089,7 +1093,7 @@ function(corrosion_link_libraries target_name) corrosion_add_target_local_rustflags(${target_name} "-L$") corrosion_add_target_local_rustflags(${target_name} "-l$") endforeach() -endfunction(corrosion_link_libraries) +endfunction() function(corrosion_install) # Default install dirs diff --git a/doc/src/advanced.md b/doc/src/advanced.md index 52238ad8..203c48bf 100644 --- a/doc/src/advanced.md +++ b/doc/src/advanced.md @@ -40,17 +40,21 @@ For rust `cdylib`s and `bin`s, the linker is invoked via `rustc` and CMake just When CMake invokes the linker, everything is as usual. CMake will call the linker with the compiler as the linker driver and users can just use the regular CMake functions to -modify linking behaviour. The corrosion functions mentioned below have **no effect**. +modify linking behaviour. `corrosion_set_linker()` has **no effect**. +As a convenience, `corrosion_link_libraries()` will forward its arguments to `target_link_libraries()`. #### Rustc invokes the linker Rust `cdylib`s and `bin`s are linked via `rustc`. Corrosion provides several helper functions to influence the linker invocation for such targets. -`corrosion_link_libraries()` is essentially the equivalent to `target_link_libraries()`, -if the target is a rust `cdylib` or `bin`. +`corrosion_link_libraries()` is a limited version of `target_link_libraries()` +for rust `cdylib` or `bin` targets. Under the hood this function passes `-l` and `-L` flags to the linker invocation and ensures the linked libraries are built first. +Much of the advanced functionality available in `target_link_libraries()` is not implemented yet, +but pull-requests are welcome! In the meantime, users may want to use +`corrosion_add_target_local_rustflags()` to pass customized linking flags. `corrosion_set_linker()` can be used to specify a custom linker, in case the default one chosen by corrosion is not what you want.