-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Update git2 to 0.7.0 #5091
Update git2 to 0.7.0 #5091
Conversation
r? @matklad (rust_highfive has picked a reviewer for you, use r? to override) |
@bors r+ |
📌 Commit ca99f8f has been approved by |
⌛ Testing commit ca99f8fb618520e4504160dccfbb7fafd2336ec4 with merge 2b0885e2308f104162b095df0d8cba1e1bce9ac6... |
@@ -28,16 +28,16 @@ failure = "0.1.1" | |||
filetime = "0.1" | |||
flate2 = "1.0" | |||
fs2 = "0.4" | |||
git2 = "0.6.11" | |||
git2-curl = "0.7" | |||
git2 = "0.7.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't know if this is intentional or not, but looks like /~https://github.com/alexcrichton/git2-rs does not contain commit which actually bumps the versions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops forgot to do git push
Verified that it actually does work on a win7 virtualbox! |
CI failure seems legit: https://ci.appveyor.com/project/rust-lang-libs/cargo/build/1.0.3722#L1458 |
💔 Test failed - status-appveyor |
ca99f8f
to
a85c917
Compare
@bors: r=matklad |
📌 Commit a85c917 has been approved by |
☀️ Test successful - status-appveyor, status-travis |
cc #5066