Isaac
October 25, 2018, 3:37pm
1
Hey everyone,
I cloned the PARSEC git repo and tried to execute the test with cargo test --release --features=dump-graphs -- --nocapture
but I get following error:
error[E0446]: private type `meta_voting::meta_elections::MetaElectionHandle` in public interface
--> src/meta_voting/meta_elections.rs:87:33
|
87 | pub fn all<'a>(&'a self) -> impl Iterator<Item = MetaElectionHandle> + 'a {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't leak private type
error[E0446]: private type `meta_voting::meta_elections::MetaElectionHandle` in public interface
--> src/meta_voting/meta_elections.rs:95:37
|
95 | pub fn decided<'a>(&'a self) -> impl Iterator<Item = MetaElectionHandle> + 'a {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't leak private type
error: aborting due to 2 previous errors
Any advice?
Cheers!
3 Likes
Hi Isaac, what version of rustc are you using?
3 Likes
Isaac
October 25, 2018, 3:53pm
3
Hey,
The version: rustc 1.27.0
1 Like
Iād recommend trying with the latest stable version. Works for me locally with 1.29.0 (which is also what CI is set to I think).
EDIT: something like
rustup self update
rustup update 1.29.0
rustup default 1.29.0
although I think latest is actually 1.29.2 these days.
5 Likes
May I suggest changing the title of this thread to be something more appropriate? It sounds like there is nothing wrong with the tests, but rather the environment they are being run in.
2 Likes
Actually, a more accurate description would be:
compilation error when attempting to run PARSEC tests
1 Like
urrtag
October 25, 2018, 4:31pm
7
Rust 1.30.0
they just released 1.30 (itās so new, that the blog link isnāt working yet )
Isaac
October 25, 2018, 4:37pm
8
Okey it works now! Thanks!
Maybe someone should add, that at least rustc 1.29.0 should be installed as prerequisite int he tutorial.md
2 Likes
urrtag
October 25, 2018, 4:42pm
9
Why? Just use the latest stable, so thereās no need to updated the readme every 6 weeks. Also we are talking about a not-production-ready/work-in-progress software here.
JPL
October 25, 2018, 4:49pm
10
Still worth doing though. Just say āmake sure youāre using the latest stable version of Rustā and a link to Rust Programming Language would save people wasting time
7 Likes
Good point. Weāll merge this PR later today hopefully.
Thanks for the idea!
6 Likes
Ah so cheeky of you to mention appveyor in favour of travis there @anon86652309 hahaha
6 Likes
I thought Iād favour the one which focuses on the premier OS!
[hiding behind desk awaiting incoming flak]
7 Likes
lol, ya ya you āmouseā users and your fancy OS
8 Likes