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

Issue 661447: First version of the new devserver client (Closed)

Created:
10 years, 9 months ago by rtc
Modified:
9 years, 7 months ago
Reviewers:
seano, adlr
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

First version of the new devserver client

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+45 lines, -117 lines) Patch
M src/platform/dev/buildutil.py View 1 chunk +0 lines, -101 lines 0 comments Download
M src/platform/dev/devserver.py View 3 chunks +6 lines, -16 lines 0 comments Download
A src/platform/dev/gmerge View 1 chunk +38 lines, -0 lines 1 comment Download
A src/platform/dev/static/pkgroot View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
rtc
10 years, 9 months ago (2010-03-03 05:03:47 UTC) #1
adlr
LGTM w/ comment http://codereview.chromium.org/661447/diff/1/4 File src/platform/dev/gmerge (right): http://codereview.chromium.org/661447/diff/1/4#newcode29 src/platform/dev/gmerge:29: ESCAPED_BOARD=$(python -c "import urllib; print urllib.quote('''${BOARD_NAME}''')") ...
10 years, 9 months ago (2010-03-03 05:07:26 UTC) #2
seano
10 years, 9 months ago (2010-03-03 05:35:14 UTC) #3
lgtm

Powered by Google App Engine
This is Rietveld 408576698