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

Issue 248011: gclient: don't make SVN-specific calls in RunOnDeps (Closed)

Created:
11 years, 3 months ago by Mandeep Singh Baines
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

gclient: don't make SVN-specific calls in RunOnDeps Replaced a call to CaptureSVNStatus with a SCMWrapper::status gclient: store gclient_entries as a dict Currently we only store the relative path for entries. We need to also store the url so that learn the repo type: git or snv. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27370

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -9 lines) Patch
M gclient.py View 3 chunks +9 lines, -9 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Mandeep Singh Baines
11 years, 3 months ago (2009-09-26 00:17:27 UTC) #1
M-A Ruel
DO NOT USE tells you: lgtm
11 years, 3 months ago (2009-09-26 00:26:54 UTC) #2
Lei Zhang
11 years, 2 months ago (2009-09-29 02:37:23 UTC) #3
FYI, I think this broke a bunch of depot_tool unit tests.

Powered by Google App Engine
This is Rietveld 408576698