Change update logic to be backwards compatible with old format.

Change update logic to be backwards compatible with old format.

The key change is transforming var/...* to var_new/...* and dev_image/...* to dev_image_new/...* so that the tar file can be directly extracted to the stateful partition and have the same format for chromeos_startup as the old format.

Change-Id: I0148ae76bd664e3cc4b7762c23513ceac38d517e

BUG=8885
TEST=Tested with other CL by running stateful_update on a VM

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=4195fad

[sosa, 2010-11-09 21:27:48]

Bonus ... less lines of code.

[petkov, 2010-11-09 21:31:12]

http://codereview.chromium.org/4755001/diff/1/autoupdate.py
File autoupdate.py (right):

http://codereview.chromium.org/4755001/diff/1/autoupdate.py#newcode264
autoupdate.py:264: '--transform="s,dev_image,dev_image_new,"',
Maybe s,^dev_image,... and s,^var,... Otherwise if any filename under dev_image
contains the "var" substring it will be renamed.

Also, you can probably do a single sed expression with semicolon but I'm not
sure that would improve readability.

[sosa, 2010-11-09 21:51:46]

PTAL

http://codereview.chromium.org/4755001/diff/1/autoupdate.py
File autoupdate.py (right):

http://codereview.chromium.org/4755001/diff/1/autoupdate.py#newcode264
autoupdate.py:264: '--transform="s,dev_image,dev_image_new,"',
Good call on first.

I personally prefer what's there ... i thought about using one expression but
since this is easier to read I decided to stick with it

On 2010/11/09 21:31:12, petkov wrote:
> Maybe s,^dev_image,... and s,^var,... Otherwise if any filename under
dev_image
> contains the "var" substring it will be renamed.
> 
> Also, you can probably do a single sed expression with semicolon but I'm not
> sure that would improve readability.
>

[sosa, 2010-11-09 22:41:42]

PTAL, removed . as it was wrong since I copy the individual folders.

Added test= stanza as well

[petkov, 2010-11-09 22:46:31]

LGTM