`cargo ws rename` misses platform-specific dependencies
jonas-schievink opened this issue · 2 comments
jonas-schievink commented
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" }
pksunkara commented
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.
pksunkara commented
Released a new version that should work. Please let me know if it doesn't.