← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/msgeditor.py

  • Committer: Martin Pool
  • Date: 2005-09-05 09:11:03 UTC
  • Revision ID: mbp@sourcefrog.net-20050905091103-1e51e146be0f08b4
- add test for deserialization from a canned XML inventory

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/msgeditor.py
20
20
"""Commit message editor support."""
21
21
 
22
22
import os
23
 
from subprocess import call
24
 
 
25
 
import bzrlib.config as config
26
23
from bzrlib.errors import BzrError
27
24
 
28
25
def _get_editor():
29
26
    """Return a sequence of possible editor binaries for the current platform"""
30
 
    try:
31
 
        yield os.environ["BZR_EDITOR"]
32
 
    except KeyError:
33
 
        pass
34
 
 
35
 
    e = config.GlobalConfig().get_editor()
 
27
    from bzrlib.osutils import _read_config_value
 
28
    
 
29
    e = _read_config_value("editor")
36
30
    if e is not None:
37
31
        yield e
38
32
        
39
 
    try:
40
 
        yield os.environ["EDITOR"]
41
 
    except KeyError:
42
 
        pass
43
 
 
44
 
    if os.name == "nt":
 
33
    if os.name == "windows":
45
34
        yield "notepad.exe"
46
35
    elif os.name == "posix":
47
 
        yield "/usr/bin/vi"
 
36
        try:
 
37
            yield os.environ["EDITOR"]
 
38
        except KeyError:
 
39
            yield "/usr/bin/vi"
48
40
 
49
41
 
50
42
def _run_editor(filename):
51
43
    """Try to execute an editor to edit the commit message."""
52
44
    for e in _get_editor():
53
45
        edargs = e.split(' ')
54
 
        x = call(edargs + [filename])
 
46
        x = os.spawnvp(os.P_WAIT, edargs[0],
 
47
                       edargs + [filename])
55
48
        if x == 0:
56
49
            return True
57
50
        elif x == 127:
79
72
        ignoreline = "-- This line and the following will be ignored --"
80
73
        
81
74
    try:
82
 
        tmp_fileno, msgfilename = tempfile.mkstemp(prefix='bzr_log.', dir=u'.')
 
75
        tmp_fileno, msgfilename = tempfile.mkstemp()
83
76
        msgfile = os.close(tmp_fileno)
84
77
        if infotext is not None and infotext != "":
85
78
            hasinfo = True
114
107
            msg.append(line)
115
108
            
116
109
        if len(msg) == 0:
117
 
            return ""
 
110
            return None
118
111
        # delete empty lines at the end
119
112
        del msg[lastline:]
120
113
        # add a newline at the end, if needed
127
120
        try: os.unlink(msgfilename)
128
121
        except IOError: pass
129
122
 
130
 
 
131
 
def make_commit_message_template(working_tree, specific_files):
132
 
    """Prepare a template file for a commit into a branch.
133
 
 
134
 
    Returns a unicode string containing the template.
135
 
    """
136
 
    # TODO: Should probably be given the WorkingTree not the branch
137
 
    #
138
 
    # TODO: make provision for this to be overridden or modified by a hook
139
 
    #
140
 
    # TODO: Rather than running the status command, should prepare a draft of
141
 
    # the revision to be committed, then pause and ask the user to
142
 
    # confirm/write a message.
143
 
    from StringIO import StringIO       # must be unicode-safe
144
 
    from bzrlib.status import show_status
145
 
    status_tmp = StringIO()
146
 
    show_status(working_tree.branch, specific_files=specific_files, to_file=status_tmp)
147
 
    return status_tmp.getvalue()