nabijaczleweli / cargo-update

A cargo subcommand for checking and applying updates to installed executables
MIT License
1.22k stars 42 forks source link

Update cargo-update first #122

Closed ThinkChaos closed 4 years ago

ThinkChaos commented 4 years ago

I think it would be a good idea to always update cargo-update first when running cargo install-update --all.

nabijaczleweli commented 4 years ago

Hmm, any reasoning so as to why? It's not like we reexec from disk after updating, so I can't really see any obvious benefit to it?

ThinkChaos commented 4 years ago

Yeah sorry that wasn't quite clear. With the goal of starting the new version afterwards. It's just a thing that would be nice to have.

nabijaczleweli commented 4 years ago

I've thought about it and I don't think the added code complexity (especially for Windows self-updates/alt reexec path entirely) is worth the benefits, especially nowadays when this crate is mostly in maintenance mode, but thank you for the suggestion.