Skip to content

Fix Does not compile using the latest nightly #7 #8

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

Merged
merged 3 commits into from
Aug 19, 2018
Merged

Fix Does not compile using the latest nightly #7 #8

merged 3 commits into from
Aug 19, 2018

Conversation

fmckeogh
Copy link
Contributor

@fmckeogh fmckeogh commented Jun 5, 2018

I bumped dependencies and added new proc-macro2 syntax. I don't think what I wrote is best practice though :(

Fixes #7

@fmckeogh fmckeogh changed the title Fix #7 Fix Does not compile using the latest nightly #7 Jun 5, 2018
@fmckeogh
Copy link
Contributor Author

fmckeogh commented Jun 5, 2018

I think similar fixes are required for cortex-m-rtfm-macros, but seem a bit more in depth. Dependency bumps will then be required for quite a few crates.

@et1975
Copy link

et1975 commented Jun 19, 2018

In the meanwhile, is there a workaround?

@fmckeogh
Copy link
Contributor Author

@et1975 You can add my fork to your Cargo.toml as a git crate. I think you might have to use the second to last commit as the latest will not build locally.

@et1975
Copy link

et1975 commented Jun 20, 2018

@Chocol4te thanks, had to tweak it, if anyone needs this.

@bergus
Copy link

bergus commented Jul 6, 2018

@japaric Can you merge this?

@bergus
Copy link

bergus commented Jul 7, 2018

@Chocol4te I've successfully used your version with the latest nightly, the necessary updates for cortext-m-rtfm-macros can be found in this commit.

@japaric
Copy link
Collaborator

japaric commented Aug 19, 2018

@Chocol4te Thanks!

bors r+

bors bot added a commit that referenced this pull request Aug 19, 2018
8: Fix `Does not compile using the latest nightly #7` r=japaric a=chocol4te

I bumped dependencies and added new `proc-macro2` syntax. I don't think what I wrote is best practice though :(

Fixes #7 

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

bors bot commented Aug 19, 2018

Build succeeded

@bors bors bot merged commit 21d2661 into rtic-rs:master Aug 19, 2018
bors bot added a commit to rtic-rs/rtic 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]>
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