Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Update README.md #4762

Merged
merged 2 commits into from
Mar 6, 2017
Merged

Update README.md #4762

merged 2 commits into from
Mar 6, 2017

Conversation

jdetychey
Copy link
Contributor

Fix for Cargo fail when building from source

Fix for cargo fail when building from source
@parity-cla-bot
Copy link

It looks like @jdetychey signed our Contributor License Agreement. 👍

Many thanks,

Ethcore CLA Bot

@jdetychey
Copy link
Contributor Author

Build from source with:

$ cargo build --release

Can lead to this error:

failed to parse manifest at `~/parity/Cargo.toml`
Caused by:
  could not parse input as TOML
Cargo.toml:58:9 expected a key but found an empty string
Cargo.toml:58:9-58:10 expected `.`, but found `'`

The one line installer of rust leads to multiple version of cargo.:

$ curl https://sh.rustup.rs -sSf | sh

Notably:

cargo --version
cargo 0.8.0 (built 2016-03-22)

and:

$ ~/.cargo/bin/cargo build --version
cargo 0.16.0-nightly (6e0c18c 2017-01-27)

Therefore using instead:

~/.cargo/bin/cargo build --release

Fix it

@gavofyork gavofyork added the A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). label Mar 6, 2017
@gavofyork gavofyork merged commit 944dcdc into openethereum:master Mar 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants