Don't extract the name of a package from its description.

utils/pub/pub.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.
 
 /**
  * The main entrypoint for the pub command line application.
  */
 #library('pub');
 
 #import('../../pkg/args/args.dart');
@@ skipping 138 matching lines @@
   /**
    * A one-line description of this command.
    */
   abstract String get description;
 
   /**
    * How to invoke this command (e.g. `"pub install [package]"`).
    */
   abstract String get usage;
 
+  /// Whether or not this command uses [entrypoint]. If false, Pub won't look
+  /// for a pubspec and [entrypoint] will be null when the command runs.
+  bool get usesEntrypoint => true;

[Bob Nystrom, 2012/09/18 01:34:47]

How about "requiresEntrypoint"?

[nweiz, 2012/09/18 19:55:30]

Done.

+
   /**
    * Override this to define command-specific options. The results will be made
    * available in [commandOptions].
    */
   ArgParser get commandParser => new ArgParser();
 
   void run(SystemCache cache_, ArgResults globalOptions_,
       List<String> commandArgs) {
     cache = cache_;
     globalOptions = globalOptions_;
@@ skipping 19 matching lines @@
       printError(message);
       if (globalOptions['trace'] && trace != null) {
         printError(trace);
       }
 
       // TODO(nweiz): Use the more semantic error codes in
       // http://www.freebsd.org/cgi/man.cgi?query=sysexits
       exit(1);
     }
 
-    // TODO(rnystrom): Will eventually need better logic to walk up
-    // subdirectories until we hit one that looks package-like. For now, just
-    // assume the cwd is it.
-    var future = Package.load(workingDir, cache.sources).chain((package) {
-      entrypoint = new Entrypoint(package, cache);
+    var future = new Future.immediate(null);
+    if (usesEntrypoint) {
+      // TODO(rnystrom): Will eventually need better logic to walk up
+      // subdirectories until we hit one that looks package-like. For now, just
+      // assume the cwd is it.
+      future = Package.load(null, workingDir, cache.sources)
+        .transform((package) => new Entrypoint(package, cache));

[Bob Nystrom, 2012/09/18 01:34:47]

Indent +2.

[nweiz, 2012/09/18 19:55:30]

Done.

+    }
 
+    future = future.chain((entrypoint) {
+      this.entrypoint = entrypoint;
       try {
         var commandFuture = onRun();
         if (commandFuture == null) return new Future.immediate(true);
 
         return commandFuture;
       } catch (error, trace) {
         handleError(error, trace);
         return new Future.immediate(null);
       }
     });
-    future.handleException((e) => handleError(e, future.stackTrace));
+
+    future.handleException((e) {
+      if (e is PubspecNotFoundException && e.name == null) {
+        e = 'Could not find a file named "pubspec.yaml" in the directory '
+          '$workingDir.';
+      } else if (e is PubspecHasNoNameException && e.name == null) {
+        e = 'pubspec.yaml is missing the required "name" field (e.g. "name: '
+          '${basename(workingDir)}").';
+      }
+
+      handleError(e, future.stackTrace);
+    });
     // Explicitly exit on success to ensure that any dangling dart:io handles
     // don't cause the process to never terminate.
     future.then((_) => exit(0));
   }
 
   /**
    * Override this to perform the specific command. Return a future that
    * completes when the command is done or fails if the command fails. If the
    * command is synchronous, it may return `null`.
    */
   abstract Future onRun();
 
   /** Displays usage information for this command. */
   void printUsage([String description]) {
     if (description == null) description = this.description;
     print(description);
     print('');
     print('Usage: $usage');
 
     var commandUsage = commandParser.getUsage();
     if (!commandUsage.isEmpty()) {
       print('');
       print(commandUsage);
     }
   }
 }