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

Issue 7111045: Add --depth option to repo. (Closed)

Created:
9 years, 6 months ago by diandersAtChromium
Modified:
9 years, 6 months ago
Reviewers:
msb, sop, M-A Ruel, anush
CC:
chromium-reviews, Dirk Pranke
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -8 lines) Patch
M repo View 1 3 chunks +6 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
diandersAtChromium
See related repo change: https://review.source.android.com/#change,22722 I just manually made the changes to the repo wrapper. ...
9 years, 6 months ago (2011-06-14 04:16:13 UTC) #1
diandersAtChromium
...shoot--this might have been premature. I didn't actually test that the '--depth' worked properly. Let ...
9 years, 6 months ago (2011-06-14 04:17:46 UTC) #2
diandersAtChromium
OK, it worked fine. ...but I also realized that I should update the 'repo' wrapper ...
9 years, 6 months ago (2011-06-14 04:53:21 UTC) #3
M-A Ruel
Make sure you grab the latest version of repo and not simply apply your patch. ...
9 years, 6 months ago (2011-06-14 12:33:45 UTC) #4
M-A Ruel
On 2011/06/14 12:33:45, Marc-Antoine Ruel wrote: > Make sure you grab the latest version of ...
9 years, 6 months ago (2011-06-16 13:09:53 UTC) #5
diandersAtChromium
9 years, 6 months ago (2011-06-16 15:44:42 UTC) #6
Marc-Antoine,

Can you commit for me?  I don't actually have write access for depot_tools.

-Doug

---

On Thu, Jun 16, 2011 at 6:09 AM, <maruel@chromium.org> wrote:

> On 2011/06/14 12:33:45, Marc-Antoine Ruel wrote:
>
>> Make sure you grab the latest version of repo and not simply apply your
>> patch.
>>
>
>  Rubberstamp lgtm, I'm not a repo reviewer.
>>
>
> The depot_tools commit queue is broken, please commit manually.
>
>
>
http://codereview.chromium.**org/7111045/<http://codereview.chromium.org/7111...
>

Powered by Google App Engine
This is Rietveld 408576698