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

Issue 661005: Fix autoupdate to be backwards compatible with previous releases. (Closed)

Created:
10 years, 10 months ago by Charlie Lee
Modified:
9 years, 7 months ago
Reviewers:
petkov
CC:
chromium-os-reviews_googlegroups.com
Visibility:
Public.

Description

Fix autoupdate to be backwards compatible with previous releases. If the "board" attribute is not set, then assume "x86-generic"

Patch Set 1 #

Patch Set 2 : fix extra space #

Total comments: 4

Patch Set 3 : fix syntax #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -8 lines) Patch
M src/platform/dev/autoupdate.py View 1 2 4 chunks +13 lines, -8 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
petkov
http://codereview.chromium.org/661005/diff/1001/4 File src/platform/dev/autoupdate.py (right): http://codereview.chromium.org/661005/diff/1001/4#newcode98 src/platform/dev/autoupdate.py:98: client_version = query.getAttribute('version'); No semicolon at the end http://codereview.chromium.org/661005/diff/1001/4#newcode99 ...
10 years, 10 months ago (2010-02-24 20:42:44 UTC) #1
Charlie Lee
I went ahead and fixed the other 80 chars issues. http://codereview.chromium.org/661005/diff/1001/4 File src/platform/dev/autoupdate.py (right): http://codereview.chromium.org/661005/diff/1001/4#newcode98 ...
10 years, 10 months ago (2010-02-24 20:54:10 UTC) #2
petkov
10 years, 10 months ago (2010-02-24 20:59:39 UTC) #3
LGTM. Thanks and sorry about the confusion.

Powered by Google App Engine
This is Rietveld 408576698