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

Issue 11000031: Copy README.dart-sdk to dart-sdk/README . (Closed)

Created:
8 years, 2 months ago by dgrove
Modified:
8 years, 2 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Copy README.dart-sdk to dart-sdk/README . Use this opportunity to get rid of create.stamp, and use the README file as a marker for the build. Fixes issue 2062 and is the last piece of fixing issue 4129. Committed: https://code.google.com/p/dart/source/detail?r=12980

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -3 lines) Patch
M dart.gyp View 2 chunks +2 lines, -2 lines 0 comments Download
M tools/create_sdk.py View 1 chunk +2 lines, -1 line 1 comment Download

Messages

Total messages: 5 (0 generated)
dgrove
8 years, 2 months ago (2012-09-27 17:37:43 UTC) #1
Siggi Cherem (dart-lang)
https://codereview.chromium.org/11000031/diff/1/tools/create_sdk.py File tools/create_sdk.py (right): https://codereview.chromium.org/11000031/diff/1/tools/create_sdk.py#newcode329 tools/create_sdk.py:329: Copy(join(HOME, 'README.dart-sdk'), join(SDK_tmp, 'README')) will this update the time-stamp ...
8 years, 2 months ago (2012-09-27 17:57:59 UTC) #2
dgrove
On 2012/09/27 17:57:59, sigmund wrote: > https://codereview.chromium.org/11000031/diff/1/tools/create_sdk.py > File tools/create_sdk.py (right): > > https://codereview.chromium.org/11000031/diff/1/tools/create_sdk.py#newcode329 > ...
8 years, 2 months ago (2012-09-27 18:01:21 UTC) #3
dgrove
8 years, 2 months ago (2012-09-27 18:01:25 UTC) #4
Siggi Cherem (dart-lang)
8 years, 2 months ago (2012-09-27 18:06:20 UTC) #5
lgtm

Powered by Google App Engine
This is Rietveld 408576698