This isn't going to make it into -dev.4.0.

This will make it easier to add features to the pubspec and verify that
they're only published with appropriate SDK constraints.

It also simplifies the dependency validator logic around caret
constraints, since it's a lot less likely that anyone will want to
support an SDK older than 1.8.0 at this point.

This more clearly indicates the type of data the class contains. We
were already using in more places than just dependencies, so it makes
sense to have a name that reflects that.

These classes are largely independent of the Package class.

This switches pub's tests over to using the stream matchers built into
test, as well as the test_descriptor, test_process, and
shelf_test_handler packages. It uses async/await rather than
scheduled_test's task queue.

This removes a layer of indirection which should make it much easier to
work on pub's test infrastructure, especially for people unfamiliar with
scheduled_test.

* remove unused imports

* use SourceFile.fromString, the default constructor is deprecated

Fix #1560.

Just like test and tool
Also expanded and generalized these tests

Closes https://github.com/dart-lang/pub/issues/1537

* Add metadata to requests to pub.dartlang.org.

This will help pub.dartlang.org better track how packages relate to one
another.

* Fix broken depenency tests.

Those unit tests import pub libraries directly without invoking a
command, so command was throwing an exception. Worse, the Pokemon
exception handler in DependencyValidator was then catching the
StateError, hiding the bug.

Also applied some other changes I suggested.

This adds a validator that warns when a package incorrectly imports a non-dependency.

This allows packages that override only dev dependencies to be
published.

Closes #1444

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//2276143002 .

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//2184303002 .

Packages with Flutter constraints should only be published with SDK
constraints that restrict the SDK to versions that support Flutter
constraints.

Closes #1432

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//2172883002 .

We still need to verify that a Flutter SDK constraint is only
used with an appropriate Dart SDK constraint.

See #1432

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//2165423002 .

This allows PackageName equality to include description equality, and it
substantially cleans up other logic dealing with sources.

See #912

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//2079303003 .

This splits Source into two classes, Source and LiveSource. The former
is completely independent of the system cache directory, and mostly
contains source metadata and logic about manipulating descriptions.

SourceRegistry now deals strictly in Sources, with LiveSources being
handled by SystemCache. This means that classes like Pubspec and
LockFile no longer have transitive references to the system cache
directory.

This will also make it possible to clean up PackageName equality logic
by making PackageNames track their concrete Sources rather than just
their source names.

See #912

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//2044253003 .

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1585513002 .

This folds the distinction between "resolved" and "unresolved"
descriptions into the distinction between PackageIds and PackageRefs. In
other words, all PackageIds now have resolved descriptions and all
PackageRefs now have unresolved descriptions. This makes it easier to
track which descriptions are resolved and express resolution constraints
via the type system.

This also restricts the creation of PackageRefs and PackageIds to source
classes, which helps ensure that the description format—which is an
implementation detail—doesn't leak into surrounding code.

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1528523003 .

Previously these names were only disallowed when publishing, but now the
package config format requires that we be more stringent about them.

Closes #1361

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1493463002 .

This allows getVersion() to return a list of package IDs rather than a
list of pubspecs, which in turn will allow the Git source to return
resolved IDs in #1335.

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1459733002 .

Pub won't upload gitignored files, so its validation shouldn't consider
them to exist either.

See #1321

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1310253007 .

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1123323003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@45572 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1113363004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@45571 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1101343004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@45508 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//1060713002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@44885 260f80e4-7a28-3924-810f-c04153c831b5

This gets rid of the suggested prerelease suffix for SDK constraints
for pubspecs using the ^ operator, since 1.8.0 itself is out now and
probably no one is using a prerelease version of it.

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//937503002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@43823 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//868463003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@43057 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
TBR

Review URL: https://codereview.chromium.org//702343002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@41542 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
BUG=17172

Review URL: https://codereview.chromium.org//686323002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@41538 260f80e4-7a28-3924-810f-c04153c831b5

R=nweiz@google.com

Review URL: https://codereview.chromium.org//602253002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@40849 260f80e4-7a28-3924-810f-c04153c831b5

(Still use it for publishing.)

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

Review URL: https://codereview.chromium.org//599303002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@40661 260f80e4-7a28-3924-810f-c04153c831b5

BUG=20859
R=rnystrom@google.com

Review URL: https://codereview.chromium.org//583853002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@40612 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//581283003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@40559 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//566093003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@40471 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//486213004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@39529 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//354763006

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37963 260f80e4-7a28-3924-810f-c04153c831b5

In particular, this makes barback and the "pub lish" validators use gitignore
rules.

R=rnystrom@google.com
BUG=12102, 11198

Review URL: https://codereview.chromium.org//342553002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37469 260f80e4-7a28-3924-810f-c04153c831b5