Include platform.dill files in sdk builds

sdk/BUILD.gn

Index: sdk/BUILD.gn
diff --git a/sdk/BUILD.gn b/sdk/BUILD.gn
index 116f72104a2791b6341687d1014a9df9ff85cf87..fa015d2ab6cbd21b355f81285fd8a915f58ce172 100644
--- a/sdk/BUILD.gn
+++ b/sdk/BUILD.gn
@@ -65,6 +65,8 @@ if (is_fuchsia ||
 # ......_internal/
 #.........spec.sum
 #.........strong.sum
+#.........web_full.dill

[Siggi Cherem (dart-lang), 2017/07/26 04:51:56]

I've been debating the names here, and I'm not even convinced of my current
choice. Let me know if you any suggestions or preferences.

The names I've considered:
 - vm_outline & dart2js_platform (& possibly ddc_platform one day)
 - vm_platform & dart2js_full 
 - server_outline & web_platform
 - vm_summary (summary is a name we use in the FE public APIs) & dart2js_full
 - vm_sdk_summary.dill, dart2js_sdk.dill

At this moment I'm not sure if dart2js and ddc will be able to share the same
code for the sdk libraries. If they do, "web" is a better choice.

[Siggi Cherem (dart-lang), 2017/07/26 19:52:39]

I continued iterating on this - I decided for now to keep the outline/platform
suffixes and use the target name as a prefix. We can iterate on this more
afterwards.

  vm_outline.dill
  dart2js_platform.dill

+#.........server_outline.dill
 #.........dev_compiler/
 # ......analysis_server/
 # ......analyzer/
@@ -444,6 +446,44 @@ copy("copy_analysis_summaries") {
   ]
 }
 
+# This rule copies .dill files to lib/_internal
+group("copy_dill_files") {
+  visibility = [ ":create_common_sdk" ]
+  deps = [
+    ":copy_libraries",
+    ":copy_vm_dill_files",
+    ":copy_dart2js_dill_files",

[zra, 2017/07/26 05:04:58]

The "platform" SDK shouldn't include the dart2js dill file. Instead,
create_common_sdk should depend on copy_vm_dill_files, and create_full_sdk
should depend on copy_dart2js_dill_files.

[Siggi Cherem (dart-lang), 2017/07/26 19:52:39]

Thanks Zach. Done.

+  ]
+}
+
+copy("copy_vm_dill_files") {
+  visibility = [ ":copy_dill_files" ]
+  deps = [
+    ":copy_libraries",
+    "../runtime/vm:patched_sdk",
+  ]
+  sources = [
+    "$root_out_dir/patched_sdk/outline.dill",
+  ]
+  outputs = [
+    "$root_out_dir/dart-sdk/lib/_internal/server_{{source_file_part}}",
+  ]
+}
+
+copy("copy_dart2js_dill_files") {
+  visibility = [ ":copy_dill_files" ]
+  deps = [
+    ":copy_libraries",
+    "../utils/compiler:patched_dart2js_sdk",
+  ]
+  sources = [
+    "$root_out_dir/patched_dart2js_sdk/platform.dill",
+  ]
+  outputs = [
+    "$root_out_dir/dart-sdk/lib/_internal/web_full.dill",
+  ]
+}
+
 # This rule copies ddc summaries to lib/_internal
 copy("copy_dev_compiler_summary") {
   visibility = [ ":copy_dev_compiler_sdk" ]
@@ -704,6 +744,7 @@ group("create_common_sdk") {
     ":copy_api_readme",
     ":copy_dart",
     ":copy_dartdoc_files",
+    ":copy_dill_files",
     ":copy_headers",
     ":copy_libraries_dart",
     ":copy_license",