Get codebase ready for actually supporting versioning.

utils/pub/source.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.
 
 /**
  * A source from which to install packages.
  *
  * Each source has many packages that it looks up using [PackageId]s. The source
  * is responsible for installing these packages to the package cache.
  */
@@ skipping 23 matching lines @@
    * If [shouldCache] is true, [path] will be a path to this source's
    * subdirectory of the [PackageCache]'s cache directory. If [shouldCache] is
    * false, [path] will be a path to the application's "packages" directory.
    *
    * [path] is guaranteed not to exist, and its parent directory is guaranteed
    * to exist.
    */
   abstract Future<bool> install(PackageId id, String path);
 
   /**
-   * Returns the name of the package identified by [id]. By default, this is
-   * just `id.fullName`, but some sources (e.g. Git) may have more complicated
-   * resolution logic.
-   *
-   * This method should be light-weight. It doesn't need to validate that the
-   * given package exists.
+   * Returns a human-friendly name for the package identified by [id]. This
+   * method should be light-weight. It doesn't need to validate that the given
+   * package exists.
    *
    * The package name should be lower-case and suitable for use in a filename.
    * It may contain forward slashes.
    */
-  String packageName(PackageId id) => id.fullName;
+  abstract String packageName(PackageId id);

[nweiz, 2012/05/18 00:08:24]

I feel like id.description is still a good default for this.

[Bob Nystrom, 2012/05/18 20:02:38]

Done.

 }