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

[fetch] use cli git for fetching crates.io index #334

Merged
merged 3 commits into from
Oct 9, 2019

Conversation

ordian
Copy link
Collaborator

@ordian ordian commented Sep 8, 2019

Fixes #333.
Cargo has a fair amount of code for git authentication handling (link), but still has a cli fallback (link).
See rust-lang/cargo#2078 for details on all the arguments passed to git.

@ordian ordian mentioned this pull request Sep 8, 2019
@ordian ordian requested a review from killercup September 16, 2019 15:17
@ordian
Copy link
Collaborator Author

ordian commented Oct 9, 2019

bors r+

bors bot added a commit that referenced this pull request Oct 9, 2019
334: [fetch] use cli git for fetching crates.io index r=ordian a=ordian

Fixes #333.
Cargo has a fair amount of code for git authentication handling ([link](https://github.com/rust-lang/cargo/blob/57986eac7157261c33f0123bade7ccd20f15200f/src/cargo/sources/git/utils.rs#L410-L624)), but still has a cli fallback ([link](https://github.com/rust-lang/cargo/blob/57986eac7157261c33f0123bade7ccd20f15200f/src/cargo/sources/git/utils.rs#L704-L714)).
See rust-lang/cargo#2078 for details on all the arguments passed to git.

Co-authored-by: Andronik Ordian <[email protected]>
@bors
Copy link
Contributor

bors bot commented Oct 9, 2019

Build failed

@ordian
Copy link
Collaborator Author

ordian commented Oct 9, 2019

bors r+

bors bot added a commit that referenced this pull request Oct 9, 2019
334: [fetch] use cli git for fetching crates.io index r=ordian a=ordian

Fixes #333.
Cargo has a fair amount of code for git authentication handling ([link](https://github.com/rust-lang/cargo/blob/57986eac7157261c33f0123bade7ccd20f15200f/src/cargo/sources/git/utils.rs#L410-L624)), but still has a cli fallback ([link](https://github.com/rust-lang/cargo/blob/57986eac7157261c33f0123bade7ccd20f15200f/src/cargo/sources/git/utils.rs#L704-L714)).
See rust-lang/cargo#2078 for details on all the arguments passed to git.

Co-authored-by: Andronik Ordian <[email protected]>
@bors
Copy link
Contributor

bors bot commented Oct 9, 2019

Build failed

@ordian
Copy link
Collaborator Author

ordian commented Oct 9, 2019

bors retry

bors bot added a commit that referenced this pull request Oct 9, 2019
334: [fetch] use cli git for fetching crates.io index r=ordian a=ordian

Fixes #333.
Cargo has a fair amount of code for git authentication handling ([link](https://github.com/rust-lang/cargo/blob/57986eac7157261c33f0123bade7ccd20f15200f/src/cargo/sources/git/utils.rs#L410-L624)), but still has a cli fallback ([link](https://github.com/rust-lang/cargo/blob/57986eac7157261c33f0123bade7ccd20f15200f/src/cargo/sources/git/utils.rs#L704-L714)).
See rust-lang/cargo#2078 for details on all the arguments passed to git.

Co-authored-by: Andronik Ordian <[email protected]>
@bors
Copy link
Contributor

bors bot commented Oct 9, 2019

@bors bors bot merged commit f8d1ecc into killercup:master Oct 9, 2019
@ordian ordian deleted the fix-git-fetch-via-ssh branch October 9, 2019 17:19
@ordian ordian mentioned this pull request Nov 14, 2019
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

Successfully merging this pull request may close these issues.

SSH support
1 participant