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

Issue 5265009: [crosutils] Add confirmation warning to cros_workon_make --scrub (Closed)

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

Description

[crosutils] Add confirmation warning to cros_workon_make --scrub BUG=9717 TEST=run cros_workon_make --scrub and ensure that it only scrubs if the user types y or yes (case insensitive) Change-Id: Ic1f8020fcca8bd57924391fcbf075d3e2adc5e4a Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=330375f

Patch Set 1 #

Patch Set 2 : quiet grep instead of redirecting #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -6 lines) Patch
M bin/cros_workon_make View 1 1 chunk +12 lines, -6 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Chris Masone
10 years ago (2010-11-29 22:59:25 UTC) #1
rginda
10 years ago (2010-11-29 23:06:19 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698