← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/msgeditor.py

  • Committer: John Arbash Meinel
  • Date: 2008-08-18 22:34:21 UTC
  • mto: (3606.5.6 1.6)
  • mto: This revision was merged to the branch mainline in revision 3641.
  • Revision ID: john@arbash-meinel.com-20080818223421-todjny24vj4faj4t
Add tests for the fetching behavior.

The proper parameter passed is 'unordered' add an assert for it, and
fix callers that were passing 'unsorted' instead.
Add tests that we make the right get_record_stream call based
on the value of _fetch_uses_deltas.
Fix the fetch request for signatures.

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