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

Issue 3231007: Add 9999 revving logic to cros_mark_all_as_stable. (Closed)

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

Description

Add 9999 revving logic to cros_mark_all_as_stable. TEST=Ran it and saw appropriate changes. Change-Id: I5f51e16918993b9dbf53b38cb3a365fafe77c8eb

Patch Set 1 #

Patch Set 2 : ws #

Total comments: 1

Patch Set 3 : Fix egrep command #

Patch Set 4 : Fix comment #

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

Messages

Total messages: 2 (0 generated)
sosa
10 years, 3 months ago (2010-08-30 17:21:28 UTC) #1
Mandeep Singh Baines
10 years, 3 months ago (2010-08-30 18:27:57 UTC) #2
LGTM but suggested possible alternative to consider.

http://codereview.chromium.org/3231007/diff/2001/3001
File cros_mark_all_as_stable (right):

http://codereview.chromium.org/3231007/diff/2001/3001#newcode86
cros_mark_all_as_stable:86: grep -c [KEYWORDS,CROS_WORKON_COMMIT]) -ne 3 ]; then
This seems a little fragile. What if instead you tested whether, or not,
converting the 9999 ebuild to stable resulted in a diff? iow, would this 9999
ebuild have generated the current stable.

Powered by Google App Engine
This is Rietveld 408576698