added package_root flag to standalone dart vm

runtime/bin/builtin.dart

Index: runtime/bin/builtin.dart
diff --git a/runtime/bin/builtin.dart b/runtime/bin/builtin.dart
index e0108096f6c8934346edbf6b103b0ea3dfe26d75..0b58d4300c0d28241e1b6e00ce63dc4a4ea4d79e 100644
--- a/runtime/bin/builtin.dart
+++ b/runtime/bin/builtin.dart
@@ -32,6 +32,9 @@ var _is_windows;
 // package imports can be resolved relative to it.
 Uri _entrypoint;
 
+// The directory to look in to resolve "package:" scheme URIs.
+String _packageRoot;
+
 void _logResolution(String msg) {
   final enabled = false;
   if (enabled) {
@@ -39,6 +42,15 @@ void _logResolution(String msg) {
   }
 }
 
+String _setPackageRoot(String packageRoot) {
+  // Note, on Windows we require use of forward slash, and do not
+  // support backslash.

[Ivan Posva, 2012/04/26 05:32:09]

If you check a couple of lines below, there is example code how to deal with
back slashes on Windows. Only supporting forward slashes is a restriction that
we do not have in other areas of the system and it is inconsistent and
surprising to have this restriction here. Please fix.

[siva, 2012/04/26 13:57:54]

It is not just the backslash that is a problem. Consider somebody specifying
--package_root="d:" meaning they want to use the current directory on drive d:,
a perfectly normal operation on windows. In that case we are not supposed to
stick a backslash in.

On 2012/04/26 05:32:09, Ivan Posva wrote:

[mattsh, 2012/04/26 17:52:02]

If it's OK, I'll put in a TODO now, because I think we want to rework the other
code slightly that handles windows so it can be reused here, but that should be
a separate CL. (I.e., we don't want to have two places of code that do the
backslash replacement.)

+  if (!packageRoot.endsWith("/")) {
+    packageRoot += "/";

[siva, 2012/04/25 23:26:12]

The '+' operator has been deprecated and is being removed from the VM code base.
Would be preferable if we did not use it in any new dart code additions.

[mattsh, 2012/04/26 17:52:02]

Done.

+  }
+  _packageRoot = packageRoot;
+}
+
 String _resolveScriptUri(String cwd, String scriptName, bool windows) {
   _is_windows = windows;
   _logResolution("# Current working directory: $cwd");
@@ -141,7 +153,13 @@ String _filePathFromPackageUri(Uri uri) {
           "'$right', not '$wrong'.";
   }
 
-  var path = _entrypoint.resolve('packages/${uri.path}').path;
+  String path;

[Ivan Posva, 2012/04/26 05:32:09]

No types for locals. Please preserve the style of this file. Thank you.

[mattsh, 2012/04/26 17:52:02]

Done.

+  if (_packageRoot !== null) {
+    path = _packageRoot + uri.path;

[siva, 2012/04/25 23:26:12]

Ditto comment regarding '+' operator.

[mattsh, 2012/04/26 17:52:02]

Done.

+  } else {
+    path = _entrypoint.resolve('packages/${uri.path}').path;
+  }
+
   _logResolution("# Package: $path");
   return path;
 }