Don't choke on SSH Git URLs.

lib/src/source/git.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 pub.source.git;
 
 import 'dart:async';
 
 import 'package:path/path.dart' as path;
 import 'package:pub_semver/pub_semver.dart';
@@ skipping 117 matching lines @@
     if (description is! Map) {
       throw new FormatException("The description must be a Git URL or a map "
           "with a 'url' key.");
     }
 
     if (description["url"] is! String) {
       throw new FormatException("The 'url' field of the description must be a "
           "string.");
     }
 
-    // Ensure that it's a valid URL.
-    Uri.parse(description["url"]);
+    _validateUrl(description["url"]);
 
     var ref = description["ref"];
     if (ref != null && ref is! String) {
       throw new FormatException("The 'ref' field of the description must be a "
           "string.");
     }
 
     return new PackageRef(name, this.name, {
       "url": description["url"],
       "ref": description["ref"] ?? "HEAD"
     });
   }
 
   PackageId parseId(String name, Version version, description) {
     if (description is! Map) {
       throw new FormatException("The description must be a map with a 'url' "
           "key.");
     }
 
     if (description["url"] is! String) {
       throw new FormatException("The 'url' field of the description must be a "
           "string.");
     }
 
-    // Ensure that it's a valid URL.
-    Uri.parse(description["url"]);
+    _validateUrl(description["url"]);
 
     var ref = description["ref"];
     if (ref != null && ref is! String) {
       throw new FormatException("The 'ref' field of the description must be a "
           "string.");
     }
 
     if (description["resolved-ref"] is! String) {
       throw new FormatException("The 'resolved-ref' field of the description "
           "must be a string.");
     }
 
     return new PackageId(name, this.name, version, {
       "url": description["url"],
       "ref": description["ref"] ?? "HEAD",
       "resolved-ref": description["resolved-ref"]
     });
   }
 
+  /// Throws a [FormatException] if [url] isn't a valid Git URL.
+  void _validateUrl(String url) {
+    // If the URL contains an @, it's probably an SSH hostname, which we don't
+    // know how to validate.
+    if (url.contains("@")) return;
+
+    // Otherwise, we use Dart's URL parser to validate the URL.
+    Uri.parse(url);
+  }
+
   /// If [description] has a resolved ref, print it out in short-form.
   ///
   /// This helps distinguish different git commits with the same pubspec
   /// version.
   String formatDescription(String containingPath, description) {
     if (description is Map && description.containsKey('resolved-ref')) {
       return "${description['url']} at "
           "${description['resolved-ref'].substring(0, 6)}";
     } else {
       return super.formatDescription(containingPath, description);
@@ skipping 161 matching lines @@
         (result) => null);
   }
 
   /// Returns the path to the canonical clone of the repository referred to by
   /// [id] (the one in `<system cache>/git/cache`).
   String _repoCachePath(PackageRef ref) {
     var repoCacheName = '${ref.name}-${sha1(ref.description['url'])}';
     return path.join(systemCacheRoot, 'cache', repoCacheName);
   }
 }