Skip to content

Commit

Permalink
corrosion_import_crate: Fix PROFILE argument in combination with CRAT…
Browse files Browse the repository at this point in the history
…ES (#496)

Do not forward the PROFILE argument from `corrosion_import_crate`
to `_generator_add_cargo_targets`.
The PROFILE argument is forwarded via a target property set during 
`corrosion_import_crate`. The remaining argument forwarding can
cause issues when combined with a multi-value-argument,
such as `CRATES`.

---------

Co-authored-by: Jonathan Schwender <schwenderjonathan@gmail.com>
  • Loading branch information
Felix-El and jschwe authored Apr 3, 2024
1 parent 366066c commit d5bdbfa
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions cmake/Corrosion.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -894,7 +894,6 @@ function(corrosion_import_crate)
_corrosion_option_passthrough_helper(FROZEN COR frozen)
_corrosion_arg_passthrough_helper(CRATES COR crate_allowlist)
_corrosion_arg_passthrough_helper(CRATE_TYPES COR crate_types)
_corrosion_arg_passthrough_helper(PROFILE COR cargo_profile)

if(COR_PROFILE)
if(Rust_VERSION VERSION_LESS 1.57.0)
Expand Down Expand Up @@ -934,7 +933,6 @@ function(corrosion_import_crate)
imported_crates
${crate_allowlist}
${crate_types}
${cargo_profile}
${no_linker_override}
)

Expand Down

0 comments on commit d5bdbfa

Please sign in to comment.