changeset 17159:36a3016811d1

localrepo: use the path relative to "self.vfs" instead of "path" argument As a part of migration to vfs, this patch uses "self.root", which can be recognized as the path relative to "self.vfs", instead of "path" argument. This fix allows to make invocations of "util.makedirs()" and "os.path.exists()" while ensuring repository directory in "localrepository.__init__()" ones indirectly via vfs. But this fix also raises issue 2528: "hg clone" with empty destination. "path" argument is empty in many cases, so this issue can't be fixed in the view of "localrepository.__init__()". Before this patch, it is fixed by empty-ness check ("not name") of exception handler in "util.makedirs()". try: os.mkdir(name) except OSError, err: if err.errno == errno.EEXIST: return if err.errno != errno.ENOENT or not name: raise This requires "localrepository.__init__()" to invoke "util.makedirs()" with "path" instead of "self.root", because empty "path" is treated as "current directory" and "self.root" becomes valid path. But "hg clone" with empty destination can be detected also in "hg.clone()" before "localrepository.__init__()" invocation, so this patch re-fixes issue2528 by checking it in "hg.clone()".
author FUJIWARA Katsunori <foozy@lares.dti.ne.jp>
date Fri, 06 Jul 2012 18:45:27 +0900
parents 60338880d265
children 22b9b1d2f5d4
files mercurial/hg.py mercurial/localrepo.py tests/test-bundle.t tests/test-clone.t
diffstat 4 files changed, 7 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/hg.py	Fri Jul 06 18:45:27 2012 +0900
+++ b/mercurial/hg.py	Fri Jul 06 18:45:27 2012 +0900
@@ -263,6 +263,8 @@
     dest = util.urllocalpath(dest)
     source = util.urllocalpath(source)
 
+    if not dest:
+        raise util.Abort(_("empty destination path is not valid"))
     if os.path.exists(dest):
         if not os.path.isdir(dest):
             raise util.Abort(_("destination '%s' already exists") % dest)
--- a/mercurial/localrepo.py	Fri Jul 06 18:45:27 2012 +0900
+++ b/mercurial/localrepo.py	Fri Jul 06 18:45:27 2012 +0900
@@ -60,8 +60,8 @@
 
         if not os.path.isdir(self.path):
             if create:
-                if not os.path.exists(path):
-                    util.makedirs(path)
+                if not os.path.exists(self.root):
+                    util.makedirs(self.root)
                 util.makedir(self.path, notindexed=True)
                 requirements = self._baserequirements(create)
                 if self.ui.configbool('format', 'usestore', True):
--- a/tests/test-bundle.t	Fri Jul 06 18:45:27 2012 +0900
+++ b/tests/test-bundle.t	Fri Jul 06 18:45:27 2012 +0900
@@ -420,7 +420,7 @@
 recurse infinitely (issue 2528)
 
   $ hg clone full.hg ''
-  abort: * (glob)
+  abort: empty destination path is not valid
   [255]
 
 test for http://mercurial.selenic.com/bts/issue216
--- a/tests/test-clone.t	Fri Jul 06 18:45:27 2012 +0900
+++ b/tests/test-clone.t	Fri Jul 06 18:45:27 2012 +0900
@@ -43,7 +43,7 @@
 Invalid dest '' must abort:
 
   $ hg clone . ''
-  abort: * (glob)
+  abort: empty destination path is not valid
   [255]
 
 No update, with debug option:
@@ -112,7 +112,7 @@
 Invalid dest '' with --pull must abort (issue2528):
 
   $ hg clone --pull a ''
-  abort: * (glob)
+  abort: empty destination path is not valid
   [255]
 
 Clone to '.':