Attempt to update to more recent nightly compiler #4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wez
force-pushed
the
update
branch
4 times, most recently
from
28e0cd6
to
e1bd18d
Compare
I was seeing rust-embedded/cortex-m-rt#92 here too, so needed to upgrade gcc both on my local system and in the CI config. rtfm's `app!` macro doesn't know how to emit an appropriate `entry!` call, so the rtfm based examples now have a little bit of glue for that.
|
@wez did you get the linker issues sorted out? I have also been working on moving to a newer nighlty (although I didn't go quite as far as updating svd2rust) and I have a working crate that compiles. IIRC the issues I was having mainly stemmed from using I'll try things out with this version instead of my hack. |
|
Bear in mind the linker syntax has changed in the latest nightly - see the
changes to the cargo config in my pull request.
…On Tue, 28 Aug 2018, 3:49 pm zklapow, ***@***.***> wrote:
@wez <https://github.com/wez> did you get the linker issues sorted out? I
have also been working on moving to a newer nighlty (although I didn't go
quite as far as updating svd2rust) and I have a working crate that
compiles. IIRC the issues I was having mainly stemmed from using
cortex-m-rtfm which is still way out of date, but I was able to get
things to compile by using a custom version with some updated deps
(although my interrupts dont work now so ¯_(ツ)_/¯).
I'll try things out with this version instead of my hack.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABOyBVbmdvoS2if8wh8kfqEgYHHoCiyoks5uVVhjgaJpZM4WItAY>
.
|
|
Yep, this took care of getting us back on nightly and even got rtfm working there too. If you have linker issues locally it indicates that you'll need to upgrade your gcc arm tool chain; there is information on that in the readme. |
kaizensparc
pushed a commit
to kaizensparc/atsamd
that referenced
this pull request
Dec 24, 2021
I was seeing rust-embedded/cortex-m-rt#92 here too, so needed to upgrade gcc both on my local system and in the CI config. rtfm's `app!` macro doesn't know how to emit an appropriate `entry!` call, so the rtfm based examples now have a little bit of glue for that.
TDHolmes
added a commit
that referenced
this pull request
Mar 28, 2022
* Cleanup clippy lints * Changelog * rustfmt * Don't allow missing_safety_docs, revert From/Into GClock-Hertz impls * Docs: NVM, PUKCC (#4) Co-authored-by: Henrik Tjäder <henrik.tjader@volvocars.com> * rustfmt * Allow double underscores in entire pukcc module * Fix rust_2018_idioms lints * Fix clippy lints in T1 examples Co-authored-by: Henrik Tjäder <henrik@tjaders.com> Co-authored-by: Henrik Tjäder <henrik.tjader@volvocars.com> Co-authored-by: Tyler Holmes <tyler@holmesengineering.com>
rtsuk
pushed a commit
to rtsuk/atsamd
that referenced
this pull request
Mar 29, 2022
* Cleanup clippy lints * Changelog * rustfmt * Don't allow missing_safety_docs, revert From/Into GClock-Hertz impls * Docs: NVM, PUKCC (atsamd-rs#4) Co-authored-by: Henrik Tjäder <henrik.tjader@volvocars.com> * rustfmt * Allow double underscores in entire pukcc module * Fix rust_2018_idioms lints * Fix clippy lints in T1 examples Co-authored-by: Henrik Tjäder <henrik@tjaders.com> Co-authored-by: Henrik Tjäder <henrik.tjader@volvocars.com> Co-authored-by: Tyler Holmes <tyler@holmesengineering.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It's been a little while since I had time to hack on this, and I wanted to address the issue behind #3 by making things build on the current versions of the tools.
I wasn't entirely successful but I think we're close. I'm hoping that someone else has the flash of inspiration needed to carry this across the line.
This commit:
Things almost build; the libraries compile but the examples fail to link:
AFAICT,
Resetis the symbol that it wants and this should be provided bycortex-m-rtwhen built inrtmode.I'm probably missing something subtle and simple; help wanted! :-)