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

Issue 1560029: Fix File() to work with SVN 1.4 by using svn export in place... (Closed)

Created:
10 years, 8 months ago by tony
Modified:
9 years, 7 months ago
Reviewers:
M-A Ruel
CC:
chromium-reviews, M-A Ruel
Visibility:
Public.

Description

Fix File() to work with SVN 1.4 by using svn export in place of svn co --depth. This should work even if you change svn versions between gclient syncs. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=44441

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+134 lines, -16 lines) Patch
M gclient_scm.py View 1 2 1 chunk +24 lines, -9 lines 0 comments Download
M scm.py View 1 2 2 chunks +20 lines, -0 lines 0 comments Download
M tests/gclient_scm_test.py View 1 2 3 3 chunks +85 lines, -3 lines 0 comments Download
M tests/scm_unittest.py View 2 1 chunk +5 lines, -4 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
tony
10 years, 8 months ago (2010-04-13 04:47:33 UTC) #1
M-A Ruel
Can you make SVN.AssertVersion cache the current installed version? lgtm with that.
10 years, 8 months ago (2010-04-13 19:43:54 UTC) #2
tony
10 years, 8 months ago (2010-04-14 02:35:54 UTC) #3
On 2010/04/13 19:43:54, Marc-Antoine Ruel wrote:
> Can you make SVN.AssertVersion cache the current installed version?

Done.

> lgtm with that.

Thanks!

Powered by Google App Engine
This is Rietveld 408576698