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

Issue 6760030: Print a grit debugging hint to stdout when a parse exception occurs (Closed)

Created:
9 years, 8 months ago by rhashimoto
Modified:
9 years, 7 months ago
CC:
chromium-reviews, pam+watch_chromium.org, Emmanuel Saint-loubert-Bié
Visibility:
Public.

Description

Print a grit debugging hint to stdout when a parse exception occurs to help with locating the error in the .grd file. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=80470

Patch Set 1 #

Patch Set 2 : Updated copyright year. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M tools/grit/grit/grd_reader.py View 1 2 chunks +3 lines, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
Jói
LGTM
9 years, 8 months ago (2011-04-05 15:34:28 UTC) #1
commit-bot: I haz the power
Presubmit check for 6760030-1 failed and returned exit status 1. Running presubmit commit checks ...
9 years, 8 months ago (2011-04-05 16:08:07 UTC) #2
Jói
rhashimoto@, please make sure you run your change through the trybots and that they complete ...
9 years, 8 months ago (2011-04-05 16:29:29 UTC) #3
Jói
Actually... the commit queue does this for you before committing. So for a simple change ...
9 years, 8 months ago (2011-04-05 16:31:18 UTC) #4
commit-bot: I haz the power
Change committed as 80470
9 years, 8 months ago (2011-04-05 17:04:40 UTC) #5
Jói
9 years, 8 months ago (2011-04-05 17:09:27 UTC) #6
Congratulations Roy, on your first change to Chrome.  Now make sure to
be available on the IRC channel or by IM for the next couple of hours,
and check http://build.chromium.org/p/chromium/console occasionally to
make sure your change seems to be building and passing tests OK.  Take
note that your change does not usually get compiled/tested on its own,
it usually gets grouped with others, and different bots on the
waterfall may group different sets of changes.  A nice way to
visualize this is to click the "merge" link at the bottom of the page.

If you have any questions, feel free to check with me on IM, or ask on
the IRC channel.

Cheers,
Jói


On Tue, Apr 5, 2011 at 1:04 PM,  <commit-bot@chromium.org> wrote:
> Change committed as 80470
>
> http://codereview.chromium.org/6760030/
>

Powered by Google App Engine
This is Rietveld 408576698