~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/msgeditor.py

  • Committer: Aaron Bentley
  • Date: 2007-09-14 14:45:55 UTC
  • mto: This revision was merged to the branch mainline in revision 2826.
  • Revision ID: abentley@panoramicfeedback.com-20070914144555-3rynerknpoivm2xv
Documentation updates from review

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Bazaar-NG -- distributed version control
2
 
 
3
 
# Copyright (C) 2005 by Canonical Ltd
4
 
 
 
1
# Copyright (C) 2005, 2006 Canonical Ltd
 
2
#
5
3
# This program is free software; you can redistribute it and/or modify
6
4
# it under the terms of the GNU General Public License as published by
7
5
# the Free Software Foundation; either version 2 of the License, or
8
6
# (at your option) any later version.
9
 
 
 
7
#
10
8
# This program is distributed in the hope that it will be useful,
11
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
12
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
13
11
# GNU General Public License for more details.
14
 
 
 
12
#
15
13
# You should have received a copy of the GNU General Public License
16
14
# along with this program; if not, write to the Free Software
17
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
19
17
 
20
18
"""Commit message editor support."""
21
19
 
 
20
import codecs
 
21
import errno
22
22
import os
23
23
from subprocess import call
 
24
import sys
24
25
 
 
26
import bzrlib
25
27
import bzrlib.config as config
26
 
from bzrlib.errors import BzrError
 
28
from bzrlib.errors import BzrError, BadCommitMessageEncoding
 
29
from bzrlib.trace import warning, mutter
 
30
 
27
31
 
28
32
def _get_editor():
29
33
    """Return a sequence of possible editor binaries for the current platform"""
36
40
    if e is not None:
37
41
        yield e
38
42
        
39
 
    try:
40
 
        yield os.environ["EDITOR"]
41
 
    except KeyError:
42
 
        pass
 
43
    for varname in 'VISUAL', 'EDITOR':
 
44
        if varname in os.environ:
 
45
            yield os.environ[varname]
43
46
 
44
 
    if os.name == "nt":
45
 
        yield "notepad.exe"
46
 
    elif os.name == "posix":
47
 
        yield "/usr/bin/vi"
 
47
    if sys.platform == 'win32':
 
48
        for editor in 'wordpad.exe', 'notepad.exe':
 
49
            yield editor
 
50
    else:
 
51
        for editor in ['/usr/bin/editor', 'vi', 'pico', 'nano', 'joe']:
 
52
            yield editor
48
53
 
49
54
 
50
55
def _run_editor(filename):
51
56
    """Try to execute an editor to edit the commit message."""
52
57
    for e in _get_editor():
53
58
        edargs = e.split(' ')
54
 
        x = call(edargs + [filename])
 
59
        try:
 
60
            ## mutter("trying editor: %r", (edargs +[filename]))
 
61
            x = call(edargs + [filename])
 
62
        except OSError, e:
 
63
           # We're searching for an editor, so catch safe errors and continue
 
64
           if e.errno in (errno.ENOENT, ):
 
65
               continue
 
66
           raise
55
67
        if x == 0:
56
68
            return True
57
69
        elif x == 127:
58
70
            continue
59
71
        else:
60
72
            break
61
 
    raise BzrError("Could not start any editor. "
62
 
                   "Please specify $EDITOR or use ~/.bzr.conf/editor")
63
 
                          
64
 
 
65
 
def edit_commit_message(infotext, ignoreline=None):
66
 
    """Let the user edit a commit message in a temp file.
67
 
 
68
 
    This is run if they don't give a message or
69
 
    message-containing file on the command line.
70
 
 
71
 
    infotext:
72
 
        Text to be displayed at bottom of message for
73
 
        the user's reference; currently similar to
74
 
        'bzr status'.
75
 
    """
76
 
    import tempfile
77
 
    
78
 
    if ignoreline is None:
79
 
        ignoreline = "-- This line and the following will be ignored --"
80
 
        
 
73
    raise BzrError("Could not start any editor.\nPlease specify one with:\n"
 
74
                   " - $BZR_EDITOR\n - editor=/some/path in %s\n"
 
75
                   " - $VISUAL\n - $EDITOR" % \
 
76
                    config.config_filename())
 
77
 
 
78
 
 
79
DEFAULT_IGNORE_LINE = "%(bar)s %(msg)s %(bar)s" % \
 
80
    { 'bar' : '-' * 14, 'msg' : 'This line and the following will be ignored' }
 
81
 
 
82
 
 
83
def edit_commit_message(infotext, ignoreline=DEFAULT_IGNORE_LINE,
 
84
                        start_message=None):
 
85
    """Let the user edit a commit message in a temp file.
 
86
 
 
87
    This is run if they don't give a message or
 
88
    message-containing file on the command line.
 
89
 
 
90
    :param infotext:    Text to be displayed at bottom of message
 
91
                        for the user's reference;
 
92
                        currently similar to 'bzr status'.
 
93
 
 
94
    :param ignoreline:  The separator to use above the infotext.
 
95
 
 
96
    :param start_message:   The text to place above the separator, if any.
 
97
                            This will not be removed from the message
 
98
                            after the user has edited it.
 
99
 
 
100
    :return:    commit message or None.
 
101
    """
 
102
 
 
103
    if not start_message is None:
 
104
        start_message = start_message.encode(bzrlib.user_encoding)
 
105
    return edit_commit_message_encoded(infotext.encode(bzrlib.user_encoding),
 
106
                                       ignoreline, start_message)
 
107
 
 
108
 
 
109
def edit_commit_message_encoded(infotext, ignoreline=DEFAULT_IGNORE_LINE,
 
110
                                start_message=None):
 
111
    """Let the user edit a commit message in a temp file.
 
112
 
 
113
    This is run if they don't give a message or
 
114
    message-containing file on the command line.
 
115
 
 
116
    :param infotext:    Text to be displayed at bottom of message
 
117
                        for the user's reference;
 
118
                        currently similar to 'bzr status'.
 
119
                        The string is already encoded
 
120
 
 
121
    :param ignoreline:  The separator to use above the infotext.
 
122
 
 
123
    :param start_message:   The text to place above the separator, if any.
 
124
                            This will not be removed from the message
 
125
                            after the user has edited it.
 
126
                            The string is already encoded
 
127
 
 
128
    :return:    commit message or None.
 
129
    """
 
130
    msgfilename = None
81
131
    try:
82
 
        tmp_fileno, msgfilename = tempfile.mkstemp(prefix='bzr_log.', dir=u'.')
83
 
        msgfile = os.close(tmp_fileno)
84
 
        if infotext is not None and infotext != "":
85
 
            hasinfo = True
86
 
            msgfile = file(msgfilename, "w")
87
 
            msgfile.write("\n\n%s\n\n%s" % (ignoreline, infotext))
88
 
            msgfile.close()
89
 
        else:
90
 
            hasinfo = False
 
132
        msgfilename, hasinfo = _create_temp_file_with_commit_template(
 
133
                                    infotext, ignoreline, start_message)
91
134
 
92
 
        if not _run_editor(msgfilename):
 
135
        if not msgfilename or not _run_editor(msgfilename):
93
136
            return None
94
137
        
95
138
        started = False
96
139
        msg = []
97
140
        lastline, nlines = 0, 0
98
 
        for line in file(msgfilename, "r"):
99
 
            stripped_line = line.strip()
100
 
            # strip empty line before the log message starts
101
 
            if not started:
102
 
                if stripped_line != "":
103
 
                    started = True
104
 
                else:
105
 
                    continue
106
 
            # check for the ignore line only if there
107
 
            # is additional information at the end
108
 
            if hasinfo and stripped_line == ignoreline:
109
 
                break
110
 
            nlines += 1
111
 
            # keep track of the last line that had some content
112
 
            if stripped_line != "":
113
 
                lastline = nlines
114
 
            msg.append(line)
115
 
            
 
141
        # codecs.open() ALWAYS opens file in binary mode but we need text mode
 
142
        # 'rU' mode useful when bzr.exe used on Cygwin (bialix 20070430)
 
143
        f = file(msgfilename, 'rU')
 
144
        try:
 
145
            try:
 
146
                for line in codecs.getreader(bzrlib.user_encoding)(f):
 
147
                    stripped_line = line.strip()
 
148
                    # strip empty line before the log message starts
 
149
                    if not started:
 
150
                        if stripped_line != "":
 
151
                            started = True
 
152
                        else:
 
153
                            continue
 
154
                    # check for the ignore line only if there
 
155
                    # is additional information at the end
 
156
                    if hasinfo and stripped_line == ignoreline:
 
157
                        break
 
158
                    nlines += 1
 
159
                    # keep track of the last line that had some content
 
160
                    if stripped_line != "":
 
161
                        lastline = nlines
 
162
                    msg.append(line)
 
163
            except UnicodeDecodeError:
 
164
                raise BadCommitMessageEncoding()
 
165
        finally:
 
166
            f.close()
 
167
 
116
168
        if len(msg) == 0:
117
169
            return ""
118
170
        # delete empty lines at the end
124
176
            return "".join(msg)
125
177
    finally:
126
178
        # delete the msg file in any case
127
 
        try: os.unlink(msgfilename)
128
 
        except IOError: pass
 
179
        if msgfilename is not None:
 
180
            try:
 
181
                os.unlink(msgfilename)
 
182
            except IOError, e:
 
183
                warning("failed to unlink %s: %s; ignored", msgfilename, e)
 
184
 
 
185
 
 
186
def _create_temp_file_with_commit_template(infotext,
 
187
                                           ignoreline=DEFAULT_IGNORE_LINE,
 
188
                                           start_message=None):
 
189
    """Create temp file and write commit template in it.
 
190
 
 
191
    :param infotext:    Text to be displayed at bottom of message
 
192
                        for the user's reference;
 
193
                        currently similar to 'bzr status'.
 
194
                        The text is already encoded.
 
195
 
 
196
    :param ignoreline:  The separator to use above the infotext.
 
197
 
 
198
    :param start_message:   The text to place above the separator, if any.
 
199
                            This will not be removed from the message
 
200
                            after the user has edited it.
 
201
                            The string is already encoded
 
202
 
 
203
    :return:    2-tuple (temp file name, hasinfo)
 
204
    """
 
205
    import tempfile
 
206
    tmp_fileno, msgfilename = tempfile.mkstemp(prefix='bzr_log.',
 
207
                                               dir=u'.',
 
208
                                               text=True)
 
209
    msgfile = os.fdopen(tmp_fileno, 'w')
 
210
    try:
 
211
        if start_message is not None:
 
212
            msgfile.write("%s\n" % start_message)
 
213
 
 
214
        if infotext is not None and infotext != "":
 
215
            hasinfo = True
 
216
            msgfile.write("\n\n%s\n\n%s" %(ignoreline, infotext))
 
217
        else:
 
218
            hasinfo = False
 
219
    finally:
 
220
        msgfile.close()
 
221
 
 
222
    return (msgfilename, hasinfo)
129
223
 
130
224
 
131
225
def make_commit_message_template(working_tree, specific_files):
133
227
 
134
228
    Returns a unicode string containing the template.
135
229
    """
136
 
    # TODO: Should probably be given the WorkingTree not the branch
137
 
    #
138
230
    # TODO: make provision for this to be overridden or modified by a hook
139
231
    #
140
232
    # TODO: Rather than running the status command, should prepare a draft of
141
233
    # the revision to be committed, then pause and ask the user to
142
234
    # confirm/write a message.
143
235
    from StringIO import StringIO       # must be unicode-safe
144
 
    from bzrlib.status import show_status
 
236
    from bzrlib.status import show_tree_status
145
237
    status_tmp = StringIO()
146
 
    show_status(working_tree.branch, specific_files=specific_files, to_file=status_tmp)
 
238
    show_tree_status(working_tree, specific_files=specific_files, 
 
239
                     to_file=status_tmp)
147
240
    return status_tmp.getvalue()
 
241
 
 
242
 
 
243
def make_commit_message_template_encoded(working_tree, specific_files,
 
244
                                         diff=None, output_encoding='utf-8'):
 
245
    """Prepare a template file for a commit into a branch.
 
246
 
 
247
    Returns an encoded string.
 
248
    """
 
249
    # TODO: make provision for this to be overridden or modified by a hook
 
250
    #
 
251
    # TODO: Rather than running the status command, should prepare a draft of
 
252
    # the revision to be committed, then pause and ask the user to
 
253
    # confirm/write a message.
 
254
    from StringIO import StringIO       # must be unicode-safe
 
255
    from bzrlib.diff import show_diff_trees
 
256
 
 
257
    template = make_commit_message_template(working_tree, specific_files)
 
258
    template = template.encode(output_encoding, "replace")
 
259
 
 
260
    if diff:
 
261
        stream = StringIO()
 
262
        show_diff_trees(working_tree.basis_tree(),
 
263
                        working_tree, stream, specific_files,
 
264
                        path_encoding=output_encoding)
 
265
        template = template + '\n' + stream.getvalue()
 
266
 
 
267
    return template