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

Issue 6575022: Revert "RunCommand was catching and rethrowing an exception, which seemed to generate" (Closed)

Created:
9 years, 10 months ago by Luigi Semenzato
Modified:
9 years, 7 months ago
Reviewers:
davidjames, dgarrett
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

Revert "RunCommand was catching and rethrowing an exception, which seemed to generate" This reverts commit 01c84235827527cd2f938897a4a8dfcdbf877281. BUG=chromium-os:11717 TEST=none Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=3635aaa

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -102 lines) Patch
M lib/cros_build_lib.py View 1 chunk +19 lines, -26 lines 0 comments Download
D lib/cros_build_lib_unittest.py View 1 chunk +0 lines, -76 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Luigi Semenzato
This broke the build. http://build.chromium.org/p/chromiumos/builders/x86%20pineview%20binary/builds/3323/steps/cbuildbot/logs/stdio
9 years, 10 months ago (2011-02-24 03:12:52 UTC) #1
davidjames
9 years, 10 months ago (2011-02-24 03:14:26 UTC) #2
LGTM

Root cause of bug is that the new code sometimes throws exceptions when
exit_code=True, when it's just supposed to return the error code. Shouldn't be
hard to fix, but let's save that for regular working hours.

Powered by Google App Engine
This is Rietveld 408576698