On Fri, 19 May 2006 18:41:33 +0530, Giovanni Bajo <[EMAIL PROTECTED]>
wrote:
Madan U Sreenivasan wrote:
+ oldest_src_rev =
get_created_rev(opts["head-url"])
+ src_pre_exist_range =
RevisionSet("1-%d" % oldest_src_rev)
Is this test really necessary? Doesn't the
merge command just fail if you
try reverting a revision at which the URL
did not exist?
yes, it does... but wouldnt it make sense to say the rollback command
failed instead of letting the user see a merge failure? Much more
cleaner, I think.
What happens, exactly, if you remove that test? What's the actual output?
======================================================================
ERROR: `svnmerge rollback' should error out if range contains revisions
prior to
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/madan/wc/svnmerge/contrib/client-side/svnmerge_test.py",
line 771, in testRollbackOutsidePossibleRange
match = expected_error)
File "/home/madan/wc/svnmerge/contrib/client-side/svnmerge_test.py",
line 180, in svnmerge
return self.svnmerge2(cmds.split(), *args, **kwargs)
File "/home/madan/wc/svnmerge/contrib/client-side/svnmerge_test.py",
line 191, in svnmerge2
ret = svnmerge.main(args)
File "/home/madan/wc/svnmerge/contrib/client-side/svnmerge.py", line
1849, in main
cmd(branch_dir, branch_props)
File "/home/madan/wc/svnmerge/contrib/client-side/svnmerge.py", line
1364, in __call__
return self.func(*args, **kwargs)
File "/home/madan/wc/svnmerge/contrib/client-side/svnmerge.py", line
1265, in action_rollback
svn_command("merge -r %d:%d %s %s" % \
File "/home/madan/wc/svnmerge/contrib/client-side/svnmerge.py", line
246, in svn_command
split_lines=False)
File "/home/madan/wc/svnmerge/contrib/client-side/svnmerge.py", line
240, in launchsvn
return launch(cmd, **kwargs)
File "/home/madan/wc/svnmerge/contrib/client-side/svnmerge.py", line
231, in launch
raise LaunchError(ret, cmd, out)
LaunchError: (1, 'svn merge -r 13:1 file:///tmp/__svnmerge_test/repo/trunk
.', "svn: The location for 'file:///tmp/__svnmerge_test/repo/trunk' for
revision 1 does not exist in the repository or refers to an unrelated
object\n")
The last line gives the 'svn merge' error....
Regards,
Madan.
_______________________________________________
Svnmerge mailing list
[email protected]
http://www.orcaware.com/mailman/listinfo/svnmerge
