pksunkara/cargo-workspaces

`cargo ws rename` misses platform-specific dependencies

jonas-schievink opened this issue · 2 comments

The rust-analyzer auto-publish flow runs cargo ws rename ra_ap_%n to prefix all package names with ra_ap_, but it has recently started failing because that step leaves this platform-specific dependency declaration untouched:

[target.'cfg(not(any(target_arch = "wasm32", target_os = "emscripten")))'.dependencies]
toolchain = { path = "../toolchain", version = "0.0.0" }

All other dependencies on toolchain have been updated to instead say

toolchain = { path = "../toolchain", version = "0.0.0", package = "ra_ap_toolchain" }

Currently, we have a weird toml parser as discussed in #24 . Recently there have been improvements to toml_edit, so maybe I could use that to fix this. But I am not sure if that parser correctly recognizes this dependency either.

Released a new version that should work. Please let me know if it doesn't.