Remove support for SDK dependencies.

utils/pub/validator/dependency.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 dependency_validator;
 
 import 'dart:async';
 
 import '../entrypoint.dart';
 import '../hosted_source.dart';
@@ skipping 16 matching lines @@
       }
 
       if (dependency.name == entrypoint.root.name) {
         warnings.add('You don\'t need to explicitly depend on your own '
                 'package.\n'
             'Pub enables "package:${entrypoint.root.name}" imports '
                 'implicitly.');
         return new Future.immediate(null);
       }
 
-      if (dependency.constraint.isAny &&
-          // TODO(nweiz): once we have development dependencies (issue 5358), we
-          // should warn about unittest. Until then, it's reasonable not to put
-          // a constraint on it.
-          dependency.name != 'unittest') {
-        _warnAboutConstraint(dependency);
-      }

[nweiz, 2013/03/15 18:21:54]

Should we maybe suggest that people use a dev dependency for "unittest: any"
dependencies for a while, as a migration path?

[Bob Nystrom, 2013/03/15 18:31:56]

I think the fix for users here is pretty simple, so I was just going to mention
"move your unittest deps to dev_deps" in the email when I tell people about
this.

I don't mind having a migration window, but it seemed like just one more TODO to
add to our pile. It probably would be a little more graceful to do this, but I
kind of want to just get this done completely. :)

+      if (dependency.constraint.isAny) _warnAboutConstraint(dependency);
 
       return new Future.immediate(null);
     });
   }
 
   /// Warn that dependencies should use the hosted source.
   Future _warnAboutSource(PackageRef ref) {
     return entrypoint.cache.sources['hosted']
         .getVersions(ref.name, ref.name)
         .catchError((e) => <Version>[])
@@ skipping 44 matching lines @@
   }
 
   /// Returns the suggested version constraint for a dependency that was tested
   /// against [version].
   String _constraintForVersion(Version version) {
     if (version.major != 0) return '">=$version <${version.major + 1}.0.0"';
     return '">=$version <${version.major}.${version.minor}.'
         '${version.patch + 1}"';
   }
 }