#220 rust2rpm: don't remove stripped foreign dependencies if they're not optional
Closed: Fixed a year ago by decathorpe. Opened 2 years ago by decathorpe.

For example, the flate2 crate defines two different dependencies on miniz_oxide:

[dependencies.miniz_oxide]
version = "0.5.0"
optional = true
default-features = false
[target."cfg(all(target_arch = \"wasm32\", not(target_os = \"emscripten\")))".dependencies.miniz_oxide]
version = "0.5.0"
default-features = false

The second dependency on miniz_oxide is stripped by the automatic patch, but it is also removed from the rust-backend feature:

rust_backend = ["miniz_oxide"]

which breaks the build of the crate.

If a stripped dependency is not optional, it cannot be named in feature dependencies, so it should also not be removed.


Metadata Update from @decathorpe:
- Issue set to the milestone: 23

2 years ago

Metadata Update from @decathorpe:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

a year ago

Login to comment on this ticket.

Metadata
Related Pull Requests
  • #225 Merged a year ago
  • #216 Closed a year ago