Lazily load the entrypoint.

Lazily load the entrypoint.

This means commands don't have to declare that they need.
It also ensures the pubspec isn't parsed until JSON output has been selected.

BUG=http://dartbug.com/20066, http://dartbug.com/20065
R=nweiz@google.com

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

[nweiz, 2014-07-16 00:33:30]

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

https://codereview.chromium.org/397843005/diff/1/sdk/lib/_internal/pub/lib/sr...
sdk/lib/_internal/pub/lib/src/command/list_package_dirs.dart:55:
entrypoint.pubspecPath
This doesn't actually read the pubspec anymore, does it? Seems weird to have the
editor re-run it when the pubspec changes if that won't do anything.

[nweiz, 2014-07-16 00:33:36]

lgtm

[Bob Nystrom, 2014-07-16 17:01:32]

Committed patchset #1 manually as r38291 (presubmit successful).

[Bob Nystrom, 2014-07-16 17:33:24]

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

https://codereview.chromium.org/397843005/diff/1/sdk/lib/_internal/pub/lib/sr...
sdk/lib/_internal/pub/lib/src/command/list_package_dirs.dart:55:
entrypoint.pubspecPath
On 2014/07/16 00:33:29, nweiz wrote:
> This doesn't actually read the pubspec anymore, does it?

It does, since we are touching "entrypoint" the root package's pubspec gets
read. I could make that lazy within Entrypoint, but I didn't think it was worth
doing.

> Seems weird to have the editor re-run it when the pubspec changes if that
won't do anything.

Yeah, I guess so. I think in practice it doesn't get touched that often.