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

Issue 6487003: Update to chromite wrapper to properly report import errors.... (Closed)

Created:
9 years, 10 months ago by diandersAtChromium
Modified:
9 years, 6 months ago
CC:
chromium-reviews, M-A Ruel, sjg
Visibility:
Public.

Description

Update to chromite wrapper to properly report import errors. The recent changes to the chromite wrapper hid all import errors in chromite, simply giving an error that chromite couldn't be found. This fixes the problem. For associated review in chromiumos git: http://codereview.chromium.org/6461009

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -0 lines) Patch
M chromite View 1 chunk +10 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
commit-bot: I haz the power
No reviewers yet.
9 years, 10 months ago (2011-02-10 17:57:15 UTC) #1
diandersAtChromium
Marc, Sorry for yet another bother. Hopefully (fingers crossed!) this is the last change to ...
9 years, 10 months ago (2011-02-10 17:59:26 UTC) #2
M-A Ruel
lgtm
9 years, 10 months ago (2011-02-10 17:59:47 UTC) #3
commit-bot: I haz the power
9 years, 10 months ago (2011-02-10 18:02:55 UTC) #4
Change committed as 74452

Powered by Google App Engine
This is Rietveld 408576698