Conversation
|
I don't think a feature like this should belong in the main branch. It's only a temporary fix until we get better syntax to do it properly from only the interop crate. |
|
@MiniaczQ I'm not sure there is much of an alternative to this currently (maybe doing just wrapper types in the interop crate?). I don't think we are getting better syntax for doing this anytime soon based on looking at rust-lang/rfcs#2393. |
It sounds like wrappers in the interop crate is the way to go, but that requires the most knowledge of the innards. This change was relatively easy to make as an outsider without changing how |
|
Thank you for this PR @deontologician! I’m closing it since I ended up using wrapper types instead: dimforge/bevy_rapier#102. |
After bevyengine/bevy#2254 there is a requirement that anything used as a component must derive
Componentexplicitly.From what I can tell, most of the data structures that are used as components in
bevy_rapierare defined inrapier, butSharedShapeis re-exported fromparry, so the annotation has to go here.I attempted to minimize the effect on
parryby:One thing I'm unsure of is specifying the version of bevy. I've included it as
0.5here, because that's what's on crates.io right now, but this code won't actually work with that version, only with a particular branch of bevy in this PR (which will hopefully land soon):bevyengine/bevy#2305