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

Issue 3604011: AU update generator script: support for new style full updates (Closed)

Created:
10 years, 2 months ago by adlr
Modified:
9 years ago
Reviewers:
petkov
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Base URL:
ssh://git@chromiumos-git/crosutils.git
Visibility:
Public.

Description

AU update generator script: support for new style full updates A corresponding CL adds support for the delta generator to generate new style full updates. This CL adds support for creating new style full updates to the common update payload generator script. This does not change the defaults, so generating a full update still uses the old style. BUG=7248 TEST=generated/applied full update on host Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=45c24df

Patch Set 1 #

Total comments: 1

Patch Set 2 : fix for review #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -20 lines) Patch
M cros_generate_update_payload View 1 3 chunks +33 lines, -20 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
adlr
10 years, 2 months ago (2010-10-07 02:35:24 UTC) #1
petkov
10 years, 2 months ago (2010-10-07 04:23:35 UTC) #2
LGTM

http://codereview.chromium.org/3604011/diff/1/2
File cros_generate_update_payload (right):

http://codereview.chromium.org/3604011/diff/1/2#newcode193
cros_generate_update_payload:193: SRC_MNT=$(mktemp -d /tmp/src_root.XXXXXX)
You could move this code down into the "then" block below.

Powered by Google App Engine
This is Rietveld 408576698