Skip to content

Reduce git clone --depth from 50 (default) to 1 #35085

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Aug 1, 2016

Conversation

dns2utf8
Copy link
Contributor

This should speedup the builds a little.

@rust-highfive
Copy link
Contributor

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@dns2utf8
Copy link
Contributor Author

dns2utf8 commented Jul 28, 2016

This change reduces the clone size from ~90MB to ~60MB, cloning 8'460 instead of 84'595 objects.

The build time seems to be the same

@alexcrichton
Copy link
Member

@bors: r+ ca25966

Thanks!

@alexcrichton
Copy link
Member

@bors: rollup

@dns2utf8 dns2utf8 closed this Jul 29, 2016
@dns2utf8 dns2utf8 deleted the git_clone_depth branch July 29, 2016 23:06
@dns2utf8 dns2utf8 restored the git_clone_depth branch July 30, 2016 14:30
@dns2utf8 dns2utf8 reopened this Jul 30, 2016
@dns2utf8
Copy link
Contributor Author

Sorry, I did not know that removing the branch would automatically close the PR

sanxiyn added a commit to sanxiyn/rust that referenced this pull request Aug 1, 2016
…chton

Reduce git clone --depth from 50 (default) to 1

This should speedup the builds a little.
bors added a commit that referenced this pull request Aug 1, 2016
Rollup of 8 pull requests

- Successful merges: #34802, #35033, #35085, #35114, #35134, #35140, #35141, #35157
- Failed merges:
@bors bors merged commit ca25966 into rust-lang:master Aug 1, 2016
@dns2utf8 dns2utf8 deleted the git_clone_depth branch August 2, 2016 09:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants