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

Unable to parse lock file #11

Closed
jcholder-thoughtworks opened this issue Feb 7, 2020 · 4 comments · Fixed by #12
Closed

Unable to parse lock file #11

jcholder-thoughtworks opened this issue Feb 7, 2020 · 4 comments · Fixed by #12

Comments

@jcholder-thoughtworks
Copy link

Howdy! I get the following error when I try to run cargo instruments (no arguments):

Failed failed to parse lock file at: /[redacted but correct]/Cargo.lock.

The Cargo.lock is freshly generated from a new project created a couple weeks ago.

The full Cargo.lock file.

cargo version: cargo 1.42.0-nightly (9d32b7b01 2020-01-26).
rustc version: rustc 1.42.0-nightly (3761dcd34 2020-01-28).

Environment: macOS Mojave 10.14.6.

Initial guess is that the cargo crate in this crate needs to be updated to the current version, 0.42.0.

Glad to provide additional info as needed. Thanks!

@jcholder-thoughtworks
Copy link
Author

Heads up that cargo versions 0.36.0 and higher have breaking changes to the API.

@cmyr
Copy link
Owner

cmyr commented Feb 7, 2020

@jcholder-thoughtworks gotcha, thanks for opening the issue. To check my understanding, this is about the new lockfile format; the new cargo can work with the old format, but old cargo doesn't work with the new format?

Just want to make sure that updating cargo doesn't have any unintended consequences.

@cmyr
Copy link
Owner

cmyr commented Feb 7, 2020

@jcholder-thoughtworks I've just pushed a patch would you like to try it out? You'll have to check it out locally and then do cargo install --force --path=. (assuming this is executed from the source directory). If this is working for you, I'll do a crates.io release.

@jcholder-thoughtworks
Copy link
Author

@cmyr Success! Thanks!

@cmyr cmyr closed this as completed in #12 Feb 7, 2020
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

Successfully merging a pull request may close this issue.

2 participants