← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/msgeditor.py

merge merge tweaks from aaron, which includes latest .dev

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/msgeditor.py
1
 
# Copyright (C) 2005, 2006 by Canonical Ltd
2
 
#
 
1
# Bazaar-NG -- distributed version control
 
2
 
 
3
# Copyright (C) 2005 by Canonical Ltd
 
4
 
3
5
# This program is free software; you can redistribute it and/or modify
4
6
# it under the terms of the GNU General Public License as published by
5
7
# the Free Software Foundation; either version 2 of the License, or
6
8
# (at your option) any later version.
7
 
#
 
9
 
8
10
# This program is distributed in the hope that it will be useful,
9
11
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
12
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
13
# GNU General Public License for more details.
12
 
#
 
14
 
13
15
# You should have received a copy of the GNU General Public License
14
16
# along with this program; if not, write to the Free Software
15
17
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
17
19
 
18
20
"""Commit message editor support."""
19
21
 
20
 
import codecs
21
 
import errno
22
22
import os
23
23
from subprocess import call
24
 
import sys
25
 
 
26
 
import bzrlib
27
 
import bzrlib.config as config
28
24
from bzrlib.errors import BzrError
29
 
from bzrlib.trace import warning, mutter
30
 
 
31
25
 
32
26
def _get_editor():
33
27
    """Return a sequence of possible editor binaries for the current platform"""
34
 
    try:
35
 
        yield os.environ["BZR_EDITOR"]
36
 
    except KeyError:
37
 
        pass
38
 
 
39
 
    e = config.GlobalConfig().get_editor()
 
28
    from bzrlib.osutils import _read_config_value
 
29
    
 
30
    e = _read_config_value("editor")
40
31
    if e is not None:
41
32
        yield e
42
33
        
43
 
    for varname in 'VISUAL', 'EDITOR':
44
 
        if os.environ.has_key(varname):
45
 
            yield os.environ[varname]
46
 
 
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
 
34
    try:
 
35
        yield os.environ["EDITOR"]
 
36
    except KeyError:
 
37
        if os.name == "nt":
 
38
            yield "notepad.exe"
 
39
        elif os.name == "posix":
 
40
            yield "/usr/bin/vi"
53
41
 
54
42
 
55
43
def _run_editor(filename):
56
44
    """Try to execute an editor to edit the commit message."""
57
45
    for e in _get_editor():
58
46
        edargs = e.split(' ')
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
 
47
        x = call(edargs + [filename])
67
48
        if x == 0:
68
49
            return True
69
50
        elif x == 127:
70
51
            continue
71
52
        else:
72
53
            break
73
 
    raise BzrError("Could not start any editor.\nPlease specify one with:\n"
74
 
                   " - $BZR_EDITOR\n - editor=/some/path in %s\n - $EDITOR" % \
75
 
                    config.config_filename())
76
 
 
77
 
 
78
 
DEFAULT_IGNORE_LINE = "%(bar)s %(msg)s %(bar)s" % \
79
 
    { 'bar' : '-' * 14, 'msg' : 'This line and the following will be ignored' }
80
 
 
81
 
 
82
 
def edit_commit_message(infotext, ignoreline=DEFAULT_IGNORE_LINE):
 
54
    raise BzrError("Could not start any editor. "
 
55
                   "Please specify $EDITOR or use ~/.bzr.conf/editor")
 
56
                          
 
57
 
 
58
def edit_commit_message(infotext, ignoreline=None):
83
59
    """Let the user edit a commit message in a temp file.
84
60
 
85
61
    This is run if they don't give a message or
91
67
        'bzr status'.
92
68
    """
93
69
    import tempfile
94
 
 
95
 
    msgfilename = None
 
70
    
 
71
    if ignoreline is None:
 
72
        ignoreline = "-- This line and the following will be ignored --"
 
73
        
96
74
    try:
97
 
        tmp_fileno, msgfilename = tempfile.mkstemp(prefix='bzr_log.', dir=u'.')
 
75
        tmp_fileno, msgfilename = tempfile.mkstemp()
98
76
        msgfile = os.close(tmp_fileno)
99
77
        if infotext is not None and infotext != "":
100
78
            hasinfo = True
101
79
            msgfile = file(msgfilename, "w")
102
 
            msgfile.write("\n\n%s\n\n%s" % (ignoreline,
103
 
                infotext.encode(bzrlib.user_encoding, 'replace')))
 
80
            msgfile.write("\n\n%s\n\n%s" % (ignoreline, infotext))
104
81
            msgfile.close()
105
82
        else:
106
83
            hasinfo = False
111
88
        started = False
112
89
        msg = []
113
90
        lastline, nlines = 0, 0
114
 
        for line in codecs.open(msgfilename, 'r', bzrlib.user_encoding):
 
91
        for line in file(msgfilename, "r"):
115
92
            stripped_line = line.strip()
116
93
            # strip empty line before the log message starts
117
94
            if not started:
130
107
            msg.append(line)
131
108
            
132
109
        if len(msg) == 0:
133
 
            return ""
 
110
            return None
134
111
        # delete empty lines at the end
135
112
        del msg[lastline:]
136
113
        # add a newline at the end, if needed
140
117
            return "".join(msg)
141
118
    finally:
142
119
        # delete the msg file in any case
143
 
        if msgfilename is not None:
144
 
            try:
145
 
                os.unlink(msgfilename)
146
 
            except IOError, e:
147
 
                warning("failed to unlink %s: %s; ignored", msgfilename, e)
148
 
 
149
 
 
150
 
def make_commit_message_template(working_tree, specific_files):
151
 
    """Prepare a template file for a commit into a branch.
152
 
 
153
 
    Returns a unicode string containing the template.
154
 
    """
155
 
    # TODO: Should probably be given the WorkingTree not the branch
156
 
    #
157
 
    # TODO: make provision for this to be overridden or modified by a hook
158
 
    #
159
 
    # TODO: Rather than running the status command, should prepare a draft of
160
 
    # the revision to be committed, then pause and ask the user to
161
 
    # confirm/write a message.
162
 
    from StringIO import StringIO       # must be unicode-safe
163
 
    from bzrlib.status import show_tree_status
164
 
    status_tmp = StringIO()
165
 
    show_tree_status(working_tree, specific_files=specific_files, 
166
 
                     to_file=status_tmp)
167
 
    return status_tmp.getvalue()
 
120
        try: os.unlink(msgfilename)
 
121
        except IOError: pass
 
122