← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/diff.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2006-06-17 02:18:48 UTC
  • mfrom: (1711.2.58 jam-integration)
  • Revision ID: pqm@pqm.ubuntu.com-20060617021848-38906188fc2a0300
(jamesh, jam) fix external_diff to support any file-like object (#4047, #48914)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/diff.py
1
1
# Copyright (C) 2004, 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
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
16
16
 
 
17
import errno
 
18
import os
 
19
import subprocess
 
20
import sys
 
21
import tempfile
17
22
import time
18
23
 
19
24
from bzrlib.delta import compare_trees
20
25
from bzrlib.errors import BzrError
21
26
import bzrlib.errors as errors
 
27
import bzrlib.osutils
22
28
from bzrlib.patiencediff import unified_diff
23
29
import bzrlib.patiencediff
24
30
from bzrlib.symbol_versioning import *
25
31
from bzrlib.textfile import check_text_lines
26
 
from bzrlib.trace import mutter
 
32
from bzrlib.trace import mutter, warning
27
33
 
28
34
 
29
35
# TODO: Rather than building a changeset object, we should probably
81
87
def external_diff(old_filename, oldlines, new_filename, newlines, to_file,
82
88
                  diff_opts):
83
89
    """Display a diff by calling out to the external diff program."""
84
 
    import sys
 
90
    if hasattr(to_file, 'fileno'):
 
91
        out_file = to_file
 
92
        have_fileno = True
 
93
    else:
 
94
        out_file = subprocess.PIPE
 
95
        have_fileno = False
85
96
    
86
 
    if to_file != sys.stdout:
87
 
        raise NotImplementedError("sorry, can't send external diff other than to stdout yet",
88
 
                                  to_file)
89
 
 
90
97
    # make sure our own output is properly ordered before the diff
91
98
    to_file.flush()
92
99
 
93
 
    from tempfile import NamedTemporaryFile
94
 
    import os
95
 
 
96
 
    oldtmpf = NamedTemporaryFile()
97
 
    newtmpf = NamedTemporaryFile()
 
100
    oldtmp_fd, old_abspath = tempfile.mkstemp(prefix='bzr-diff-old-')
 
101
    newtmp_fd, new_abspath = tempfile.mkstemp(prefix='bzr-diff-new-')
 
102
    oldtmpf = os.fdopen(oldtmp_fd, 'wb')
 
103
    newtmpf = os.fdopen(newtmp_fd, 'wb')
98
104
 
99
105
    try:
100
106
        # TODO: perhaps a special case for comparing to or from the empty
107
113
        oldtmpf.writelines(oldlines)
108
114
        newtmpf.writelines(newlines)
109
115
 
110
 
        oldtmpf.flush()
111
 
        newtmpf.flush()
 
116
        oldtmpf.close()
 
117
        newtmpf.close()
112
118
 
113
119
        if not diff_opts:
114
120
            diff_opts = []
115
121
        diffcmd = ['diff',
116
122
                   '--label', old_filename,
117
 
                   oldtmpf.name,
 
123
                   old_abspath,
118
124
                   '--label', new_filename,
119
 
                   newtmpf.name]
 
125
                   new_abspath,
 
126
                   '--binary',
 
127
                  ]
120
128
 
121
129
        # diff only allows one style to be specified; they don't override.
122
130
        # note that some of these take optargs, and the optargs can be
142
150
        if diff_opts:
143
151
            diffcmd.extend(diff_opts)
144
152
 
145
 
        rc = os.spawnvp(os.P_WAIT, 'diff', diffcmd)
 
153
        try:
 
154
            pipe = subprocess.Popen(diffcmd,
 
155
                                    stdin=subprocess.PIPE,
 
156
                                    stdout=out_file)
 
157
        except OSError, e:
 
158
            if e.errno == errno.ENOENT:
 
159
                raise errors.NoDiff(str(e))
 
160
            raise
 
161
        pipe.stdin.close()
 
162
 
 
163
        if not have_fileno:
 
164
            bzrlib.osutils.pumpfile(pipe.stdout, to_file)
 
165
        rc = pipe.wait()
146
166
        
147
167
        if rc != 0 and rc != 1:
148
168
            # returns 1 if files differ; that's OK
155
175
    finally:
156
176
        oldtmpf.close()                 # and delete
157
177
        newtmpf.close()
 
178
        # Clean up. Warn in case the files couldn't be deleted
 
179
        # (in case windows still holds the file open, but not
 
180
        # if the files have already been deleted)
 
181
        try:
 
182
            os.remove(old_abspath)
 
183
        except OSError, e:
 
184
            if e.errno not in (errno.ENOENT,):
 
185
                warning('Failed to delete temporary file: %s %s',
 
186
                        old_abspath, e)
 
187
        try:
 
188
            os.remove(new_abspath)
 
189
        except OSError:
 
190
            if e.errno not in (errno.ENOENT,):
 
191
                warning('Failed to delete temporary file: %s %s',
 
192
                        new_abspath, e)
158
193
 
159
194
 
160
195
@deprecated_function(zero_eight)
174
209
    supplies any two trees.
175
210
    """
176
211
    if output is None:
177
 
        import sys
178
212
        output = sys.stdout
179
213
 
180
214
    if from_spec is None:
221
255
    The more general form is show_diff_trees(), where the caller
222
256
    supplies any two trees.
223
257
    """
224
 
    import sys
225
258
    output = sys.stdout
226
259
    def spec_tree(spec):
227
260
        revision_id = spec.in_store(tree.branch).rev_id