Support resources in "pub run" and "pub global run".

lib/src/executable.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.executable;
 
 import 'dart:async';
 import 'dart:io';
 
 import 'package:barback/barback.dart';
 import 'package:path/path.dart' as p;
-import 'package:stack_trace/stack_trace.dart';
 
 import 'barback/asset_environment.dart';
 import 'entrypoint.dart';
 import 'exit_codes.dart' as exit_codes;
 import 'io.dart';
 import 'log.dart' as log;
 import 'utils.dart';
 
 /// All signals that can be caught by a Dart process.
 ///
@@ skipping 84 matching lines @@
       message += " in package ${log.bold(package)}";
     }
     log.error("$message.");
     return exit_codes.NO_INPUT;
   }
 
   // If we're running an executable directly from the filesystem, make sure that
   // it knows where to load the packages. If it's a dependency's executable, for
   // example, it may not have the right packages directory itself.
   if (executableUrl.scheme == 'file' || executableUrl.scheme == '') {
-    // TODO(nweiz): use a .packages file once sdk#23369 is fixed.
-
     // We use an absolute path here not because the VM insists but because it's
     // helpful for the subprocess to be able to spawn Dart with
     // Platform.executableArguments and have that work regardless of the working
     // directory.
     vmArgs.add('--packages=${p.toUri(p.absolute(entrypoint.packagesFile))}');
   }
 
   vmArgs.add(executableUrl.toString());
   vmArgs.addAll(args);
 
@@ skipping 74 matching lines @@
 /// If [recompile] is passed, it's called if the snapshot is out-of-date. It's
 /// expected to regenerate a snapshot at [path], after which the snapshot will
 /// be re-run. It may return a Future.
 ///
 /// If [checked] is set, runs the snapshot in checked mode.
 ///
 /// Returns the snapshot's exit code.
 ///
 /// This doesn't do any validation of the snapshot's SDK version.
 Future<int> runSnapshot(String path, Iterable<String> args, {recompile(),
-    bool checked: false}) async {
-  var vmArgs = [path]..addAll(args);
-
+    String packagesFile, bool checked: false}) async {
   // TODO(nweiz): pass a flag to silence the "Wrong full snapshot version"
   // message when issue 20784 is fixed.
-  if (checked) vmArgs.insert(0, "--checked");
+  var vmArgs = [];
+  if (checked) vmArgs.add("--checked");
+
+  if (packagesFile != null) {
+    // We use an absolute path here not because the VM insists but because it's
+    // helpful for the subprocess to be able to spawn Dart with
+    // Platform.executableArguments and have that work regardless of the working
+    // directory.
+    vmArgs.add("--packages=${p.toUri(p.absolute(packagesFile))}");
+  }
+
+  vmArgs.add(path);
+  vmArgs.addAll(args);
 
   // We need to split stdin so that we can send the same input both to the
   // first and second process, if we start more than one.
   var stdin1;
   var stdin2;
   if (recompile == null) {
     stdin1 = stdin;
   } else {
     var pair = tee(stdin);
     stdin1 = pair.first;
@@ skipping 33 matching lines @@
     signal.watch().listen((_) {
       log.fine("Forwarding $signal to running process.");
       process.kill(signal);
     });
   }
 }
 
 /// Runs the executable snapshot at [snapshotPath].
 Future<int> _runCachedExecutable(Entrypoint entrypoint, String snapshotPath,
     List<String> args, {bool checked: false}) {
-  return runSnapshot(snapshotPath, args, checked: checked, recompile: () {
+  return runSnapshot(snapshotPath, args,
+      packagesFile: entrypoint.packagesFile,
+      checked: checked,
+      recompile: () {
     log.fine("Precompiled executable is out of date.");
     return entrypoint.precompileExecutables();
   });
 }