/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
4763.2.4 by John Arbash Meinel
merge bzr.2.1 in preparation for NEWS entry.
1
# Copyright (C) 2005-2010 Canonical Ltd
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
2
#
1167 by Martin Pool
- split commit message editor functions out into own file
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
7
#
1167 by Martin Pool
- split commit message editor functions out into own file
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
12
#
1167 by Martin Pool
- split commit message editor functions out into own file
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
4183.7.1 by Sabin Iacob
update FSF mailing address
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
1167 by Martin Pool
- split commit message editor functions out into own file
16
17
18
"""Commit message editor support."""
19
1185.85.2 by John Arbash Meinel
[patch] from Ivan Vilata i Balaguer: infotext should be encoded from unicode to flat strings
20
import codecs
1167 by Martin Pool
- split commit message editor functions out into own file
21
import os
1185.50.91 by John Arbash Meinel
Fix import order for msgeditor.py
22
from subprocess import call
1185.50.90 by John Arbash Meinel
[patch] Olaf Conradi: missing sys import in msgeditor.py after platform test fix
23
import sys
1442.1.3 by Robert Collins
move editor into the config file too
24
3224.5.33 by Andrew Bennetts
Tidy imports in bzrlib/msgeditor.py as suggested by John's review.
25
from bzrlib import (
26
    config,
27
    osutils,
3910.1.1 by Andrew Bennetts
Improve error handling in msgeditor._run_editor.
28
    trace,
5137.1.1 by Robert Collins
* ``bzr commit`` will prompt before using a commit message that was
29
    transport,
30
    ui,
3224.5.33 by Andrew Bennetts
Tidy imports in bzrlib/msgeditor.py as suggested by John's review.
31
    )
2625.9.4 by Daniel Watkins
Fixed editor commit message to die cleanly if a UnicodeDecodeError is thrown.
32
from bzrlib.errors import BzrError, BadCommitMessageEncoding
4119.3.2 by Robert Collins
Migrate existing hooks over to the new HookPoint infrastructure.
33
from bzrlib.hooks import HookPoint, Hooks
1167 by Martin Pool
- split commit message editor functions out into own file
34
1185.50.90 by John Arbash Meinel
[patch] Olaf Conradi: missing sys import in msgeditor.py after platform test fix
35
1167 by Martin Pool
- split commit message editor functions out into own file
36
def _get_editor():
37
    """Return a sequence of possible editor binaries for the current platform"""
1185.1.30 by Robert Collins
Accept and tweak David Clymers BZREDITOR support patch
38
    try:
3910.1.1 by Andrew Bennetts
Improve error handling in msgeditor._run_editor.
39
        yield os.environ["BZR_EDITOR"], '$BZR_EDITOR'
1185.1.30 by Robert Collins
Accept and tweak David Clymers BZREDITOR support patch
40
    except KeyError:
41
        pass
42
1442.1.6 by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs
43
    e = config.GlobalConfig().get_editor()
1167 by Martin Pool
- split commit message editor functions out into own file
44
    if e is not None:
3910.1.1 by Andrew Bennetts
Improve error handling in msgeditor._run_editor.
45
        yield e, config.config_filename()
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
46
1668.1.13 by Martin Pool
msgeditor and bzr info improvements (olaf)
47
    for varname in 'VISUAL', 'EDITOR':
1963.2.1 by Robey Pointer
remove usage of has_key()
48
        if varname in os.environ:
3910.1.2 by Andrew Bennetts
Fix thinko.
49
            yield os.environ[varname], '$' + varname
1185.1.30 by Robert Collins
Accept and tweak David Clymers BZREDITOR support patch
50
1684.1.7 by Martin Pool
(patch) better detection of the write commit message editor to use (mpe, robertc)
51
    if sys.platform == 'win32':
52
        for editor in 'wordpad.exe', 'notepad.exe':
3910.1.1 by Andrew Bennetts
Improve error handling in msgeditor._run_editor.
53
            yield editor, None
1684.1.7 by Martin Pool
(patch) better detection of the write commit message editor to use (mpe, robertc)
54
    else:
1668.4.1 by Olaf Conradi
Make msgeditor invocation comply with Debian Policy.
55
        for editor in ['/usr/bin/editor', 'vi', 'pico', 'nano', 'joe']:
3910.1.1 by Andrew Bennetts
Improve error handling in msgeditor._run_editor.
56
            yield editor, None
1167 by Martin Pool
- split commit message editor functions out into own file
57
58
59
def _run_editor(filename):
1168 by Martin Pool
- work properly when $EDITOR contains multiple words
60
    """Try to execute an editor to edit the commit message."""
3910.1.1 by Andrew Bennetts
Improve error handling in msgeditor._run_editor.
61
    for candidate, candidate_source in _get_editor():
62
        edargs = candidate.split(' ')
1185.62.19 by John Arbash Meinel
Fix error when we can't find an editor
63
        try:
1711.4.2 by jfmeinel
current python may be running in a path that has a space, so properly quote the python exe name. for test_msgeditor
64
            ## mutter("trying editor: %r", (edargs +[filename]))
1185.62.19 by John Arbash Meinel
Fix error when we can't find an editor
65
            x = call(edargs + [filename])
66
        except OSError, e:
4948.2.2 by Martin Pool
If there's an OS error running a user-specified editor, always show it,
67
            if candidate_source is not None:
68
                # We tried this editor because some user configuration (an
69
                # environment variable or config file) said to try it.  Let
70
                # the user know their configuration is broken.
71
                trace.warning(
72
                    'Could not start editor "%s" (specified by %s): %s\n'
73
                    % (candidate, candidate_source, str(e)))
74
            continue
3376.1.1 by Martin Pool
Fix incorrect indentation (#220333)
75
            raise
1167 by Martin Pool
- split commit message editor functions out into own file
76
        if x == 0:
77
            return True
78
        elif x == 127:
79
            continue
80
        else:
81
            break
1684.1.7 by Martin Pool
(patch) better detection of the write commit message editor to use (mpe, robertc)
82
    raise BzrError("Could not start any editor.\nPlease specify one with:\n"
2135.1.3 by Matthew Fuller
Include $VISUAL in the list of env variables when we can't start an
83
                   " - $BZR_EDITOR\n - editor=/some/path in %s\n"
84
                   " - $VISUAL\n - $EDITOR" % \
1684.1.7 by Martin Pool
(patch) better detection of the write commit message editor to use (mpe, robertc)
85
                    config.config_filename())
1553.4.9 by Michael Ellerman
Make the "ignore line" in the commit message editor the "right" width, so
86
87
88
DEFAULT_IGNORE_LINE = "%(bar)s %(msg)s %(bar)s" % \
89
    { 'bar' : '-' * 14, 'msg' : 'This line and the following will be ignored' }
90
91
2258.3.1 by James Westby
Add a way to specify a template commit message.
92
def edit_commit_message(infotext, ignoreline=DEFAULT_IGNORE_LINE,
2598.6.12 by ghigo
Move the encoding of the commit message at the command line level
93
                        start_message=None):
1167 by Martin Pool
- split commit message editor functions out into own file
94
    """Let the user edit a commit message in a temp file.
95
96
    This is run if they don't give a message or
97
    message-containing file on the command line.
98
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
99
    :param infotext:    Text to be displayed at bottom of message
100
                        for the user's reference;
101
                        currently similar to 'bzr status'.
2598.6.17 by ghigo
Revert the behaviour of the function make_commit_message_template() and make_commit_message_template() to the original one.
102
103
    :param ignoreline:  The separator to use above the infotext.
104
105
    :param start_message:   The text to place above the separator, if any.
106
                            This will not be removed from the message
107
                            after the user has edited it.
108
109
    :return:    commit message or None.
110
    """
111
2598.6.18 by ghigo
Update the tests to the new *_encoded() functions
112
    if not start_message is None:
3224.5.1 by Andrew Bennetts
Lots of assorted hackery to reduce the number of imports for common operations. Improves 'rocks', 'st' and 'help' times by ~50ms on my laptop.
113
        start_message = start_message.encode(osutils.get_user_encoding())
114
    infotext = infotext.encode(osutils.get_user_encoding(), 'replace')
2804.4.1 by Alexander Belchenko
some win32-specific fixes for selftest
115
    return edit_commit_message_encoded(infotext, ignoreline, start_message)
2598.6.17 by ghigo
Revert the behaviour of the function make_commit_message_template() and make_commit_message_template() to the original one.
116
2598.6.21 by ghigo
updates on the basis of Jhon email
117
2598.6.17 by ghigo
Revert the behaviour of the function make_commit_message_template() and make_commit_message_template() to the original one.
118
def edit_commit_message_encoded(infotext, ignoreline=DEFAULT_IGNORE_LINE,
2598.6.23 by ghigo
Cosmetic clean up
119
                                start_message=None):
2598.6.17 by ghigo
Revert the behaviour of the function make_commit_message_template() and make_commit_message_template() to the original one.
120
    """Let the user edit a commit message in a temp file.
121
122
    This is run if they don't give a message or
123
    message-containing file on the command line.
124
125
    :param infotext:    Text to be displayed at bottom of message
126
                        for the user's reference;
127
                        currently similar to 'bzr status'.
2598.6.12 by ghigo
Move the encoding of the commit message at the command line level
128
                        The string is already encoded
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
129
130
    :param ignoreline:  The separator to use above the infotext.
131
132
    :param start_message:   The text to place above the separator, if any.
133
                            This will not be removed from the message
134
                            after the user has edited it.
2598.6.15 by ghigo
Update the param description
135
                            The string is already encoded
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
136
137
    :return:    commit message or None.
1167 by Martin Pool
- split commit message editor functions out into own file
138
    """
1704.2.21 by Martin Pool
Fix unbound local when handling failure to write message template
139
    msgfilename = None
1167 by Martin Pool
- split commit message editor functions out into own file
140
    try:
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
141
        msgfilename, hasinfo = _create_temp_file_with_commit_template(
2598.6.12 by ghigo
Move the encoding of the commit message at the command line level
142
                                    infotext, ignoreline, start_message)
5137.1.1 by Robert Collins
* ``bzr commit`` will prompt before using a commit message that was
143
        if not msgfilename:
144
            return None
145
        basename = osutils.basename(msgfilename)
146
        msg_transport = transport.get_transport(osutils.dirname(msgfilename))
147
        reference_content = msg_transport.get_bytes(basename)
148
        if not _run_editor(msgfilename):
149
            return None
150
        edited_content = msg_transport.get_bytes(basename)
151
        if edited_content == reference_content:
5416.1.10 by Martin Pool
Use confirm_action before proceeding with an unedited commit message template
152
            if not ui.ui_factory.confirm_action(
153
                "Commit message was not edited, use anyway",
154
                "bzrlib.msgeditor.unchanged",
155
                {}):
5137.1.2 by Robert Collins
Review feedback: add comment about return value and an additional test.
156
                # Returning "" makes cmd_commit raise 'empty commit message
157
                # specified' which is a reasonable error, given the user has
158
                # rejected using the unedited template.
5137.1.1 by Robert Collins
* ``bzr commit`` will prompt before using a commit message that was
159
                return ""
1167 by Martin Pool
- split commit message editor functions out into own file
160
        started = False
161
        msg = []
162
        lastline, nlines = 0, 0
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
163
        # codecs.open() ALWAYS opens file in binary mode but we need text mode
164
        # 'rU' mode useful when bzr.exe used on Cygwin (bialix 20070430)
165
        f = file(msgfilename, 'rU')
2625.9.4 by Daniel Watkins
Fixed editor commit message to die cleanly if a UnicodeDecodeError is thrown.
166
        try:
2765.1.2 by Ian Clatworthy
fix try/except/finally nesting for Python 2.4
167
            try:
3224.5.1 by Andrew Bennetts
Lots of assorted hackery to reduce the number of imports for common operations. Improves 'rocks', 'st' and 'help' times by ~50ms on my laptop.
168
                for line in codecs.getreader(osutils.get_user_encoding())(f):
2765.1.2 by Ian Clatworthy
fix try/except/finally nesting for Python 2.4
169
                    stripped_line = line.strip()
170
                    # strip empty line before the log message starts
171
                    if not started:
172
                        if stripped_line != "":
173
                            started = True
174
                        else:
175
                            continue
176
                    # check for the ignore line only if there
177
                    # is additional information at the end
178
                    if hasinfo and stripped_line == ignoreline:
179
                        break
180
                    nlines += 1
181
                    # keep track of the last line that had some content
2625.9.4 by Daniel Watkins
Fixed editor commit message to die cleanly if a UnicodeDecodeError is thrown.
182
                    if stripped_line != "":
2765.1.2 by Ian Clatworthy
fix try/except/finally nesting for Python 2.4
183
                        lastline = nlines
184
                    msg.append(line)
185
            except UnicodeDecodeError:
186
                raise BadCommitMessageEncoding()
2625.9.8 by Daniel Watkins
Updated as per poolie's !tweak.
187
        finally:
188
            f.close()
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
189
1393.3.3 by Jelmer Vernooij
Add test for empty commit messages.
190
        if len(msg) == 0:
1393.3.2 by Jelmer Vernooij
Fix error message when an empty commit message was specified (when using an editor). Previously bzr warned that it wanted either --message or --file.
191
            return ""
1167 by Martin Pool
- split commit message editor functions out into own file
192
        # delete empty lines at the end
193
        del msg[lastline:]
194
        # add a newline at the end, if needed
195
        if not msg[-1].endswith("\n"):
196
            return "%s%s" % ("".join(msg), "\n")
197
        else:
198
            return "".join(msg)
199
    finally:
200
        # delete the msg file in any case
1704.2.21 by Martin Pool
Fix unbound local when handling failure to write message template
201
        if msgfilename is not None:
202
            try:
203
                os.unlink(msgfilename)
204
            except IOError, e:
3910.1.1 by Andrew Bennetts
Improve error handling in msgeditor._run_editor.
205
                trace.warning(
206
                    "failed to unlink %s: %s; ignored", msgfilename, e)
1167 by Martin Pool
- split commit message editor functions out into own file
207
1185.33.72 by Martin Pool
Fix commit message template for non-ascii files, and add test for handling of
208
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
209
def _create_temp_file_with_commit_template(infotext,
210
                                           ignoreline=DEFAULT_IGNORE_LINE,
5537.1.1 by Vincent Ladeuil
Create commit message files in TMPDIR instead of the current dir
211
                                           start_message=None,
212
                                           tmpdir=None):
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
213
    """Create temp file and write commit template in it.
214
5537.1.1 by Vincent Ladeuil
Create commit message files in TMPDIR instead of the current dir
215
    :param infotext: Text to be displayed at bottom of message for the
216
        user's reference; currently similar to 'bzr status'.  The text is
217
        already encoded.
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
218
219
    :param ignoreline:  The separator to use above the infotext.
220
5537.1.1 by Vincent Ladeuil
Create commit message files in TMPDIR instead of the current dir
221
    :param start_message: The text to place above the separator, if any.
222
        This will not be removed from the message after the user has edited
223
        it.  The string is already encoded
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
224
225
    :return:    2-tuple (temp file name, hasinfo)
226
    """
227
    import tempfile
228
    tmp_fileno, msgfilename = tempfile.mkstemp(prefix='bzr_log.',
5537.1.1 by Vincent Ladeuil
Create commit message files in TMPDIR instead of the current dir
229
                                               dir=tmpdir, text=True)
3004.1.4 by Daniel Watkins
Modified fix as suggested by John on-list.
230
    msgfile = os.fdopen(tmp_fileno, 'w')
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
231
    try:
232
        if start_message is not None:
2598.6.12 by ghigo
Move the encoding of the commit message at the command line level
233
            msgfile.write("%s\n" % start_message)
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
234
2598.6.20 by ghigo
Small clean up
235
        if infotext is not None and infotext != "":
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
236
            hasinfo = True
2598.6.12 by ghigo
Move the encoding of the commit message at the command line level
237
            msgfile.write("\n\n%s\n\n%s" %(ignoreline, infotext))
2472.4.1 by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added
238
        else:
239
            hasinfo = False
240
    finally:
241
        msgfile.close()
242
243
    return (msgfilename, hasinfo)
244
2598.6.20 by ghigo
Small clean up
245
2598.6.17 by ghigo
Revert the behaviour of the function make_commit_message_template() and make_commit_message_template() to the original one.
246
def make_commit_message_template(working_tree, specific_files):
1185.33.72 by Martin Pool
Fix commit message template for non-ascii files, and add test for handling of
247
    """Prepare a template file for a commit into a branch.
248
2598.6.17 by ghigo
Revert the behaviour of the function make_commit_message_template() and make_commit_message_template() to the original one.
249
    Returns a unicode string containing the template.
1185.33.72 by Martin Pool
Fix commit message template for non-ascii files, and add test for handling of
250
    """
251
    # TODO: make provision for this to be overridden or modified by a hook
252
    #
253
    # TODO: Rather than running the status command, should prepare a draft of
254
    # the revision to be committed, then pause and ask the user to
255
    # confirm/write a message.
256
    from StringIO import StringIO       # must be unicode-safe
1551.2.9 by Aaron Bentley
Fix status to work with checkouts
257
    from bzrlib.status import show_tree_status
2598.6.5 by ghigo
On the basis of the email from Martin, Aaron I changed the encoding logic
258
    status_tmp = StringIO()
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
259
    show_tree_status(working_tree, specific_files=specific_files,
3968.1.1 by Colin D Bennett
Show all pending merge revisions in the commit message template.
260
                     to_file=status_tmp, verbose=True)
2598.6.17 by ghigo
Revert the behaviour of the function make_commit_message_template() and make_commit_message_template() to the original one.
261
    return status_tmp.getvalue()
262
263
264
def make_commit_message_template_encoded(working_tree, specific_files,
2598.6.30 by ghigo
- Updated the identation on the basis of Aaron suggestions
265
                                         diff=None, output_encoding='utf-8'):
2598.6.17 by ghigo
Revert the behaviour of the function make_commit_message_template() and make_commit_message_template() to the original one.
266
    """Prepare a template file for a commit into a branch.
267
268
    Returns an encoded string.
269
    """
270
    # TODO: make provision for this to be overridden or modified by a hook
271
    #
272
    # TODO: Rather than running the status command, should prepare a draft of
273
    # the revision to be committed, then pause and ask the user to
274
    # confirm/write a message.
275
    from StringIO import StringIO       # must be unicode-safe
276
    from bzrlib.diff import show_diff_trees
277
2598.6.24 by ghigo
update on the basis of Aaron suggestions
278
    template = make_commit_message_template(working_tree, specific_files)
279
    template = template.encode(output_encoding, "replace")
2598.6.17 by ghigo
Revert the behaviour of the function make_commit_message_template() and make_commit_message_template() to the original one.
280
2598.6.1 by ghigo
add support for the diff at the end of the commit messages
281
    if diff:
2598.6.5 by ghigo
On the basis of the email from Martin, Aaron I changed the encoding logic
282
        stream = StringIO()
2598.6.12 by ghigo
Move the encoding of the commit message at the command line level
283
        show_diff_trees(working_tree.basis_tree(),
2598.6.26 by ghigo
small cleanup
284
                        working_tree, stream, specific_files,
285
                        path_encoding=output_encoding)
2598.6.21 by ghigo
updates on the basis of Jhon email
286
        template = template + '\n' + stream.getvalue()
2598.6.12 by ghigo
Move the encoding of the commit message at the command line level
287
288
    return template
3642.2.1 by Jelmer Vernooij
Add simple commit message template hook.
289
290
291
class MessageEditorHooks(Hooks):
292
    """A dictionary mapping hook name to a list of callables for message editor
293
    hooks.
294
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
295
    e.g. ['commit_message_template'] is the list of items to be called to
3642.2.1 by Jelmer Vernooij
Add simple commit message template hook.
296
    generate a commit message template
297
    """
298
299
    def __init__(self):
300
        """Create the default hooks.
301
302
        These are all empty initially.
303
        """
304
        Hooks.__init__(self)
4119.3.2 by Robert Collins
Migrate existing hooks over to the new HookPoint infrastructure.
305
        self.create_hook(HookPoint('commit_message_template',
306
            "Called when a commit message is being generated. "
307
            "commit_message_template is called with the bzrlib.commit.Commit "
308
            "object and the message that is known so far. "
309
            "commit_message_template must return a new message to use (which "
310
            "could be the same as it was given. When there are multiple "
311
            "hooks registered for commit_message_template, they are chained "
312
            "with the result from the first passed into the second, and so "
313
            "on.", (1, 10), None))
3642.2.1 by Jelmer Vernooij
Add simple commit message template hook.
314
315
316
hooks = MessageEditorHooks()
317
318
319
def generate_commit_message_template(commit, start_message=None):
320
    """Generate a commit message template.
321
322
    :param commit: Commit object for the active commit.
323
    :param start_message: Message to start with.
324
    :return: A start commit message or None for an empty start commit message.
325
    """
326
    start_message = None
327
    for hook in hooks['commit_message_template']:
3642.2.2 by Jelmer Vernooij
Add tests for commit_message_template hooks.
328
        start_message = hook(commit, start_message)
3642.2.1 by Jelmer Vernooij
Add simple commit message template hook.
329
    return start_message