Kill factory partition dead.

Don't allow booting of factory image after wipe.

BUG=6655
TEST=manual run

Change-Id: If774c7025005cb2f7c0cbe4209691aaf0aa6da7b

[Tom Wai-Hong Tam, 2010-09-11 07:16:32]

http://codereview.chromium.org/3362018/diff/8001/9002
File client/site_tests/factory_Wipe/src/switch_partitions.sh (right):

http://codereview.chromium.org/3362018/diff/8001/9002#newcode19
client/site_tests/factory_Wipe/src/switch_partitions.sh:19: ROOT_PART=$(echo
"${ROOT_DEV}" | sed -e 's/^.*\([0-9]\)$/\1/')
You can use substring operation: ROOT_PART=${ROOT_DEV:${#ROOT_DEV}-1:1}

http://codereview.chromium.org/3362018/diff/8001/9002#newcode32
client/site_tests/factory_Wipe/src/switch_partitions.sh:32: # Destroy this root
partition if we've successfully switched..
Two periods at the end.

[Nick Sanders, 2010-09-11 07:24:05]

http://codereview.chromium.org/3362018/diff/8001/9002
File client/site_tests/factory_Wipe/src/switch_partitions.sh (right):

http://codereview.chromium.org/3362018/diff/8001/9002#newcode19
client/site_tests/factory_Wipe/src/switch_partitions.sh:19: ROOT_PART=$(echo
"${ROOT_DEV}" | sed -e 's/^.*\([0-9]\)$/\1/')
not with dash you cant =(

http://codereview.chromium.org/3362018/diff/8001/9002#newcode32
client/site_tests/factory_Wipe/src/switch_partitions.sh:32: # Destroy this root
partition if we've successfully switched..
On 2010/09/11 07:16:33, Tom Wai-Hong Tam wrote:
> Two periods at the end.

Done.

[Tom Wai-Hong Tam, 2010-09-11 07:34:43]

http://codereview.chromium.org/3362018/diff/8001/9002
File client/site_tests/factory_Wipe/src/switch_partitions.sh (right):

http://codereview.chromium.org/3362018/diff/8001/9002#newcode19
client/site_tests/factory_Wipe/src/switch_partitions.sh:19: ROOT_PART=$(echo
"${ROOT_DEV}" | sed -e 's/^.*\([0-9]\)$/\1/')
:( How about using tail:
ROOT_PART=$(echo -n "${ROOT_DEV}" | tail -c 1)

[Nick Sanders, 2010-09-11 07:41:43]

I just copied the sed command from chromeos common. I'd prefer to use the
standard way if I can't use the nice shell substring way.

[Tom Wai-Hong Tam, 2010-09-11 07:48:28]

OK. LGTM.

[hungte, 2010-09-13 02:37:32]

http://codereview.chromium.org/3362018/diff/23001/24002
File client/site_tests/factory_Wipe/src/switch_partitions.sh (right):

http://codereview.chromium.org/3362018/diff/23001/24002#newcode20
client/site_tests/factory_Wipe/src/switch_partitions.sh:20: ROOT_PART=$(echo
"${ROOT_DEV}" | sed -e 's/^.*\([0-9]\)$/\1/')
sed -e 's/.*[^0-9]//' would be simpler.