Validate packages against their SDK constraints.

utils/pub/git_source.dart

Index: utils/pub/git_source.dart
diff --git a/utils/pub/git_source.dart b/utils/pub/git_source.dart
index 477cc17a7e9bd17ed250476516aed5e259e423f4..2b46706c8a8b168187fa463610ade1b2dddad1dd 100644
--- a/utils/pub/git_source.dart
+++ b/utils/pub/git_source.dart
@@ -60,6 +60,8 @@ class GitSource extends Source {
     });
   }
 
+  Future<String> systemCacheDirectory(PackageId id) => _revisionCachePath(id);

[nweiz, 2013/01/31 22:38:53]

Why not just rename "_revisionCachePath" to "systemCacheDirectory"?

[Bob Nystrom, 2013/02/01 16:53:29]

I kept it because I like that it helps clarify the git-specific semantics of
that path.

[nweiz, 2013/02/01 22:24:54]

I don't like having two differently-named functions that do identical things.
I'd rather just document on this method that it's the part for the specific
revision.

[Bob Nystrom, 2013/02/02 00:02:31]

Done.

+
   /// Ensures [description] is a Git URL.
   void validateDescription(description, {bool fromLockFile: false}) {
     // A single string is assumed to be a Git URL.