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

Unified Diff: tool/build_sdk.sh

Issue 2249233002: fix #626, add AMD module format and make it default (Closed) Base URL: git@github.com:dart-lang/dev_compiler.git@master
Patch Set: merged Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/not_yet_strong_tests.dart ('k') | tool/build_test_pkgs.sh » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tool/build_sdk.sh
diff --git a/tool/build_sdk.sh b/tool/build_sdk.sh
index 156e82672dc15d134a902afb06cd06d20327f317..71f6db4e4c3498406c369667515782b802323829 100755
--- a/tool/build_sdk.sh
+++ b/tool/build_sdk.sh
@@ -9,14 +9,14 @@ dart -c tool/patch_sdk.dart tool/input_sdk gen/patched_sdk
echo "*** Compiling SDK to JavaScript"
# TODO(jmesserly): break out dart:html & friends.
-#
-# Right now we can't summarize our SDK, so we can't treat it as a normal
-# explicit input (instead we're implicitly compiling against the user's SDK).
-#
-# Another possible approach is to hard code the dart:* library->module mapping
-# into the compiler itself, so it can emit the correct import.
-#
dart -c tool/build_sdk.dart \
--dart-sdk gen/patched_sdk \
- -o lib/runtime/dart_sdk.js \
+ --modules=amd \
+ -o lib/js/amd/dart_sdk.js \
+ --modules=es6 \
+ -o lib/js/es6/dart_sdk.js \
+ --modules=common \
+ -o lib/js/common/dart_sdk.js \
+ --modules=legacy \
+ -o lib/js/legacy/dart_sdk.js \
"$@" > tool/sdk_expected_errors.txt
« no previous file with comments | « test/not_yet_strong_tests.dart ('k') | tool/build_test_pkgs.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698