search

hackndev / zinc

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

Dev fix nightly #277

Closed 0xc0170 closed 9 years ago

0xc0170 commented 9 years ago

Some fixes, in progress as there are still some errors to be resolved and review the current fixes.

0xc0170 commented 9 years ago

Is it just me, or the warning list is so long ,that errors are not shown in Travis? Looks like 10k is a limit? :8ball:

bgamari commented 9 years ago

Thanks for doing this!

0xc0170 commented 9 years ago

@simias Can you look at Regex and its usage in your tiva port? 2 peripherals are using it, if possible to replace it with what rust currently provides?

0xc0170 commented 9 years ago

@bgamari You might know how to solve that regex and some other errors, right now are in platformtree. will you find time to look at this?

simias commented 9 years ago

I thought regexes were convenient for parsing the platform tree, is it really annoying to maintain?

Of course it could be replaced with "manual" parsing but it would make it much more tedious and potentially bogus (and harder to maintain IMO)

Unless we standardize on a well defined grammar and have helper functions to parse the entries for all platforms, that might actually help with consistency.

farcaller commented 9 years ago

I'm working on porting the build process over to cargo where that will be a non-issue.

simias commented 9 years ago

Sounds great! :)