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

Issue 3212012: image_to_usb: List available disks when /dev/sdX is specified (Closed)

Created:
10 years, 3 months ago by Olof Johansson
Modified:
9 years, 7 months ago
Reviewers:
sosa
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

image_to_usb: List available disks when /dev/sdX is specified I'm lazy and sometimes copy and paste the output at the end of build_image verbatim and accidentally include the newline at the end. This gives a more useful output for that case. Change-Id: I1a4abefa884a91cb75dfe2779c79b3ef4b60e807 BUG=none TEST=./image_to_usb --from=../build/images/x86-generic/latest --to=/dev/sdX (and /dev/sde)

Patch Set 1 #

Patch Set 2 : Just change the default #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M image_to_usb.sh View 1 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Olof Johansson
10 years, 3 months ago (2010-08-31 03:06:42 UTC) #1
sosa
Why not just change the default to /dev/sdX?
10 years, 3 months ago (2010-08-31 03:13:06 UTC) #2
Olof Johansson
Instead of also checking for empty --to? Good idea. Done, PTAL.
10 years, 3 months ago (2010-08-31 16:01:07 UTC) #3
sosa
10 years, 3 months ago (2010-08-31 18:53:14 UTC) #4
LGTM, nice

Powered by Google App Engine
This is Rietveld 408576698