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

Issue 1200993002: Update README.md and HACKING.md and resulting yak shave. (Closed)

Created:
5 years, 6 months ago by Hixie
Modified:
5 years, 6 months ago
Reviewers:
eseidel
CC:
abarth-chromium, gregsimon, jackson_old, mojo-reviews_chromium.org, qsr+mojo_chromium.org, yzshen+watch_chromium.org
Base URL:
https://github.com/domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Update README.md and HACKING.md and resulting yak shave. Update sky_server: - update its help string to admit that it wants the package path as well. - be quiet by default, since it makes the build chatty. - have a -v for verbose mode, for people who like it chatty. - have a default port of 8000, since that's what everyone used. - have a -p option to set the port, for people who don't use the default. - remove its mapping for gen/, since a TODO told me to. - remove the dependency on the build configuration, since it's no longer used. - teach it about .dart files. - rename sky_server/README to README.md, for consistency with other READMEs. Update shelldb to speak the new sky_server protocol. Update tests/services/iframe-embed-vmc.sky since the sky_server change broke the hack it was using. Update skyserver.py to speak the new sky_server protocol. Make webkitpy use SkyServer instead of spawning sky_server itself, since a TODO told me to. Teach webkitpy about dart files. Teach test_perf about dart files. (untested) Teach sky_tool about main.dart, main.sky is deprecated. (untested) R=eseidel@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/2914b244c54a5207d7768eaad1e9399001448d82

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+119 lines, -149 lines) Patch
M sky/HACKING.md View 2 chunks +31 lines, -39 lines 0 comments Download
M sky/README.md View 1 chunk +10 lines, -3 lines 0 comments Download
M sky/sdk/lib/sky_tool View 3 chunks +4 lines, -4 lines 0 comments Download
M sky/tests/services/iframe-embed-vmc.sky View 1 chunk +1 line, -4 lines 0 comments Download
M sky/tools/shelldb View 1 chunk +1 line, -4 lines 0 comments Download
D sky/tools/skygo/README View 1 chunk +0 lines, -19 lines 1 comment Download
A + sky/tools/skygo/README.md View 1 chunk +1 line, -1 line 0 comments Download
M sky/tools/skygo/linux64/sky_server.sha1 View 1 chunk +1 line, -1 line 0 comments Download
M sky/tools/skygo/sky_server.go View 2 chunks +42 lines, -33 lines 0 comments Download
M sky/tools/skypy/skyserver.py View 2 chunks +2 lines, -4 lines 0 comments Download
M sky/tools/test_perf View 1 chunk +1 line, -2 lines 0 comments Download
M sky/tools/webkitpy/layout_tests/controllers/manager.py View 2 chunks +3 lines, -3 lines 0 comments Download
M sky/tools/webkitpy/layout_tests/port/android.py View 2 chunks +5 lines, -5 lines 0 comments Download
M sky/tools/webkitpy/layout_tests/port/base.py View 5 chunks +13 lines, -22 lines 0 comments Download
M sky/tools/webkitpy/layout_tests/port/mock_drt.py View 2 chunks +2 lines, -2 lines 0 comments Download
M sky/tools/webkitpy/layout_tests/port/test.py View 2 chunks +2 lines, -2 lines 0 comments Download
M sky/tools/webkitpy/layout_tests/views/printing.py View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 3 (1 generated)
eseidel
lgtm https://codereview.chromium.org/1200993002/diff/1/sky/tools/skygo/README File sky/tools/skygo/README (left): https://codereview.chromium.org/1200993002/diff/1/sky/tools/skygo/README#oldcode5 sky/tools/skygo/README:5: 1. cd sky/tools/skygo/linux64 (or your current platform) Did ...
5 years, 6 months ago (2015-06-22 20:37:53 UTC) #2
Hixie
5 years, 6 months ago (2015-06-22 20:39:22 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
2914b244c54a5207d7768eaad1e9399001448d82 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698