Skip to content
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

Update dependencies in microbit/src/03-setup #573

Closed
wants to merge 4 commits into from

Conversation

nikgul
Copy link
Contributor

@nikgul nikgul commented Jun 8, 2024

Update dependencies

Cargo.toml:

  • Change the rust edition to 2021
  • Updated cortex-m 0.7.3 -> 0.7.7, added feature critical-section-single-core
  • Updated cortex-m-rt 0.7.0 -> 0.7.3
  • Updated rtt-target 0.3.1 -> 0.5.0, removed feature cortex-m
  • Updated panic-rtt-target 0.1.2 -> 0.1.3, removed feature cortex-m

src/main.rs:

  • To prevent a link error regarding undefined symbols (_critical_section_1_0_acquire and _critical_section_1_0_release) I have added use cortex_m as _;

The code compiles, and I see the RTT output from cargo embed.

Cargo.toml:
  * Change the rust edition to 2021
  * Updated cortex-m 0.7.3 -> 0.7.7, added feature
    `critical-section-single-core`
  * Updated cortex-m-rt 0.7.0 -> 0.7.3
  * Updated rtt-target 0.3.1 -> 0.5.0, removed feature `cortex-m`
  * Updated panic-rtt-target 0.1.2 -> 0.1.3, removed feature `cortex-m`

src/main.rs:
  * To prevent a link error regarding undefined symbols
    (`_critical_section_1_0_acquire` and `_critical_section_1_0_release`)
    I have added `use cortex_m as _;`

The code compiles, and I see the RTT output from cargo embed.
@nikgul nikgul requested a review from a team as a code owner June 8, 2024 23:15
@nikgul nikgul closed this Jun 12, 2024
@nikgul nikgul deleted the microbit-03 branch June 12, 2024 18:56
@nikgul
Copy link
Contributor Author

nikgul commented Jun 12, 2024

Renamed the branch to microbit in my fork, you are welcome to keep and eye on it if you like as I update the crate versions in the book and will open a new pull request, when I have something more finished

@BartMassey
Copy link
Member

Thanks much for looking into this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants