← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/msgeditor.py

Modify test_tsort_partial to accept multiple valid orderings.

This test previously checked for an exact match on the result of
tsort.topo_sort, while only a partial ordering is garantueed.  The current
implementation of topo_sort indeed returns the graph in lexicographical order,
but this depends on the order in which dict.popitem() pops and that is a
Python implementation detail not to be relied on.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/msgeditor.py
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005, 2006 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
18
18
"""Commit message editor support."""
19
19
 
20
20
import codecs
 
21
import errno
21
22
import os
22
23
from subprocess import call
23
24
import sys
24
25
 
25
26
from bzrlib import (
26
 
    cmdline,
27
27
    config,
28
28
    osutils,
29
29
    trace,
30
 
    transport,
31
 
    ui,
32
30
    )
33
31
from bzrlib.errors import BzrError, BadCommitMessageEncoding
34
 
from bzrlib.hooks import Hooks
 
32
from bzrlib.hooks import HookPoint, Hooks
35
33
 
36
34
 
37
35
def _get_editor():
41
39
    except KeyError:
42
40
        pass
43
41
 
44
 
    e = config.GlobalStack().get('editor')
 
42
    e = config.GlobalConfig().get_editor()
45
43
    if e is not None:
46
44
        yield e, config.config_filename()
47
45
 
60
58
def _run_editor(filename):
61
59
    """Try to execute an editor to edit the commit message."""
62
60
    for candidate, candidate_source in _get_editor():
63
 
        edargs = cmdline.split(candidate)
 
61
        edargs = candidate.split(' ')
64
62
        try:
65
63
            ## mutter("trying editor: %r", (edargs +[filename]))
66
64
            x = call(edargs + [filename])
67
65
        except OSError, e:
68
 
            if candidate_source is not None:
69
 
                # We tried this editor because some user configuration (an
70
 
                # environment variable or config file) said to try it.  Let
71
 
                # the user know their configuration is broken.
72
 
                trace.warning(
73
 
                    'Could not start editor "%s" (specified by %s): %s\n'
74
 
                    % (candidate, candidate_source, str(e)))
75
 
            continue
 
66
            # We're searching for an editor, so catch safe errors and continue
 
67
            if e.errno in (errno.ENOENT, errno.EACCES):
 
68
                if candidate_source is not None:
 
69
                    # We tried this editor because some user configuration (an
 
70
                    # environment variable or config file) said to try it.  Let
 
71
                    # the user know their configuration is broken.
 
72
                    trace.warning(
 
73
                        'Could not start editor "%s" (specified by %s): %s\n'
 
74
                        % (candidate, candidate_source, str(e)))
 
75
                continue
76
76
            raise
77
77
        if x == 0:
78
78
            return True
141
141
    try:
142
142
        msgfilename, hasinfo = _create_temp_file_with_commit_template(
143
143
                                    infotext, ignoreline, start_message)
144
 
        if not msgfilename:
145
 
            return None
146
 
        basename = osutils.basename(msgfilename)
147
 
        msg_transport = transport.get_transport_from_path(osutils.dirname(msgfilename))
148
 
        reference_content = msg_transport.get_bytes(basename)
149
 
        if not _run_editor(msgfilename):
150
 
            return None
151
 
        edited_content = msg_transport.get_bytes(basename)
152
 
        if edited_content == reference_content:
153
 
            if not ui.ui_factory.confirm_action(
154
 
                u"Commit message was not edited, use anyway",
155
 
                "bzrlib.msgeditor.unchanged",
156
 
                {}):
157
 
                # Returning "" makes cmd_commit raise 'empty commit message
158
 
                # specified' which is a reasonable error, given the user has
159
 
                # rejected using the unedited template.
160
 
                return ""
 
144
 
 
145
        if not msgfilename or not _run_editor(msgfilename):
 
146
            return None
 
147
 
161
148
        started = False
162
149
        msg = []
163
150
        lastline, nlines = 0, 0
209
196
 
210
197
def _create_temp_file_with_commit_template(infotext,
211
198
                                           ignoreline=DEFAULT_IGNORE_LINE,
212
 
                                           start_message=None,
213
 
                                           tmpdir=None):
 
199
                                           start_message=None):
214
200
    """Create temp file and write commit template in it.
215
201
 
216
 
    :param infotext: Text to be displayed at bottom of message for the
217
 
        user's reference; currently similar to 'bzr status'.  The text is
218
 
        already encoded.
 
202
    :param infotext:    Text to be displayed at bottom of message
 
203
                        for the user's reference;
 
204
                        currently similar to 'bzr status'.
 
205
                        The text is already encoded.
219
206
 
220
207
    :param ignoreline:  The separator to use above the infotext.
221
208
 
222
 
    :param start_message: The text to place above the separator, if any.
223
 
        This will not be removed from the message after the user has edited
224
 
        it.  The string is already encoded
 
209
    :param start_message:   The text to place above the separator, if any.
 
210
                            This will not be removed from the message
 
211
                            after the user has edited it.
 
212
                            The string is already encoded
225
213
 
226
214
    :return:    2-tuple (temp file name, hasinfo)
227
215
    """
228
216
    import tempfile
229
217
    tmp_fileno, msgfilename = tempfile.mkstemp(prefix='bzr_log.',
230
 
                                               dir=tmpdir, text=True)
 
218
                                               dir='.',
 
219
                                               text=True)
 
220
    msgfilename = osutils.basename(msgfilename)
231
221
    msgfile = os.fdopen(tmp_fileno, 'w')
232
222
    try:
233
223
        if start_message is not None:
302
292
 
303
293
        These are all empty initially.
304
294
        """
305
 
        Hooks.__init__(self, "bzrlib.msgeditor", "hooks")
306
 
        self.add_hook('set_commit_message',
307
 
            "Set a fixed commit message. "
308
 
            "set_commit_message is called with the "
309
 
            "bzrlib.commit.Commit object (so you can also change e.g. revision "
310
 
            "properties by editing commit.builder._revprops) and the message "
311
 
            "so far. set_commit_message must return the message to use or None"
312
 
            " if it should use the message editor as normal.", (2, 4))
313
 
        self.add_hook('commit_message_template',
 
295
        Hooks.__init__(self)
 
296
        self.create_hook(HookPoint('commit_message_template',
314
297
            "Called when a commit message is being generated. "
315
298
            "commit_message_template is called with the bzrlib.commit.Commit "
316
299
            "object and the message that is known so far. "
317
300
            "commit_message_template must return a new message to use (which "
318
 
            "could be the same as it was given). When there are multiple "
 
301
            "could be the same as it was given. When there are multiple "
319
302
            "hooks registered for commit_message_template, they are chained "
320
303
            "with the result from the first passed into the second, and so "
321
 
            "on.", (1, 10))
 
304
            "on.", (1, 10), None))
322
305
 
323
306
 
324
307
hooks = MessageEditorHooks()
325
308
 
326
309
 
327
 
def set_commit_message(commit, start_message=None):
328
 
    """Sets the commit message.
329
 
    :param commit: Commit object for the active commit.
330
 
    :return: The commit message or None to continue using the message editor
331
 
    """
332
 
    start_message = None
333
 
    for hook in hooks['set_commit_message']:
334
 
        start_message = hook(commit, start_message)
335
 
    return start_message
336
 
 
337
 
 
338
310
def generate_commit_message_template(commit, start_message=None):
339
311
    """Generate a commit message template.
340
312