-
Notifications
You must be signed in to change notification settings - Fork 84
New cortex-m-rt rand dependency breaks builds #105
Comments
sigh this is probably that Cargo limitation where it keeps a single dependency graph for both build-dependencies (compiled for the host) and other dependencies (compiled for the target). Making |
I meant: make our |
That works. NB: It's fairly annoying to test that... requires all local checkouts and changing everything to paths because (of course) Oh, and I run into another funkiness which prevents me from compiling
|
Due to the single dependency tree, the attempted use of a std version flips all depending crates to the std version as well which will not compile on no_std systems. Fixes rust-embedded#105 Signed-off-by: Daniel Egger <daniel@eggers-club.de>
I don't get this error when using the new 0.6.0 on a thumbv7em-none-eabi target. Everything builds fine. |
@adamgreig this only happens if your crate depend on the |
Right, I'd just realised but you beat me to pointing it out. Makes sense. |
Ah, and I get your point now. Will do. ;) |
Due to the single dependency tree, the attempted use of a std version flips all depending crates to the std version as well which will not compile on no_std systems. Fixes rust-embedded#105 Signed-off-by: Daniel Egger <daniel@eggers-club.de>
Can we move this forward? |
Due to the single dependency tree, the attempted use of a std version flips all depending crates to the std version as well which will not compile on no_std systems. Fixes rust-embedded#105 Signed-off-by: Daniel Egger <daniel@eggers-club.de>
What is the plan for this? Will merging therealprof@bddf23b suffice or is there more to it? |
107: Disable default-features on rand dependency to avoid std version r=japaric a=therealprof Due to the single dependency tree, the attempted use of a std version flips all depending crates to the std version as well which will not compile on no_std systems. Fixes #105 Signed-off-by: Daniel Egger <daniel@eggers-club.de> Co-authored-by: Daniel Egger <daniel@eggers-club.de>
I was just trying to port the
microbit
crate tocortex-m-rt
0.6.0 but I'm facing this here:@japaric Seems something is messing (or not) with the required flags for
no_std
.The text was updated successfully, but these errors were encountered: