Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(108)

Issue 440273002: Reland: Add --no-history option to fetch and gclient for shallow clones. (Closed)

Created:
6 years, 4 months ago by Primiano Tucci (use gerrit)
Modified:
6 years, 4 months ago
Reviewers:
iannucci, wtc, szager1
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, iannucci+depot_tools_chromium.org, ilevy-cc_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Project:
tools
Visibility:
Public.

Description

Reland: Add --no-history option to fetch and gclient for shallow clones. Original CL: https://codereview.chromium.org/440263002/ Many people* have complained on chromium-dev about the long times required to perform a full fetch over a DSL. This seems to be mostly due to the huge size of chromium's history (~9 GB). On the other side, not everybody is interested in downloading the full git history of the projects. The size of git packs required to fetch a working HEAD is one order of magnitude smaller (1.5 GB). This change makes it possible to perform a shallow fetch (in a way which is consistent with DEPS, leveraging git templates on clone), reducing fetch times by 80% for those not interested in the history. * See: [chromium-dev] "fetch chromium" keeps hanging/getting stuck on Windows 7 [chromium-dev] Initial checkout with git taking long [chromium-dev] Trying to get latest source code fails when fetching [chromium-dev] Gclient sync takes too long BUG=228996 TBR=iannucci@chromium.org,szager@chromium.org,wtc@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=287793

Patch Set 1 #

Patch Set 2 : ensure_value -> hasattr #

Unified diffs Side-by-side diffs Delta from patch set Stats (+85 lines, -4 lines) Patch
M fetch.py View 5 chunks +11 lines, -1 line 0 comments Download
M gclient.py View 1 chunk +3 lines, -0 lines 0 comments Download
M gclient_scm.py View 1 2 chunks +21 lines, -0 lines 0 comments Download
M gclient_utils.py View 1 chunk +5 lines, -0 lines 0 comments Download
M tests/gclient_scm_test.py View 3 chunks +4 lines, -3 lines 0 comments Download
M tests/gclient_smoketest.py View 1 chunk +41 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Primiano Tucci (use gerrit)
Relanding the --no-history CL (original: crrev.com/437903002) TBR-ing since this is a one line diff from ...
6 years, 4 months ago (2014-08-06 17:00:36 UTC) #1
Primiano Tucci (use gerrit)
The CQ bit was checked by primiano@chromium.org
6 years, 4 months ago (2014-08-06 17:02:07 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/primiano@chromium.org/440273002/1
6 years, 4 months ago (2014-08-06 17:02:19 UTC) #3
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 4 months ago (2014-08-06 17:04:11 UTC) #4
commit-bot: I haz the power
Presubmit check for 440273002-1 failed and returned exit status 1. Running presubmit commit checks ...
6 years, 4 months ago (2014-08-06 17:04:12 UTC) #5
Primiano Tucci (use gerrit)
The CQ bit was checked by primiano@chromium.org
6 years, 4 months ago (2014-08-06 17:16:14 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/primiano@chromium.org/440273002/20001
6 years, 4 months ago (2014-08-06 17:16:33 UTC) #7
commit-bot: I haz the power
6 years, 4 months ago (2014-08-06 17:18:20 UTC) #8
Message was sent while issue was closed.
Change committed as 287793

Powered by Google App Engine
This is Rietveld 408576698