-
Notifications
You must be signed in to change notification settings - Fork 495
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix compile error #341
Fix compile error #341
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @jamesmunns (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thank you!
Could you also apply these fixes to the other chapters?
OK, I'll give it a shot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks! We should test this on the real hardware, though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I think this looks great.
Would you also squash the commits into one? That would be easier for people looking at the history later on.
Since you have also tested this on hardware, once that is done this would be ready for merge :)
Okay, I'm done. @eldruin |
fix(09-clocks-and-timers): Compile failed in release mode docs: Update putting-it-all-together.md fix: compile error in chapter 11 fix: compile error in chapter 14 fix: compile error in chapter 15 fix: compile error in chapter 16 chore: handle `Result` with unwrap rather than ok chore: remove useless dependencies fix: restore main.rs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @dzvon!
Thank you very much!
bors r+
Fix #338 #340
It looks like these problems are caused by multiple versions. I have no idea of the underlying reasons, but I think the simplest workaround is to keep one version. So I remove f3 crate which depends on
cortex-m v0.5.6
to fix it.There are also the same problems in chapter 11,14,15. But it seems more complex, as I'm a newbie in this field, and I'm still learning :-). So I only submit this to fix chapter 9. Thanks a lot.