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

Issue 52003002: - Workaround SVN inconsistencies when building locally. (Closed)

Created:
7 years, 1 month ago by Ivan Posva
Modified:
7 years, 1 month ago
Reviewers:
devoncarew, kustermann
CC:
reviews_dartlang.org, turnidge
Visibility:
Public.

Description

- Workaround SVN inconsistencies when building locally. R=devoncarew@google.com Committed: https://code.google.com/p/dart/source/detail?r=29516

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M tools/utils.py View 1 chunk +4 lines, -0 lines 1 comment Download

Messages

Total messages: 4 (0 generated)
Ivan Posva
7 years, 1 month ago (2013-10-30 03:54:54 UTC) #1
devoncarew
lgtm
7 years, 1 month ago (2013-10-30 04:19:40 UTC) #2
Ivan Posva
Committed patchset #1 manually as r29516 (presubmit successful).
7 years, 1 month ago (2013-10-30 04:32:31 UTC) #3
kustermann
7 years, 1 month ago (2013-10-30 11:15:21 UTC) #4
Message was sent while issue was closed.
This is hackisch ivan. Other build scripts could invoke svn -- and all
invocations of svn will fail if the svn client version is not compatible with
the one used to checkout.

https://codereview.chromium.org/52003002/diff/1/tools/utils.py
File tools/utils.py (right):

https://codereview.chromium.org/52003002/diff/1/tools/utils.py#newcode305
tools/utils.py:305: if user != 'chrome-bot':
Where is 'user' defined?

Powered by Google App Engine
This is Rietveld 408576698