Command to print paths to dependencies.

Command to print paths to dependencies.

BUG=https://code.google.com/p/dart/issues/detail?id=12354
R=nweiz@google.com

Committed: https://code.google.com/p/dart/source/detail?r=26287

[nweiz, 2013-08-16 00:41:33]

A couple comments, otherwise lgtm.

https://codereview.chromium.org/23272002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/command/list_package_dirs.dart (right):

https://codereview.chromium.org/23272002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/command/list_package_dirs.dart:24: allowed:
["json"]);
Let's add this option later if we ever have other formats we want to emit.

https://codereview.chromium.org/23272002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/command/list_package_dirs.dart:34: var lockFile =
entrypoint.loadLockFile();
Unused variable.

[Bob Nystrom, 2013-08-16 18:17:39]

Thanks!

https://codereview.chromium.org/23272002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/command/list_package_dirs.dart (right):

https://codereview.chromium.org/23272002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/command/list_package_dirs.dart:24: allowed:
["json"]);
On 2013/08/16 00:41:33, nweiz wrote:
> Let's add this option later if we ever have other formats we want to emit.

I wanted to add it now to ensure that the Editor can pass it in now without
causing an error.

https://codereview.chromium.org/23272002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/command/list_package_dirs.dart:34: var lockFile =
entrypoint.loadLockFile();
On 2013/08/16 00:41:33, nweiz wrote:
> Unused variable.

Done.

[Bob Nystrom, 2013-08-16 18:17:52]

Committed patchset #2 manually as r26287 (presubmit successful).