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

Issue 66643006: Fix version generation for dart2js (Closed)

Created:
7 years, 1 month ago by ricow1
Modified:
7 years, 1 month ago
Reviewers:
ahe, kasperl, kustermann
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fix version generation for dart2js Review notes: This is not at all nice code, and create_snapthot.dart can be rafactored to make this nicer. This is the least intrusive way I could do this. R=kasperl@google.com Committed: https://code.google.com/p/dart/source/detail?r=30138

Patch Set 1 #

Total comments: 4

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -24 lines) Patch
M utils/compiler/compiler.gyp View 1 chunk +2 lines, -24 lines 0 comments Download
M utils/compiler/create_snapshot.dart View 1 2 chunks +24 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
ricow1
7 years, 1 month ago (2013-11-09 11:15:04 UTC) #1
kasperl
LGTM. https://codereview.chromium.org/66643006/diff/1/utils/compiler/create_snapshot.dart File utils/compiler/create_snapshot.dart (right): https://codereview.chromium.org/66643006/diff/1/utils/compiler/create_snapshot.dart#newcode25 utils/compiler/create_snapshot.dart:25: import '${dart2js.toFilePath(windows: false)}' as dart2jsMain; Is this dead ...
7 years, 1 month ago (2013-11-10 11:41:18 UTC) #2
ricow1
https://codereview.chromium.org/66643006/diff/1/utils/compiler/create_snapshot.dart File utils/compiler/create_snapshot.dart (right): https://codereview.chromium.org/66643006/diff/1/utils/compiler/create_snapshot.dart#newcode25 utils/compiler/create_snapshot.dart:25: import '${dart2js.toFilePath(windows: false)}' as dart2jsMain; On 2013/11/10 11:41:18, kasperl ...
7 years, 1 month ago (2013-11-10 12:01:59 UTC) #3
ricow1
7 years, 1 month ago (2013-11-10 12:02:11 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as r30138 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698