Reverting 7623

utils/pub/package.dart

Index: utils/pub/package.dart
diff --git a/utils/pub/package.dart b/utils/pub/package.dart
index af585ab058af4e73138c987a203d4e6023b1e88a..6b88de5a29c13bf52374d3f1304e66190e56c664 100644
--- a/utils/pub/package.dart
+++ b/utils/pub/package.dart
@@ -9,10 +9,10 @@ class Package implements Hashable {
   /**
    * Loads the package whose root directory is [packageDir].
    */
-  static Future<Package> load(String packageDir, SourceRegistry sources) {
+  static Future<Package> load(String packageDir) {
     final pubspecPath = join(packageDir, 'pubspec');
 
-    return _parsePubspec(pubspecPath, sources).transform((dependencies) {
+    return _parsePubspec(pubspecPath).transform((dependencies) {
       return new Package._(packageDir, dependencies);
     });
   }
@@ -57,8 +57,7 @@ class Package implements Hashable {
    * Parses the pubspec at the given path and returns the list of package
    * dependencies it exposes.
    */
-  static Future<List<PackageId>> _parsePubspec(String path,
-      SourceRegistry sources) {
+  static Future<List<PackageId>> _parsePubspec(String path) {
     final completer = new Completer<List<PackageId>>();
 
     // TODO(rnystrom): Handle the directory not existing.
@@ -89,63 +88,13 @@ class Package implements Hashable {
       }
 
       var dependencies = parsedPubspec['dependencies'];
-      if (dependencies is! Map ||
-          dependencies.getKeys().some((e) => e is! String)) {
+      if (dependencies.some((e) => e is! String)) {
         completer.completeException(
-            'The pubspec dependencies must be a map of package names.');
+            'The pubspec dependencies must be a list of package names.');
       }
 
-      var dependencyIds = <PackageId>[];
-      dependencies.forEach((name, spec) {
-        var fullName, source;
-        // TODO(nweiz): parse the version once we have version handling
-        if (spec == null || spec is String) {
-          fullName = name;
-          source = sources.defaultSource;
-        } else if (spec is Map) {
-          spec.remove('version');
-
-          var sourceNames = spec.getKeys();
-          if (sourceNames.length > 1) {
-            completer.completeException(
-                'Dependency $name may not have multiple sources: '
-                '$sourceNames.');
-            return;
-          }
-
-          var sourceName = only(sourceNames);
-          if (sourceName is! String) {
-            completer.completeException(
-                'Source name $sourceName must be a string.');
-            return;
-          }
-          source = sources[sourceName];
-
-          // TODO(nweiz): At some point we want fullName to be able to be an
-          // arbitrary object that's parsed by the source.
-          fullName = spec[sourceName];
-          if (fullName is! String) {
-            completer.completeException(
-                'Source identifier $fullName must be a string.');
-            return;
-          }
-        } else {
-          completer.completeException(
-              'Dependency specification $spec must be a string or a mapping.');
-          return;
-        }
-
-        var id = new PackageId(fullName, source);
-        var nameFromSource = source.packageName(id);
-        if (nameFromSource != name) {
-          completer.completeException(
-              'Dependency name "$name" doesn\'t match name "$nameFromSource" '
-              'from source "${source.name}".');
-          return;
-        }
-
-        dependencyIds.add(id);
-      });
+      var dependencyIds =
+        dependencies.map((name) => new PackageId(name, Source.defaultSource));
       completer.complete(dependencyIds);
     });