Docker file for GCCRS and Cargo-gccrs #826
Assignees
Labels
Comments
CohenArthur
added a commit
to CohenArthur/gccrs
that referenced
this issue
Feb 17, 2022
934: macromatch: Add location to abstract MacroMatch class r=philberty a=CohenArthur Closes Rust-GCC#928 This adds location to the all child classes of the `MacroMatch` abstract class. The current locations are as follow, which I believe is what is expected but might be wrong. ```rust test.rs:2:6: error: macro match fragment 2 | ($a:expr, $b:expr) => { $a + $b }; | ^ test.rs:2:15: error: macro match fragment 2 | ($a:expr, $b:expr) => { $a + $b }; | ^ test.rs:2:5: error: macro matcher 2 | ($a:expr, $b:expr) => { $a + $b }; | ^ test.rs:3:8: error: macro match fragment 3 | ($($i:ident)*) => { $($i)* } | ^ test.rs:3:17: error: macro match repetition! 3 | ($($i:ident)*) => { $($i)* } | ^ test.rs:3:5: error: macro matcher 3 | ($($i:ident)*) => { $($i)* } | ^ ``` I think this should be rebased on Rust-GCC#932 so that I can remove the FIXME 937: dockerfile: Install cargo-gccrs alongside gccrs r=philberty a=CohenArthur Closes Rust-GCC#826 Co-authored-by: Arthur Cohen <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Now that we have cargo-gccrs we should update our Dockerfile (https://github.com/Rust-GCC/gccrs/blob/0024bc2f028369b871a65ceb11b2fddfb0f9c3aa/Dockerfile) to install cargo-gccrs https://github.com/Rust-GCC/cargo-gccrs
This dockerfile will allow us to use github automation for testing upon bors merge to master to run our #682 no_core blake project
The text was updated successfully, but these errors were encountered: