-
Notifications
You must be signed in to change notification settings - Fork 663
Upgrade many dependencies #458
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
Conversation
This was a bit involved, but wasn't too bad, thanks to @simonspain's work here: https://github.com/servo/rust-url/blob/master/UPGRADING.md
This took a bit of work, but wasn't _too_ bad.
I've pushed a commit for one of the larger ones: postgres and r2d2. If we want to try to do that separately, I'm happy to pull it apart; that's why I'm making them separate commits. |
Okay, this is about as far as I can go for now. conduit-rust/conduit#10 will help kickstart the condiut stuff. Bumping oauth2 to use the 0.3 series for curl will be helpful, but it's a bigger job, as the API changed a lot. |
yay for updating the entire conduit ecosystem! ❤️ @alexcrichton Lookin like this now:
All of these things are going to take a bit more effect, since there were big changes in |
Thanks @steveklabnik! |
❤️ :confetti-ball: On Tue, Oct 18, 2016 at 6:44 PM, Alex Crichton [email protected]
|
This PR brought to you by
cargo-outdated
.This upgrades many of our dependencies to the latest versions. Almost all of them are sevmer-compatible, and therefore, have no code changes. One notable difference is
URL
, which I've upgraded from pre-1.0 to post 1.0, which did take some small tweaks.Here's
cargo oudated
after:A lot of these are due to two things: 1, some things having actual new semver-incompatible versions. 2. conduit having old dependencies
@alexcrichton , any interest in getting conduit/civit upgraded a bit? I'm willing to send some PRs. Specifically, i'd like to someday get
semver
upgraded here as well.....