~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/msgeditor.py

  • Committer: Martin Pool
  • Date: 2005-09-16 09:56:24 UTC
  • Revision ID: mbp@sourcefrog.net-20050916095623-ca0dff452934f21f
- make progress bar more tolerant of out-of-range values

Show diffs side-by-side

added added

removed removed

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