Make the override warnings prettier.

sdk/lib/_internal/pub/lib/src/entrypoint.dart

Index: sdk/lib/_internal/pub/lib/src/entrypoint.dart
diff --git a/sdk/lib/_internal/pub/lib/src/entrypoint.dart b/sdk/lib/_internal/pub/lib/src/entrypoint.dart
index b4ef32b8fe15cec78ce0a290410b824c83bcf1f9..fcf4537a1ef4d6fd729eab3c2c2d85bc82746b1e 100644
--- a/sdk/lib/_internal/pub/lib/src/entrypoint.dart
+++ b/sdk/lib/_internal/pub/lib/src/entrypoint.dart
@@ -14,9 +14,10 @@ import 'lock_file.dart';
 import 'log.dart' as log;
 import 'package.dart';
 import 'package_graph.dart';
+import 'solver/version_solver.dart';
 import 'system_cache.dart';
 import 'utils.dart';
-import 'solver/version_solver.dart';
+import 'version.dart';
 
 /// Pub operates over a directed graph of dependencies that starts at a root
 /// "entrypoint" package. This is typically the package where the current
@@ -146,9 +147,18 @@ class Entrypoint {
       // Warn the user if any overrides were in effect.
       if (result.overrides.isNotEmpty) {
         var buffer = new StringBuffer();
-        buffer.write("Warning: You are overriding these dependencies:");
+        buffer.write("Warning: You are using these overridden dependencies:");
         for (var override in result.overrides) {
-          buffer.write("\n- $override");
+          var source = cache.sources[override.source];
+          buffer.write("\n- ${override.name}");
+          if (override.constraint != VersionConstraint.any) {
+            buffer.write(" version ${override.constraint}");
+          }
+          if (source != cache.sources.defaultSource) {
+            var description = source.formatDescription(root.dir,
+                override.description);
+            buffer.write(" (from ${override.source} $description)");
+          }

[nweiz, 2013/11/26 21:14:46]

Perhaps we should encapsulate this stringifying logic in [PackageDep.toString].

[Bob Nystrom, 2013/11/26 22:29:38]

The problem is it needs to know if a source is the default source, which
requires access to the SourceRegistry. We could hardcode an assumption about
"hosted", but that feels gross.

I thought about hoisting it out to a separate function, but it seemed premature
since nothing else is using it yet. I am interested in doing a pass over the
code where we output things and making it more consistent, but I didn't want to
make this patch too sweeping.

         }
         log.warning(buffer);
       }