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

Issue 700123002: dart2js: Don't emit additional precompiled file. (Closed)

Created:
6 years, 1 month ago by floitsch
Modified:
6 years, 1 month ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Rebase after revert. #

Patch Set 3 : Updated pub tests. #

Patch Set 4 : Update non-generated files. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -44 lines) Patch
M pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart View 1 1 chunk +0 lines, -28 lines 0 comments Download
M sdk/lib/_internal/pub/test/build/copies_browser_js_next_to_entrypoints_test.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub/test/build/copies_browser_js_with_dependency_override_test.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub/test/build/copies_browser_js_with_dev_dependency_test.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub/test/build/outputs_results_to_json_test.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub/test/dart2js/compiles_entrypoints_in_root_package_test.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub/test/dart2js/omits_source_map_in_release_test.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub/test/dart2js/outputs_deferred_libraries_test.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub/test/dart2js/supports_configuration_with_build_test.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub_generated/test/build/copies_browser_js_next_to_entrypoints_test.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub_generated/test/build/copies_browser_js_with_dependency_override_test.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub_generated/test/build/copies_browser_js_with_dev_dependency_test.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub_generated/test/build/outputs_results_to_json_test.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub_generated/test/dart2js/compiles_entrypoints_in_root_package_test.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub_generated/test/dart2js/omits_source_map_in_release_test.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub_generated/test/dart2js/outputs_deferred_libraries_test.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub_generated/test/dart2js/supports_configuration_with_build_test.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 10 (2 generated)
floitsch
I intend to commit this CL early next week.
6 years, 1 month ago (2014-11-05 09:57:01 UTC) #2
Johnni Winther
lgtm
6 years, 1 month ago (2014-11-05 10:33:21 UTC) #3
floitsch
Committed patchset #1 (id:1) manually as 41657 (presubmit successful).
6 years, 1 month ago (2014-11-11 09:47:14 UTC) #4
floitsch
@Bob: I updated the pub tests. After this CL they expected too many files. PTAL.
6 years, 1 month ago (2014-11-11 13:18:54 UTC) #6
Bob Nystrom
You'll want to hand-edit the non-generated versions and then re-generate these pub sources. Just make ...
6 years, 1 month ago (2014-11-11 17:17:41 UTC) #7
floitsch
Can't you mark the generated files as read-only to avoid this? PTAL.
6 years, 1 month ago (2014-11-12 15:44:41 UTC) #8
Bob Nystrom
LGTM. Sorry for the annoyance here. > Can't you mark the generated files as read-only ...
6 years, 1 month ago (2014-11-12 17:30:09 UTC) #9
floitsch
6 years, 1 month ago (2014-11-13 12:16:08 UTC) #10
Message was sent while issue was closed.
Committed patchset #4 (id:60001) manually as 41711 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698