Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compilation fails with: feature edition is required #157

Closed
triska opened this issue Sep 6, 2019 · 7 comments
Closed

Compilation fails with: feature edition is required #157

triska opened this issue Sep 6, 2019 · 7 comments

Comments

@triska
Copy link
Contributor

triska commented Sep 6, 2019

With the latest commit, when I do:

$ cargo install scryer-prolog

I get:

    Updating registry `https://github.com/rust-lang/crates.io-index`
  Installing scryer-prolog v0.8.89
error: failed to compile `scryer-prolog v0.8.89`, intermediate artifacts can be found at `/tmp/cargo-installqEB7Fd`

Caused by:
  unable to get packages from source

Caused by:
  failed to parse manifest at `~/.cargo/registry/src/github.com-1ecc6299db9ec823/rug-1.6.0/Cargo.toml`

Caused by:
  editions are unstable

Caused by:
  feature `edition` is required

consider adding `cargo-features = ["edition"]` to the manifest

Per rust-lang/cargo#6113, I tried:

 $ cargo install --force cargo-outdated

Which yields:

    Updating registry `https://github.com/rust-lang/crates.io-index`
error: failed to parse manifest at `/home/mt/.cargo/registry/src/github.com-1ecc6299db9ec823/cargo-outdated-0.8.0/Cargo.toml`

Caused by:
  editions are unstable

Caused by:
  feature `edition` is required

consider adding `cargo-features = ["edition"]` to the manifest
@triska
Copy link
Contributor Author

triska commented Sep 6, 2019

$ cargo --version
cargo 1.28.0

Tried on a 2018 version of Ubuntu.

@mthom
Copy link
Owner

mthom commented Sep 6, 2019

Does adding cargo-features = ["edition"] to Cargo.toml help?

@triska
Copy link
Contributor Author

triska commented Sep 6, 2019

I have added this line to every section of Cargo.toml, and still get exactly the same error message.

@mthom
Copy link
Owner

mthom commented Sep 6, 2019

I'll try it on my Ubuntu when I get home.

@XVilka
Copy link
Contributor

XVilka commented Sep 12, 2019

@triska it's a problem of Ubuntu/Debian distributions. They are always too slow to update. Just use rustup, it works fine with all channels - see stable, beta, and nightly channels: #159

@triska
Copy link
Contributor Author

triska commented Sep 12, 2019

@XVilka: Thank you for looking into this. Could you please suggest a patch for the README that lets us more reliably build Scryer Prolog? For example, how do I best "use rustup"?

@triska
Copy link
Contributor Author

triska commented Sep 15, 2019

I have now successfully compiled Scryer Prolog with cargo 1.34, which ships with the newly released Debian buster (Debian 10), so I am closing this issue. Thank you for looking into it!

@triska triska closed this as completed Sep 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants