hackndev / zinc

The bare metal stack for rust
zinc.rs
Apache License 2.0
1k stars 100 forks source link

Updates for latest rust #371

Closed richo closed 8 years ago

richo commented 8 years ago

I've started trying to get Zinc working on modern rust.

I'm currently blocked on https://github.com/rust-lang/rust/pull/32269 but this at least massively cuts down on the compile errors.

richo commented 8 years ago

This gets zing building cleanly.

I will hopefully have a look at the tests tomorrow.

farcaller commented 8 years ago

Looks nice, thanks for your work. I'll keep this open until we get green on travis though.

richo commented 8 years ago

Works for me! There's some pretty severe breakage in the way that ASTs are constructed, and my lack of familiarity with the code is making debugging hard.

If it's ok with you I'll braindump into this issue and see if anything makes more sense to you?

farcaller commented 8 years ago

Sounds good.

posborne commented 8 years ago

As an aside, we've been using homu (http://homu.io/) on nix and it has worked out pretty well. https://github.com/nix-rust/nix/blob/master/CONTRIBUTING.md#homu-the-bot-who-merges-all-the-prs. Homu will verify that the resultant merged state passes before merging into master.

farcaller commented 8 years ago

Cool, thanks, I'll take a look at it (it's the next iteration of bors, right?)

posborne commented 8 years ago

Yes, I'm not sure if it is the next generation or just the generally available version. Either way, yes -- it's basically bors.

gyscos commented 8 years ago

This fixes ioreg, but it seems platformtree still uses some of the old types (for instance platformtree/src/parser.rs:19 imports syntax::ast::LitInt).

richo commented 8 years ago

Thanks to @Gyscos I was able to build blinky_lpc17xx using rust 1.9-dev.

I'm curious to see what travis will make of the tests, but this is progressing.

gyscos commented 8 years ago

374, already merged, includes most of the fixes present here.

mcoffin commented 8 years ago

already merged

Yep. Closing this.