Generate the package spec from the lockfile.

Generate the package spec from the lockfile.

The spec doesn't actually need all the data loaded from the package
graph; all it needs are the locations of packages which can be looked up
using their IDs and a SourceRegistry.

R=rnystrom@google.com

Committed: https://github.com/dart-lang/pub/commit/63ad514857c82f58fbc34589b0bca808958e17d8

[Bob Nystrom, 2015-08-07 23:51:26]

https://codereview.chromium.org/1282083002/diff/1/lib/src/lock_file.dart
File lib/src/lock_file.dart (right):

https://codereview.chromium.org/1282083002/diff/1/lib/src/lock_file.dart#newc...
lib/src/lock_file.dart:164: Dart tools, including the Dart VM and Dart analyzer,
rely on the content.
I don't think it's worth putting this in every file. How about just the first
line and drop the rest?

https://codereview.chromium.org/1282083002/diff/1/lib/src/lock_file.dart#newc...
lib/src/lock_file.dart:171: return p.toUri(p.join(source.getDirectory(id),
"lib"));
What do you think about making this path relative to the .packages file?

[nweiz, 2015-08-08 00:52:57]

https://codereview.chromium.org/1282083002/diff/1/lib/src/lock_file.dart
File lib/src/lock_file.dart (right):

https://codereview.chromium.org/1282083002/diff/1/lib/src/lock_file.dart#newc...
lib/src/lock_file.dart:164: Dart tools, including the Dart VM and Dart analyzer,
rely on the content.
On 2015/08/07 23:51:25, Bob Nystrom wrote:
> I don't think it's worth putting this in every file. How about just the first
> line and drop the rest?

Done.

https://codereview.chromium.org/1282083002/diff/1/lib/src/lock_file.dart#newc...
lib/src/lock_file.dart:171: return p.toUri(p.join(source.getDirectory(id),
"lib"));
On 2015/08/07 23:51:25, Bob Nystrom wrote:
> What do you think about making this path relative to the .packages file?

It does that for path dependencies. For cached dependencies, though, we want it
to be absolute so the package can be moved around and still have a valid
.packages file.

[Bob Nystrom, 2015-08-10 16:03:24]

lgtm

[nweiz, 2015-08-10 19:46:08]

Committed patchset #1 (id:1) manually as
63ad514857c82f58fbc34589b0bca808958e17d8 (presubmit successful).