Upload/Download dart_snapshotter

mojo/public/tools/BUILD.gn

Index: mojo/public/tools/BUILD.gn
diff --git a/mojo/public/tools/BUILD.gn b/mojo/public/tools/BUILD.gn
index 2368ae99aaa477163fb7b918030b377213830137..6959c8a006f4ee0a18ac5d699dc52a1f80cfd43b 100644
--- a/mojo/public/tools/BUILD.gn
+++ b/mojo/public/tools/BUILD.gn
@@ -28,6 +28,25 @@ if (mojo_use_prebuilt_mojo_shell) {
   }
 }
 
+if (mojo_use_prebuilt_dart_snapshotter) {
+  copy("copy_dart_snapshotter") {
+    sources = [
+      "prebuilt/dart_snapshotter/linux-x64/dart_snapshotter",

[jamesr, 2015/08/14 22:55:02]

this doesn't look like the right binary to use when developing on a mac host

[zra, 2015/08/17 15:06:16]

I didn't see a Mac builder for Mojo. Is there one?

[jamesr, 2015/08/17 17:29:39]

There is not, but developing from a Mac host is supported (on a best-effort
basis).  We provide linux and mac host binaries for other blobs.

[zra, 2015/08/17 21:59:18]

I've modified here and the upload_binaries.py script to do what I think might be
the right thing, but I think I don't understand how the mac host binaries are
provided if they aren't uploaded by a mac builder somewhere.

+    ]
+    if (is_android) {

[jamesr, 2015/08/14 22:55:01]

do you want to test for "is_android" here or "am i cross compiling"? do you need
the snapshotter for the host or target build?

[zra, 2015/08/17 15:06:16]

Right now in this CL, the snapshotter is invoked during the target build. Since
the Dart .mojo apps are platform agnostic, it could just as well be invoked for
the host build. I'm going to give that a try and see if it will help clean this
up.

[jamesr, 2015/08/17 17:29:39]

I think at a minimum you shouldn't need to invoke the copy step for both the
host and target toolchain.

You could have the copy step declare a data dependency on a copy that runs in
the host toolchain, depending on if that makes more sense (imo it would since in
the non-prebuilt state the snapshotter has to be built in the host toolchain)

[zra, 2015/08/17 21:59:18]

Done.

+      toolchain_name = get_label_info(host_toolchain, "name")
+      outputs = [
+        "$root_out_dir/$toolchain_name/dart_snapshotter",

[jamesr, 2015/08/14 22:55:01]

hm, doesn't root_out_dir already contain the current toolchain?

[zra, 2015/08/17 15:06:16]

If the current toolchain is the target toolchain, root_out_dir is e.g.
//out/android_Debug/. If the current toolchain is something else then
root_out_dir is //out/android_debug/$toolchain_name/

+      ]
+    } else {
+      assert(is_linux)
+      outputs = [
+        "$root_out_dir/dart_snapshotter",
+      ]
+    }
+  }
+}
+
 if (mojo_use_prebuilt_network_service) {
   copy("copy_network_service") {
     filename = "network_service.mojo"