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

thread 'main' panicked at 'called Result::unwrap() on an Err value: ()' #203

Closed
Boscop opened this issue Feb 11, 2020 · 7 comments
Closed

Comments

@Boscop
Copy link

Boscop commented Feb 11, 2020

Running it in my workspace doesn't seem to work:

cargo outdated -d 1
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: ()', C:\Users\me\.cargo\registry\src\git.luolix.top-1ecc6299db9ec823\cargo-0.42.0\src/cargo\util\canonical_url.rs:42:13
stack backtrace:
   0: Ordinal0
   1: Ordinal0
   2: Ordinal0
   3: Ordinal0
   4: Ordinal0
   5: Ordinal0
   6: Ordinal0
   7: Ordinal0
   8: Ordinal0
   9: Ordinal0
  10: Ordinal0
  11: Ordinal0
  12: Ordinal0
  13: Ordinal0
  14: Ordinal0
  15: Ordinal0
  16: Ordinal0
  17: Ordinal0
  18: Ordinal0
  19: Ordinal0
  20: Ordinal0
  21: git_oidarray_free
  22: BaseThreadInitThunk
  23: RtlUserThreadStart
@deg4uss3r
Copy link
Collaborator

Hi @Boscop, what version of cargo-outdated are you running (cargo outdated —version)?

Thanks!

@Boscop
Copy link
Author

Boscop commented Feb 11, 2020

@deg4uss3r 0.9.5

@deg4uss3r
Copy link
Collaborator

Thanks @Boscop, did you install as follows:

cargo install --git https://github.com/kbknapp/cargo-outdated --force --locked

I caught the --locked being left off the documentation this morning and updated that. We have to lock the url dependency that cargo is using to an older version until ssh parsing is resolved. Reinstall with above and let me know if you're still seeing this error.

@Boscop
Copy link
Author

Boscop commented Feb 12, 2020

Thanks, that worked. So the url crate stopped accepting ssh urls?

When should I go back to using the crates.io version? When this #99 is closed?

@deg4uss3r
Copy link
Collaborator

No worries @Boscop

It's a parsing issue, the url crate changed the way it does ssh:// -> https:// parsing and broke the cargo API for this. The next cargo update (0.43.0) will have a work around for the underlying url crate change. Once that happens we will test and patch into crates.io (which currently is out-of-date at the moment as is).

@Boscop
Copy link
Author

Boscop commented Feb 24, 2020

@deg4uss3r Btw, do you have a link to the upstream issue? I can't seem to find it..

@deg4uss3r
Copy link
Collaborator

deg4uss3r commented Feb 24, 2020

@Boscop This is the upstream issue servo/rust-url#577; however, I did put in a PR for this rust-lang/cargo#7849 and was told the issue was fixed in staging and awaiting the v0.43.0 release.

I hope that helps!

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

2 participants