tests/test-copy
author Stephen Darnell
Wed, 14 Sep 2005 12:22:20 -0500
changeset 1241 3b4f05ff3130
parent 1043 9344f5dd4488
child 1252 94f38724283f
permissions -rwxr-xr-x
Add support for cloning with hardlinks on windows. In order to use hardlinks, the win32file module is needed, and this is present in ActivePython. If it isn't present, or hardlinks are not supported on the underlying filesystem, a regular copy is used. When using hardlinks the biggest benefit is probably the saving in space, but cloning can be much quicker. For example cloning the Xen tree (non trivial) without an update goes from about 95s to 15s. Unix-like platforms should be unaffected, although should be more tolerant on filesystems that don't support hard links. (tweaked by mpm to deal with new copyfiles function) --- hg.orig/mercurial/commands.py 2005-09-13 19:32:53.000000000 -0500 +++ hg/mercurial/commands.py 2005-09-14 12:11:34.000000000 -0500 @@ -620,10 +620,6 @@ def clone(ui, source, dest=None, **opts) if other.dev() != -1: abspath = os.path.abspath(source) - copyfile = (os.stat(dest).st_dev == other.dev() - and getattr(os, 'link', None) or shutil.copy2) - if copyfile is not shutil.copy2: - ui.note("cloning by hardlink\n") # we use a lock here because if we race with commit, we can # end up with extra data in the cloned revlogs that's not @@ -638,7 +634,7 @@ def clone(ui, source, dest=None, **opts) for f in files.split(): src = os.path.join(source, ".hg", f) dst = os.path.join(dest, ".hg", f) - util.copyfiles(src, dst, copyfile) + util.copyfiles(src, dst) repo = hg.repository(ui, dest) Index: hg/mercurial/util.py =================================================================== --- hg.orig/mercurial/util.py 2005-09-08 00:15:25.000000000 -0500 +++ hg/mercurial/util.py 2005-09-14 12:16:49.000000000 -0500 @@ -12,7 +12,7 @@ platform-specific details from the core. import os, errno from demandload import * -demandload(globals(), "re cStringIO") +demandload(globals(), "re cStringIO shutil") def binary(s): """return true if a string is binary data using diff's heuristic""" @@ -217,17 +217,28 @@ def rename(src, dst): os.unlink(dst) os.rename(src, dst) -def copyfiles(src, dst, copyfile): - """Copy a directory tree, files are copied using 'copyfile'.""" +def copyfiles(src, dst, hardlink=None): + """Copy a directory tree using hardlinks if possible""" + + if hardlink is None: + hardlink = (os.stat(src).st_dev == + os.stat(os.path.dirname(dst)).st_dev) if os.path.isdir(src): os.mkdir(dst) for name in os.listdir(src): srcname = os.path.join(src, name) dstname = os.path.join(dst, name) - copyfiles(srcname, dstname, copyfile) + copyfiles(srcname, dstname, hardlink) else: - copyfile(src, dst) + if hardlink: + try: + os_link(src, dst) + except: + hardlink = False + shutil.copy2(src, dst) + else: + shutil.copy2(src, dst) def opener(base): """ @@ -244,13 +255,13 @@ def opener(base): if mode[0] != "r": try: - s = os.stat(f) + nlink = nlinks(f) except OSError: d = os.path.dirname(f) if not os.path.isdir(d): os.makedirs(d) else: - if s.st_nlink > 1: + if nlink > 1: file(f + ".tmp", "wb").write(file(f, "rb").read()) rename(f+".tmp", f) @@ -266,10 +277,41 @@ def _makelock_file(info, pathname): def _readlock_file(pathname): return file(pathname).read() +def nlinks(pathname): + """Return number of hardlinks for the given file.""" + return os.stat(pathname).st_nlink + +if hasattr(os, 'link'): + os_link = os.link +else: + def os_link(src, dst): + raise OSError(0, "Hardlinks not supported") + # Platform specific variants if os.name == 'nt': nulldev = 'NUL:' + try: # ActivePython can create hard links using win32file module + import win32file + + def os_link(src, dst): # NB will only succeed on NTFS + win32file.CreateHardLink(dst, src) + + def nlinks(pathname): + """Return number of hardlinks for the given file.""" + try: + fh = win32file.CreateFile(pathname, + win32file.GENERIC_READ, win32file.FILE_SHARE_READ, + None, win32file.OPEN_EXISTING, 0, None) + res = win32file.GetFileInformationByHandle(fh) + fh.Close() + return res[7] + except: + return os.stat(pathname).st_nlink + + except ImportError: + pass + def is_exec(f, last): return last
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
544
3d4d5f2aba9a Remove bashisms and use /bin/sh instead of /bin/bash.
Thomas Arendsen Hein <thomas@intevation.de>
parents: 464
diff changeset
     1
#!/bin/sh
363
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
     2
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
     3
hg init
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
     4
echo a > a
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
     5
hg add a
749
7e4843b7efd2 Update tests to use commit -m and default -u
mpm@selenic.com
parents: 591
diff changeset
     6
hg commit -m "1" -d "0 0"
363
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
     7
hg status
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
     8
cp a b
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
     9
hg copy a b
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
    10
hg status
749
7e4843b7efd2 Update tests to use commit -m and default -u
mpm@selenic.com
parents: 591
diff changeset
    11
hg --debug commit -m "2" -d "0 0"
1043
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    12
echo "we should see two history entries"
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    13
hg history -v
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    14
echo "we should see one log entry for a"
363
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
    15
hg log a
1043
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    16
echo "this should show a revision linked to changeset 0"
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    17
hg debugindex .hg/data/a.i
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    18
echo "we should see one log entry for b"
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    19
hg log b
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    20
echo "this should show a revision linked to changeset 1"
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    21
hg debugindex .hg/data/b.i
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    22
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    23
echo "this should show the rename information in the metadata"
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    24
hg debugdata .hg/data/b.d 0 | head -n 3 | tail -n 2
9344f5dd4488 Update tests
mpm@selenic.com
parents: 814
diff changeset
    25
800
ec85f9e6f3b1 Don't use 'set -x', fix exports, sed and hexdump usage for Solaris.
Thomas Arendsen Hein <thomas@intevation.de>
parents: 749
diff changeset
    26
md5sum .hg/data/b.d
462
512d76558bd4 Fix pipe timing for copy test
mpm@selenic.com
parents: 363
diff changeset
    27
hg cat b > bsum
512d76558bd4 Fix pipe timing for copy test
mpm@selenic.com
parents: 363
diff changeset
    28
md5sum bsum
512d76558bd4 Fix pipe timing for copy test
mpm@selenic.com
parents: 363
diff changeset
    29
hg cat a > asum
512d76558bd4 Fix pipe timing for copy test
mpm@selenic.com
parents: 363
diff changeset
    30
md5sum asum
363
ae96b7e1318d Add hg copy
mpm@selenic.com
parents:
diff changeset
    31
hg verify