Dev dependencies.

utils/pub/pubspec.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library pubspec;
 
 import '../../pkg/yaml/lib/yaml.dart';
 import '../../pkg/pathos/lib/path.dart' as path;
 
 import 'io.dart';
 import 'package.dart';
 import 'source.dart';
 import 'source_registry.dart';
 import 'utils.dart';
 import 'version.dart';
 
 /// The parsed and validated contents of a pubspec file.
 class Pubspec {
   /// This package's name.
   final String name;
 
   /// This package's version.
   final Version version;
 
   /// The packages this package depends on.
   final List<PackageRef> dependencies;
 
+  /// The packages this package depends on when it is the root package.
+  final List<PackageRef> devDependencies;
+
   /// The environment-related metadata.
   final PubspecEnvironment environment;
 
   /// All pubspec fields. This includes the fields from which other properties
   /// are derived.
   final Map<String, Object> fields;
 
   /// Loads the pubspec for a package [name] located in [packageDir].
   factory Pubspec.load(String name, String packageDir, SourceRegistry sources) {
     var pubspecPath = path.join(packageDir, 'pubspec.yaml');
@@ skipping 10 matching lines @@
       if (name != null && pubspec.name != name) {
         throw new PubspecNameMismatchException(name, pubspec.name);
       }
 
       return pubspec;
     } on FormatException catch (ex) {
       throw 'Could not parse $pubspecPath:\n${ex.message}';
     }
   }
 
-  Pubspec(this.name, this.version, this.dependencies, this.environment,
-      [Map<String, Object> fields])
+  Pubspec(this.name, this.version, this.dependencies, this.devDependencies,
+          this.environment, [Map<String, Object> fields])
     : this.fields = fields == null ? {} : fields;
 
   Pubspec.empty()
     : name = null,
       version = Version.none,
       dependencies = <PackageRef>[],
+      devDependencies = <PackageRef>[],
       environment = new PubspecEnvironment(),
       fields = {};
 
   /// Whether or not the pubspec has no contents.
   bool get isEmpty =>
     name == null && version == Version.none && dependencies.isEmpty;
 
   // TODO(rnystrom): Instead of allowing a null argument here, split this up
   // into load(), parse(), and _parse() like LockFile does.
   /// Parses the pubspec stored at [filePath] whose text is [contents]. If the
@@ skipping 22 matching lines @@
       }
     }
 
     if (parsedPubspec.containsKey('version')) {
       version = new Version.parse(parsedPubspec['version']);
     }
 
     var dependencies = _parseDependencies(filePath, sources,
         parsedPubspec['dependencies']);
 
+    var devDependencies = _parseDependencies(filePath, sources,
+        parsedPubspec['dev_dependencies']);
+
+    // Make sure the same package doesn't appear as both a regular and dev
+    // dependency.
+    var dependencyNames = dependencies.map((dep) => dep.name).toSet();
+    var collisions = dependencyNames.intersection(
+        devDependencies.map((dep) => dep.name).toSet());
+
+    if (!collisions.isEmpty) {
+      throw new FormatException(
+          'Package "${collisions.first}" cannot appear in both "dependencies" '
+          'and "dev_dependencies".');

[nweiz, 2013/03/13 18:46:29]

Let's list all the collisions so users don't have to run commands over and over
again.

[Bob Nystrom, 2013/03/13 19:42:44]

Done.

+    }
+
     var environmentYaml = parsedPubspec['environment'];
     var sdkConstraint = VersionConstraint.any;
     if (environmentYaml != null) {
       if (environmentYaml is! Map) {
         throw new FormatException(
             'The pubspec "environment" field should be a map, but was '
             '"$environmentYaml".');
       }
 
       var sdkYaml = environmentYaml['sdk'];
@@ skipping 44 matching lines @@
         throw new FormatException('The pubspec "authors" field should be a '
             'string or a list of strings, but was "$authors".');
       }
 
       if (parsedPubspec.containsKey('author')) {
         throw new FormatException('A pubspec should not have both an "author" '
             'and an "authors" field.');
       }
     }
 
-    return new Pubspec(name, version, dependencies, environment, parsedPubspec);
+    return new Pubspec(name, version, dependencies, devDependencies,
+        environment, parsedPubspec);
   }
 }
 
 /**
  * Evaluates whether the given [url] for [field] is valid.
  *
  * Throws [FormatException] on an invalid url.
  */
 void _validateFieldUrl(url, String field) {
   if (url is! String) {
@@ skipping 69 matching lines @@
 /// The environment-related metadata in the pubspec. Corresponds to the data
 /// under the "environment:" key in the pubspec.
 class PubspecEnvironment {
   /// The version constraint specifying which SDK versions this package works
   /// with.
   final VersionConstraint sdkVersion;
 
   PubspecEnvironment([VersionConstraint sdk])
       : sdkVersion = sdk != null ? sdk : VersionConstraint.any;
 }