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

Issue 6362008: Refactor devserver to print out relative paths to the update file (Closed)

Created:
9 years, 11 months ago by sosa
Modified:
9 years, 6 months ago
Reviewers:
dgarrett
CC:
chromium-os-reviews_chromium.org, Nick Sanders, ericli, DaleCurtis
Visibility:
Public.

Description

Refactor devserver to print out relative paths to the update file Change-Id: Idb98a1e2ff56716d1aadcfda6eb83108dff913f2 BUG=chromium-os:10723 TEST=Ran it with the full test harness -q Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=d3de9ef Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=417e55d

Patch Set 1 #

Patch Set 2 : Nits #

Patch Set 3 : Fix refactoring bug introducing before push #

Unified diffs Side-by-side diffs Delta from patch set Stats (+62 lines, -57 lines) Patch
M autoupdate.py View 1 2 8 chunks +56 lines, -56 lines 0 comments Download
M devserver.py View 1 2 3 chunks +6 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
sosa
Also cleaned up some log messages (too many repeats when failures) and cleaned up the ...
9 years, 11 months ago (2011-01-20 20:48:19 UTC) #1
dgarrett
On 2011/01/20 20:48:19, sosa wrote: > Also cleaned up some log messages (too many repeats ...
9 years, 11 months ago (2011-01-20 21:56:03 UTC) #2
sosa
PTAL. This broke the build because of a refactor bug I didn't catch (once I ...
9 years, 11 months ago (2011-01-26 00:36:50 UTC) #3
sosa
9 years, 11 months ago (2011-01-26 00:40:16 UTC) #4
Don over the shouldered re-LGTM'd before heading out.

Powered by Google App Engine
This is Rietveld 408576698