[svn r45994] Adding support for locking and non-recursive commits (a somewhat obscure

feature, but was easy to implement) to svnwc. Not tested on SVN 1.2 (anyone?).

--HG--
branch : trunk
This commit is contained in:
guido 2007-08-26 14:31:20 +02:00
parent ea8038db10
commit ee7dc229f9
2 changed files with 78 additions and 8 deletions

View File

@ -226,6 +226,50 @@ class TestWCSvnCommandPath(CommonSvnTests):
p.remove(rec=1)
f.remove()
def test_lock_unlock(self):
root = self.root
somefile = root.join('somefile')
somefile.ensure(file=True)
# not yet added to repo
py.test.raises(py.process.cmdexec.Error, 'somefile.lock()')
somefile.write('foo')
somefile.commit('test')
assert somefile.check(versioned=True)
somefile.lock()
try:
locked = root.status().locked
assert len(locked) == 1
assert str(locked[0]) == str(somefile)
#assert somefile.locked()
py.test.raises(Exception, 'somefile.lock()')
finally:
somefile.unlock()
#assert not somefile.locked()
locked = root.status().locked
assert locked == []
py.test.raises(Exception, 'somefile,unlock()')
somefile.remove()
def test_commit_nonrecursive(self):
root = self.root
somedir = root.join('sampledir')
somefile = somedir.join('otherfile')
somefile.write('foo')
somedir.propset('foo', 'bar')
status = somedir.status()
assert len(status.prop_modified) == 1
assert len(status.modified) == 1
somedir.commit('non-recursive commit', rec=0)
status = somedir.status()
assert len(status.prop_modified) == 0
assert len(status.modified) == 1
somedir.commit('recursive commit')
status = somedir.status()
assert len(status.prop_modified) == 0
assert len(status.modified) == 0
#def test_log(self):
# l = self.root.log()
# assert len(l) == 3 # might need to be upped if more tests are added

View File

@ -199,6 +199,28 @@ class SvnWCCommandPath(common.FSPathBase):
_rex_status = re.compile(r'\s+(\d+|-)\s+(\S+)\s+(\S+)\s+(.*)')
def lock(self):
""" set a lock (exclusive) on the resource """
out = self._svn('lock').strip()
if not out:
# warning or error, raise exception
raise Exception(out[4:])
def unlock(self):
""" unset a previously set lock """
out = self._svn('unlock').strip()
if out.startswith('svn:'):
# warning or error, raise exception
raise Exception(out[4:])
def cleanup(self):
""" remove any locks from the resource """
# XXX should be fixed properly!!!
try:
self.unlock()
except:
pass
def status(self, updates=0, rec=0, externals=0):
""" return (collective) Status object for this file. """
# http://svnbook.red-bean.com/book.html#svn-ch-3-sect-4.3.1
@ -234,7 +256,7 @@ class SvnWCCommandPath(common.FSPathBase):
#print "processing %r" % line
flags, rest = line[:8], line[8:]
# first column
c0,c1,c2,c3,c4,x5,x6,c7 = flags
c0,c1,c2,c3,c4,c5,x6,c7 = flags
#if '*' in line:
# print "flags", repr(flags), "rest", repr(rest)
@ -291,7 +313,8 @@ class SvnWCCommandPath(common.FSPathBase):
if c1 == 'M':
rootstatus.prop_modified.append(wcpath)
if c2 == 'L':
# XXX do we cover all client versions here?
if c2 == 'L' or c5 == 'K':
rootstatus.locked.append(wcpath)
if c7 == '*':
rootstatus.update_available.append(wcpath)
@ -333,11 +356,14 @@ class SvnWCCommandPath(common.FSPathBase):
return result
_rex_commit = re.compile(r'.*Committed revision (\d+)\.$', re.DOTALL)
def commit(self, message=""):
"""commit() returns None if there was nothing to commit
and the revision number of the commit otherwise.
"""
out = self._svn('commit -m "%s"' % message)
def commit(self, msg='', rec=1):
""" commit with support for non-recursive commits """
from py.__.path.svn import cache
# XXX i guess escaping should be done better here?!?
cmd = 'commit -m "%s" --force-log' % (msg.replace('"', '\\"'),)
if not rec:
cmd += ' -N'
out = self._svn(cmd)
try:
del cache.info[self]
except KeyError:
@ -540,7 +566,7 @@ if verbose is True, then the LogEntry instances also know which files changed.
class WCStatus:
attrnames = ('modified','added', 'conflict', 'unchanged', 'external',
'deleted', 'prop_modified', 'unknown', 'update_available',
'incomplete', 'kindmismatch', 'ignored'
'incomplete', 'kindmismatch', 'ignored', 'locked'
)
def __init__(self, wcpath, rev=None, modrev=None, author=None):