Rev 5400: Fix lines too long and vertical spaces. in file:///home/vila/src/bzr/reviews/mergetools/
Vincent Ladeuil
v.ladeuil+lp at free.fr
Wed Nov 24 16:56:34 GMT 2010
At file:///home/vila/src/bzr/reviews/mergetools/
------------------------------------------------------------
revno: 5400
revision-id: v.ladeuil+lp at free.fr-20101124165634-7j4hqvtoe3t0qqp9
parent: gordon at doxxx.net-20101112033204-csd2tf2a6czjpeae
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: mergetools
timestamp: Wed 2010-11-24 17:56:34 +0100
message:
Fix lines too long and vertical spaces.
-------------- next part --------------
=== modified file 'bzrlib/help_topics/en/configuration.txt'
--- a/bzrlib/help_topics/en/configuration.txt 2010-11-12 03:32:04 +0000
+++ b/bzrlib/help_topics/en/configuration.txt 2010-11-24 16:56:34 +0000
@@ -592,8 +592,8 @@
~~~~~~~~~~~~~~~~~
Defines an external merge tool called <name> with the given command-line.
-Arguments containing spaces should be quoted using single or double quotes. The
-executable may omit its path if it can be found on the PATH.
+Arguments containing spaces should be quoted using single or double
+quotes. The executable may omit its path if it can be found on the PATH.
The following markers can be used in the command-line to substitute filenames
involved in the merge conflict:
=== modified file 'bzrlib/mergetools.py'
--- a/bzrlib/mergetools.py 2010-11-12 01:27:00 +0000
+++ b/bzrlib/mergetools.py 2010-11-24 16:56:34 +0000
@@ -67,6 +67,7 @@
class MergeTool(object):
+
def __init__(self, name, commandline):
"""Initializes the merge tool with a name and a command-line (a string
or sequence of strings).
=== modified file 'bzrlib/tests/test_mergetools.py'
--- a/bzrlib/tests/test_mergetools.py 2010-11-12 01:27:00 +0000
+++ b/bzrlib/tests/test_mergetools.py 2010-11-24 16:56:34 +0000
@@ -27,6 +27,7 @@
class TestBasics(tests.TestCase):
+
def setUp(self):
super(TestBasics, self).setUp()
self.tool = mergetools.MergeTool('sometool',
@@ -97,6 +98,7 @@
class TestUnicodeBasics(tests.TestCase):
+
def setUp(self):
super(TestUnicodeBasics, self).setUp()
self.tool = mergetools.MergeTool(u'someb\u0414r',
@@ -168,6 +170,7 @@
class TestMergeToolOperations(tests.TestCaseInTempDir):
+
def test_filename_substitution(self):
def dummy_invoker(executable, args, cleanup):
self._commandline = [executable] + args
@@ -235,11 +238,13 @@
class TestModuleFunctions(tests.TestCaseInTempDir):
+
def test_get_merge_tools(self):
conf = config.GlobalConfig()
conf.set_user_option('mergetools.kdiff3', 'kdiff3 %b %t %o -o %r')
conf.set_user_option('mergetools.winmergeu', 'winmergeu %r')
- conf.set_user_option('mergetools.funkytool', 'funkytool "arg with spaces" %T')
+ conf.set_user_option('mergetools.funkytool',
+ 'funkytool "arg with spaces" %T')
tools = mergetools.get_merge_tools(conf)
self.log(repr(tools))
self.assertEqual(3, len(tools))
@@ -300,7 +305,8 @@
del tools[0]
mergetools.set_merge_tools(tools, conf)
self.assertEqual(sorted([
- ('mergetools.funkytool', 'funkytool "arg with spaces" %T', 'DEFAULT', 'bazaar'),
+ ('mergetools.funkytool', 'funkytool "arg with spaces" %T',
+ 'DEFAULT', 'bazaar'),
('mergetools.winmergeu', 'winmergeu %r', 'DEFAULT', 'bazaar'),
]),
sorted(conf._get_options()))
More information about the bazaar-commits
mailing list