Mercurial > hgsubversion
changeset 323:067914ecb4eb
push: Fix a bug in deletion of an entire tree.
This bug meant that if an entire subtree of the repo was
deleted and there were files at varying levels of the
hierarchy, then some of the files at higher levels
might escape deletion when the revision was pushed to svn.
author | Augie Fackler <durin42@gmail.com> |
---|---|
date | Fri, 08 May 2009 16:26:33 -0500 (2009-05-08) |
parents | 05cd4a5138bf |
children | c4061e57974c 235022089da6 |
files | cmdutil.py tests/test_push_dirs.py tests/test_push_renames.py |
diffstat | 3 files changed, 50 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/cmdutil.py +++ b/cmdutil.py @@ -245,7 +245,7 @@ def commit_from_rev(ui, repo, rev_ctx, h for d in deleteddirs2: pos = d.rfind('/') if pos >= 0 and d[:pos] in deleteddirs: - deleteddirs.remove(d[:pos]) + deleteddirs.remove(d) newcopies = {} for source, dest in copies.iteritems():
--- a/tests/test_push_dirs.py +++ b/tests/test_push_dirs.py @@ -47,7 +47,7 @@ class TestPushDirectories(test_util.Test self.commitchanges(changes) self.pushrevisions() self.assertEqual(self.svnls('trunk'), - ['d2', 'd2/b', 'd31', 'd31/d32', 'd31/d32/a', 'd31/d32/d33']) + ['d2', 'd2/b', 'd31', 'd31/d32', 'd31/d32/a', ]) class TestPushDirsNotAtRoot(test_util.TestBase):
--- a/tests/test_push_renames.py +++ b/tests/test_push_renames.py @@ -49,6 +49,54 @@ class TestPushRenames(test_util.TestBase # self._debug_print_copies(tip) self.assertchanges(changes, tip) + def test_push_rename_tree(self): + repo = self.repo + + changes = [ + ('geek/alpha', 'geek/alpha', 'content',), + ('geek/beta', 'geek/beta', 'content',), + ('geek/delta', 'geek/delta', 'content',), + ('geek/gamma', 'geek/gamma', 'content',), + ('geek/later/pi', 'geek/later/pi', 'content geek/later/pi',), + ('geek/later/rho', 'geek/later/rho', 'content geek/later/rho', ), + ('geek/other/blah', 'geek/other/blah', 'content geek/other/blah', ), + ('geek/other/another/layer', 'geek/other/another/layer', 'content deep file', ), + ] + + self.commitchanges(changes) + self.pushrevisions() + self.assertchanges(changes, self.repo['tip']) + + changes = [ + # rename (copy + remove) all of geek to greek + ('geek/alpha', 'greek/alpha', None, ), + ('geek/beta', 'greek/beta', None, ), + ('geek/delta', 'greek/delta', None, ), + ('geek/gamma', 'greek/gamma', None, ), + ('geek/later/pi', 'greek/later/pi', None, ), + ('geek/later/rho', 'greek/later/rho', None, ), + ('geek/other/blah', 'greek/other/blah', None, ), + ('geek/other/another/layer', 'greek/other/another/layer', None, ), + + ('geek/alpha', None, None, ), + ('geek/beta', None, None, ), + ('geek/delta', None, None, ), + ('geek/gamma', None, None, ), + ('geek/later/pi', None, None, ), + ('geek/later/rho', None, None, ), + ('geek/other/blah', None, None, ), + ('geek/other/another/layer', None, None, ), + ] + self.commitchanges(changes) + self.pushrevisions() + # print '\n'.join(sorted(self.svnls('trunk'))) + assert reduce(lambda x, y: x and y, + ('geek' not in f for f in self.svnls('trunk'))),( + 'This failure means rename of an entire tree is broken.' + ' There is a print on the preceding line commented out ' + 'that should help you.') + + def suite(): all = [unittest.TestLoader().loadTestsFromTestCase(TestPushRenames), ]