Compilation error when attempting to run PARSEC tests

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

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

Rust 1.30.0

they just released 1.30 (it’s so new, that the blog link isn’t working yet)

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

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.

Still worth doing though. Just say ‘make sure you’re using the latest stable version of Rust’ and a link to https://www.rust-lang.org/en-US/ 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! :laughing:
[hiding behind desk awaiting incoming flak]

7 Likes

lol, ya ya you “mouse” users and your fancy OS

8 Likes