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

Issue 1601373002: SDK gets analyzer sources. (Closed)

Created:
4 years, 11 months ago by pquitslund
Modified:
4 years, 10 months ago
CC:
reviews_dartlang.org, ricow1
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

SDK gets analyzer sources. With this change, analyzer and server sources will appear in `sdk/lib`. Having the sources in the SDK is a prerequisite to consuming them in a spawned Isolate (needed for dynamic analyzer plugin loading). R=brianwilkerson@google.com, whesse@google.com Committed: https://github.com/dart-lang/sdk/commit/68f077c47bfd782f1599725ce046bd8129f1e7ec

Patch Set 1 #

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

Messages

Total messages: 7 (2 generated)
pquitslund
4 years, 11 months ago (2016-01-19 22:13:41 UTC) #2
Brian Wilkerson
LGTM, but I could easily miss problems.
4 years, 11 months ago (2016-01-19 22:20:45 UTC) #3
Bill Hesse
LGTM. Please check on how much the size of the Dart SDK increases with this ...
4 years, 11 months ago (2016-01-20 18:51:39 UTC) #4
pquitslund
Committed patchset #1 (id:1) manually as 68f077c47bfd782f1599725ce046bd8129f1e7ec (presubmit successful).
4 years, 10 months ago (2016-02-03 17:37:31 UTC) #6
pquitslund
4 years, 10 months ago (2016-02-03 17:38:10 UTC) #7
Message was sent while issue was closed.
Thanks!

https://codereview.chromium.org/1601373002/diff/1/tools/create_sdk.py
File tools/create_sdk.py (right):

https://codereview.chromium.org/1601373002/diff/1/tools/create_sdk.py#newcode41
tools/create_sdk.py:41: # ....lib/
On 2016/01/20 18:51:39, Bill Hesse wrote:
> Please update this comment to show the new directories.

Done.

Powered by Google App Engine
This is Rietveld 408576698