Properly link building dev compiler tools to create_full_sdk.

sdk/BUILD.gn

 # Copyright (c) 2017, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 # This GN file contains build rules for assembling the Dart SDK. There are
 # two possible variants: the "Full" SDK, and the "Platform" SDK. If you want
 # to make a new subset of the Full SDK, make it the same way we make
 # the Platform SDK.
 #
 # Warning:
 # If you need to copy something into dart-sdk/lib/foo in addition to the stuff
 # copied there by :copy_libraries, then you must depend on ":copy_libraries",
 # or ":copy_libraries" may delete/overwrite your addition, and the build will
 # fail.
 
 import("../build/copy_tree.gni")
 
 declare_args() {
   # Build a SDK with less stuff. It excludes dart2js, ddc, and web libraries.
   dart_platform_sdk = false
 }
 
-if (is_fuchsia || is_fuchsia_host) {
+if (is_fuchsia ||
+    is_fuchsia_host ||
+    current_cpu == "arm64" ||
+    current_cpu == "arm") {
   dart_platform_sdk = true
 }
 
 # The directory layout of the SDK is as follows:
 #
 # ..dart-sdk/
 # ....bin/
 # ......dart or dart.exe (executable)
 # ......dart.lib (import library for VM native extensions on Windows)
 # ......dartdoc
@@ skipping 438 matching lines @@
     ":copy_dev_compiler_js",
   ]
   sources = [
     "../third_party/requirejs/require.js",
   ]
   outputs = [
     "$root_out_dir/dart-sdk/lib/dev_compiler/amd/{{source_file_part}}",
   ]
 }
 
+# This rule copies tools to go along with ddc.
+copy("copy_dev_compiler_tools") {
+  visibility = [ ":copy_dev_compiler_sdk" ]
+  deps = [

[zra, 2017/05/22 20:56:09]

This also needs to depend on ":copy_dev_compiler_js" otherwise the copy_tree()
rule can clobber the copies done by this rule.

[Jacob, 2017/05/22 21:29:02]

Done.

+    "../utils/dartdevc:dartdevc_web",
+    "../utils/dartdevc:stack_trace_mapper",
+  ]
+  dart_out = get_label_info("../utils/dartdevc:dartdevc_web", "root_out_dir")
+  sources = [
+    "$dart_out/dev_compiler/build/web/dart_stack_trace_mapper.js",
+    "$dart_out/dev_compiler/build/web/ddc_web_compiler.js",
+  ]
+  outputs = [
+    "$root_out_dir/dart-sdk/lib/dev_compiler/web/{{source_file_part}}",
+  ]
+}
+
 # This is the main rule for copying ddc's dependencies to lib/
 group("copy_dev_compiler_sdk") {
   visibility = [ ":create_full_sdk" ]
   deps = [
     ":copy_dev_compiler_js",
     ":copy_dev_compiler_require_js",
     ":copy_dev_compiler_summary",
+    ":copy_dev_compiler_tools",
   ]
 }
 
 # This rule copies header files to include/
 copy("copy_headers") {
   visibility = [ ":create_common_sdk" ]
   sources = [
     "../runtime/include/dart_api.h",
     "../runtime/include/dart_mirrors_api.h",
     "../runtime/include/dart_native_api.h",
@@ skipping 205 matching lines @@
   deps = [
     ":copy_platform_sdk_libraries",
     ":copy_platform_sdk_scripts",
     ":copy_platform_sdk_snapshots",
   ]
 }
 
 # Parts specific to the full SDK.
 group("create_full_sdk") {
   visibility = [
-    ":copy_dev_compiler_tools",
     ":create_sdk",
   ]
+
   deps = [
     ":copy_dev_compiler_sdk",
     ":copy_full_sdk_libraries",
     ":copy_full_sdk_scripts",
     ":copy_full_sdk_snapshots",
   ]
 }
 
 # The main target to depend on from ../BUILD.gn
 group("create_sdk") {
   deps = [
     ":create_common_sdk",
   ]
   if (dart_platform_sdk) {
     deps += [ ":create_platform_sdk" ]
   } else {
     deps += [ ":create_full_sdk" ]
   }
 }
-
-# This rule copies tools to go along with ddc.
-copy("copy_dev_compiler_tools") {
-  deps = [
-    ":create_full_sdk",
-    "../utils/dartdevc:dartdevc_web",
-    "../utils/dartdevc:stack_trace_mapper",
-  ]
-  dart_out = get_label_info("../utils/dartdevc:dartdevc_web", "root_out_dir")
-  sources = [
-    "$dart_out/dev_compiler/build/web/dart_stack_trace_mapper.js",
-    "$dart_out/dev_compiler/build/web/ddc_web_compiler.js",
-  ]
-  outputs = [
-    "$root_out_dir/dart-sdk/lib/dev_compiler/web/{{source_file_part}}",
-  ]
-}