# HG changeset patch
# User Matt Harbison <matt_harbison@yahoo.com>
# Date 2012-09-09 16:09:53
# Node ID 3a1c6b64e052c66a589557aeb377eea5cd034a1f
# Parent  56136786000fca929b565680730228255b2590af

largefiles: don't convert dest=None to dest=hg.defaultdest() in clone command

A status message is output if hg.clone() determines the default destination
because None was provided.  The previous code never passed None to hg.clone().

diff --git a/hgext/largefiles/overrides.py b/hgext/largefiles/overrides.py
--- a/hgext/largefiles/overrides.py
+++ b/hgext/largefiles/overrides.py
@@ -720,12 +720,13 @@ def overridepull(orig, ui, repo, source=
     return result
 
 def overrideclone(orig, ui, source, dest=None, **opts):
-    if dest is None:
-        dest = hg.defaultdest(source)
-    if opts.get('all_largefiles') and not hg.islocal(dest):
+    d = dest
+    if d is None:
+        d = hg.defaultdest(source)
+    if opts.get('all_largefiles') and not hg.islocal(d):
             raise util.Abort(_(
             '--all-largefiles is incompatible with non-local destination %s' %
-            dest))
+            d))
     result = hg.clone(ui, opts, source, dest,
                       pull=opts.get('pull'),
                       stream=opts.get('uncompressed'),
diff --git a/tests/test-largefiles.t b/tests/test-largefiles.t
--- a/tests/test-largefiles.t
+++ b/tests/test-largefiles.t
@@ -707,6 +707,16 @@ Test cloning with --all-largefiles flag
   commit: (clean)
   update: 8 new changesets (update)
 
+  $ mkdir xyz
+  $ cd xyz
+  $ hg clone ../a
+  destination directory: a
+  updating to branch default
+  5 files updated, 0 files merged, 0 files removed, 0 files unresolved
+  getting changed largefiles
+  3 largefiles updated, 0 removed
+  $ cd ..
+
   $ hg clone --all-largefiles a ssh://localhost/a
   abort: --all-largefiles is incompatible with non-local destination ssh://localhost/a
   [255]