search

kbknapp / cargo-outdated

A cargo subcommand for displaying when Rust dependencies are out of date
MIT License
1.17k stars 86 forks source link

Failure depending on format of relative `path` value #122

Open sanmai-NL opened 6 years ago

sanmai-NL commented 6 years ago 
  cargo outdated

Workspace virtual manifest Cargo.toml:

[patch.crates-io]
dep = { path = 'dep' }
error: failed to resolve patches for `https://github.com/rust-lang/crates.io-index`

Caused by:
  failed to load source for a dependency on `dep`

Caused by:
  Unable to update file:///tmp/cargo-outdated.w1X4ihVEJEe6/dep

Caused by:
  failed to read `/tmp/cargo-outdated.w1X4ihVEJEe6/dep/Cargo.toml`

Caused by:
  No such file or directory (os error 2)

Workspace virtual manifest Cargo.toml:

[patch.crates-io]
dep = { path = './dep' }

Does work, but the first form already works with Cargo.

sanmai-NL commented 6 years ago

An additional or alternative failure is that virtual manifest patch tables aren’t honored at all in the first place. After rewriting all paths to include the current directory prefix ./, the issue persists when running cargo outdated on workspace members.

Frederick888 commented 6 years ago

Related to #118 ?