Make binstubs run snapshots directly when possible.

sdk/lib/_internal/pub/lib/src/barback/asset_environment.dart

 // Copyright (c) 2014, 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.
 
 library pub.barback.asset_environment;
 
 import 'dart:async';
 import 'dart:io';
 
 import 'package:barback/barback.dart';
@@ skipping 219 matching lines @@
   Future<BarbackServer> servePackageBinDirectory(String package) {
     return _provideDirectorySources(graph.packages[package], "bin").then(
         (_) => BarbackServer.bind(this, _hostname, 0, package: package,
             rootDirectory: "bin"));
   }
 
   /// Precompiles all of [packageName]'s executables to snapshots in
   /// [directory].
   ///
   /// If [executableIds] is passed, only those executables are precompiled.
-  Future precompileExecutables(String packageName, String directory,
-      {Iterable<AssetId> executableIds}) {
+  ///
+  /// Returns a map from executable ID to path for the snapshots that were

[nweiz, 2014/09/23 00:15:44]

This says "executable ID" but actually returns the executable name.

[Bob Nystrom, 2014/09/23 18:23:27]

Done.

+  /// successfully precompiled.
+  Future<Map<String, String>> precompileExecutables(String packageName,
+      String directory, {Iterable<AssetId> executableIds}) async {
     if (executableIds == null) {
       executableIds = graph.packages[packageName].executableIds;
     }
-    log.fine("executables for $packageName: $executableIds");
-    if (executableIds.isEmpty) return null;
+
+    log.fine("Executables for $packageName: $executableIds");
+    if (executableIds.isEmpty) return [];
 
     var package = graph.packages[packageName];
-    return servePackageBinDirectory(packageName).then((server) {
-      return waitAndPrintErrors(executableIds.map((id) {
+    var server = await servePackageBinDirectory(packageName);
+    try {
+      var precompiled = {};
+      await waitAndPrintErrors(executableIds.map((id) async {
         var basename = path.url.basename(id.path);
         var snapshotPath = path.join(directory, "$basename.snapshot");
-        return runProcess(Platform.executable, [
+        var result = await runProcess(Platform.executable, [
           '--snapshot=$snapshotPath',
           server.url.resolve(basename).toString()
-        ]).then((result) {
-          if (result.success) {
-            log.message("Precompiled ${_formatExecutable(id)}.");
-          } else {
-            throw new ApplicationException(
-                log.yellow("Failed to precompile "
-                    "${_formatExecutable(id)}:\n") +
-                result.stderr.join('\n'));
-          }
-        });
-      })).whenComplete(() {
-        // Don't return this future, since we have no need to wait for the
-        // server to fully shut down.
-        server.close();
-      });
-    });
+        ]);
+        if (result.success) {
+          log.message("Precompiled ${_formatExecutable(id)}.");
+          precompiled[path.withoutExtension(basename)] = snapshotPath;
+        } else {
+          throw new ApplicationException(
+              log.yellow("Failed to precompile ${_formatExecutable(id)}:\n") +
+              result.stderr.join('\n'));
+        }
+      }));
+
+      return precompiled;
+    } finally {
+      // Don't await this future, since we have no need to wait for the server
+      // to fully shut down.
+      server.close();
+    }
   }
 
   /// Returns the executable name for [id].
   ///
   /// [id] is assumed to be an executable in a bin directory. The return value
   /// is intended for log output and may contain formatting.
   String _formatExecutable(AssetId id) =>
       log.bold("${id.package}:${path.basenameWithoutExtension(id.path)}");
 
   /// Stops the server bound to [rootDirectory].
@@ skipping 507 matching lines @@
   String toString() => "polling";
 }
 
 class _NoneWatcherType implements WatcherType {
   const _NoneWatcherType();
 
   DirectoryWatcher create(String directory) => null;
 
   String toString() => "none";
 }