Skip to content

Fix "Could not find Op in proc_macro" #80

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

Closed
wants to merge 4 commits into from

Conversation

fmckeogh
Copy link

@fmckeogh fmckeogh commented Jun 7, 2018

Fixes nightly error with proc-macro2.

Should not be merged until rtic-rs/rtic-syntax#8 is merged and new rtfm-syntax version is published.

@little-arhat
Copy link

This is addressed in tq branch as well, I belive (see japaric/cortex-m-rtfm@7b598c0). But it would be nice to have it in master!

@fmckeogh
Copy link
Author

fmckeogh commented Jun 7, 2018

Sorry, my bad, I didn't check other branches

@little-arhat
Copy link

@Chocol4te no worries, as far as I know, tq branch has other, quite significant changes, so it would be nice to backport just compatibility changes to master, as it's not clear (for me, at least) whether and when tq will be merged.

@MabezDev
Copy link
Contributor

I have a working version of rtfm for the latest cortex-m-rt here. It's not perfect though, systick can't be used as an interrupt source.

@japaric
Copy link
Collaborator

japaric commented Aug 24, 2018

Thanks @Chocol4te!

bors r+

@japaric
Copy link
Collaborator

japaric commented Aug 24, 2018

bors r+

bors bot added a commit that referenced this pull request Aug 24, 2018
80: Fix "Could not find `Op` in `proc_macro`" r=japaric a=chocol4te

Fixes nightly error with `proc-macro2`. 

Should not be merged until rtic-rs/rtic-syntax#8 is merged and new `rtfm-syntax` version is published.

Co-authored-by: Ferdia McKeogh <[email protected]>
Co-authored-by: Jorge Aparicio <[email protected]>
@bors
Copy link
Contributor

bors bot commented Aug 24, 2018

Build failed

@japaric japaric mentioned this pull request Aug 24, 2018
bors bot added a commit that referenced this pull request Aug 24, 2018
85: fix master r=japaric a=japaric

closes #80

Co-authored-by: Ferdia McKeogh <[email protected]>
Co-authored-by: Jorge Aparicio <[email protected]>
bors bot added a commit that referenced this pull request Aug 24, 2018
85: fix master r=japaric a=japaric

closes #80

Co-authored-by: Ferdia McKeogh <[email protected]>
Co-authored-by: Jorge Aparicio <[email protected]>
@bors bors bot closed this in #85 Aug 24, 2018
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.

5 participants