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

Issue 9124003: Dummy update to gypi file to force rebuild on buildbot (Closed)

Created:
8 years, 11 months ago by Søren Gjesse
Modified:
8 years, 11 months ago
Reviewers:
ahe
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Dummy update to gypi file to force rebuild on buildbot TBR=ahe@google.com BUG= TEST= Committed: https://code.google.com/p/dart/source/detail?r=3032

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M runtime/bin/builtin_sources.gypi View 1 chunk +1 line, -0 lines 2 comments Download

Messages

Total messages: 3 (0 generated)
Søren Gjesse
8 years, 11 months ago (2012-01-06 13:00:05 UTC) #1
ahe
LGTM :-) http://codereview.chromium.org/9124003/diff/1/runtime/bin/builtin_sources.gypi File runtime/bin/builtin_sources.gypi (right): http://codereview.chromium.org/9124003/diff/1/runtime/bin/builtin_sources.gypi#newcode61 runtime/bin/builtin_sources.gypi:61: 'globals.h', I think you need to remove ...
8 years, 11 months ago (2012-01-06 13:35:59 UTC) #2
Søren Gjesse
8 years, 11 months ago (2012-01-06 14:05:44 UTC) #3
http://codereview.chromium.org/9124003/diff/1/runtime/bin/builtin_sources.gypi
File runtime/bin/builtin_sources.gypi (right):

http://codereview.chromium.org/9124003/diff/1/runtime/bin/builtin_sources.gyp...
runtime/bin/builtin_sources.gypi:61: 'globals.h',
On 2012/01/06 13:35:59, ahe wrote:
> I think you need to remove this line ;-)

Why didn't you say this before :-?

Powered by Google App Engine
This is Rietveld 408576698