bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
| 1658.1.9
by Martin Pool Give an error for bzr diff on an nonexistent file (Malone #3619) | 1 | # Copyright (C) 2004, 2005, 2006 Canonical Ltd.
 | 
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 2 | #
 | 
| 1
by mbp at sourcefrog import from baz patch-364 | 3 | # This program is free software; you can redistribute it and/or modify
 | 
| 4 | # it under the terms of the GNU General Public License as published by
 | |
| 5 | # the Free Software Foundation; either version 2 of the License, or
 | |
| 6 | # (at your option) any later version.
 | |
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 7 | #
 | 
| 1
by mbp at sourcefrog import from baz patch-364 | 8 | # This program is distributed in the hope that it will be useful,
 | 
| 9 | # but WITHOUT ANY WARRANTY; without even the implied warranty of
 | |
| 10 | # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 | |
| 11 | # GNU General Public License for more details.
 | |
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 12 | #
 | 
| 1
by mbp at sourcefrog import from baz patch-364 | 13 | # You should have received a copy of the GNU General Public License
 | 
| 14 | # along with this program; if not, write to the Free Software
 | |
| 15 | # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 | |
| 16 | ||
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 17 | import os | 
| 1899.1.5
by John Arbash Meinel Always buffer the output of diff, so we can check if retcode==2 is because of Binary files | 18 | import re | 
| 1996.3.9
by John Arbash Meinel lazy_import diff.py | 19 | import sys | 
| 20 | ||
| 21 | from bzrlib.lazy_import import lazy_import | |
| 22 | lazy_import(globals(), """ | |
| 23 | import errno
 | |
| 1692.8.7
by James Henstridge changes suggested by John Meinel | 24 | import subprocess
 | 
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 25 | import tempfile
 | 
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 26 | import time
 | 
| 27 | ||
| 1955.2.10
by John Arbash Meinel Unset a few other LANG type variables when spawning diff | 28 | from bzrlib import (
 | 
| 29 |     errors,
 | |
| 30 |     osutils,
 | |
| 1996.3.9
by John Arbash Meinel lazy_import diff.py | 31 |     patiencediff,
 | 
| 32 |     textfile,
 | |
| 1551.12.29
by Aaron Bentley Copy and extend patch date formatting code, add patch-date parsing | 33 |     timestamp,
 | 
| 1955.2.10
by John Arbash Meinel Unset a few other LANG type variables when spawning diff | 34 |     )
 | 
| 1996.3.9
by John Arbash Meinel lazy_import diff.py | 35 | """) | 
| 36 | ||
| 1852.10.3
by Robert Collins Remove all uses of compare_trees and replace with Tree.changes_from throughout bzrlib. | 37 | # compatability - plugins import compare_trees from diff!!!
 | 
| 38 | # deprecated as of 0.10
 | |
| 1534.4.35
by Robert Collins Give branch its own basis tree and last_revision methods; deprecated branch.working_tree() | 39 | from bzrlib.delta import compare_trees | 
| 1996.3.9
by John Arbash Meinel lazy_import diff.py | 40 | from bzrlib.symbol_versioning import ( | 
| 41 | deprecated_function, | |
| 42 | zero_eight, | |
| 43 |         )
 | |
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 44 | from bzrlib.trace import mutter, warning | 
| 1
by mbp at sourcefrog import from baz patch-364 | 45 | |
| 1711.2.24
by John Arbash Meinel Late bind to PatienceSequenceMatcher to allow plugin to override. | 46 | |
| 767
by Martin Pool - files are only reported as modified if their name or parent has changed, | 47 | # TODO: Rather than building a changeset object, we should probably
 | 
| 48 | # invoke callbacks on an object.  That object can either accumulate a
 | |
| 49 | # list, write them out directly, etc etc.
 | |
| 50 | ||
| 1558.15.11
by Aaron Bentley Apply merge review suggestions | 51 | def internal_diff(old_filename, oldlines, new_filename, newlines, to_file, | 
| 1711.2.30
by John Arbash Meinel Fix bug in internal_diff handling of unicode paths | 52 | allow_binary=False, sequence_matcher=None, | 
| 53 | path_encoding='utf8'): | |
| 475
by Martin Pool - rewrite diff using compare_trees() | 54 |     # FIXME: difflib is wrong if there is no trailing newline.
 | 
| 55 |     # The syntax used by patch seems to be "\ No newline at
 | |
| 56 |     # end of file" following the last diff line from that
 | |
| 57 |     # file.  This is not trivial to insert into the
 | |
| 58 |     # unified_diff output and it might be better to just fix
 | |
| 59 |     # or replace that function.
 | |
| 60 | ||
| 61 |     # In the meantime we at least make sure the patch isn't
 | |
| 62 |     # mangled.
 | |
| 63 | ||
| 64 | ||
| 65 |     # Special workaround for Python2.3, where difflib fails if
 | |
| 66 |     # both sequences are empty.
 | |
| 67 | if not oldlines and not newlines: | |
| 68 |         return
 | |
| 1558.15.2
by Aaron Bentley Implemented binary file handling for diff | 69 | |
| 1558.15.11
by Aaron Bentley Apply merge review suggestions | 70 | if allow_binary is False: | 
| 1996.3.9
by John Arbash Meinel lazy_import diff.py | 71 | textfile.check_text_lines(oldlines) | 
| 72 | textfile.check_text_lines(newlines) | |
| 475
by Martin Pool - rewrite diff using compare_trees() | 73 | |
| 1185.81.8
by John Arbash Meinel Updating unified_diff to take a factory, using the new diff algorithm in the code. | 74 | if sequence_matcher is None: | 
| 1996.3.9
by John Arbash Meinel lazy_import diff.py | 75 | sequence_matcher = patiencediff.PatienceSequenceMatcher | 
| 76 | ud = patiencediff.unified_diff(oldlines, newlines, | |
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 77 | fromfile=old_filename.encode(path_encoding), | 
| 78 | tofile=new_filename.encode(path_encoding), | |
| 1185.81.8
by John Arbash Meinel Updating unified_diff to take a factory, using the new diff algorithm in the code. | 79 | sequencematcher=sequence_matcher) | 
| 475
by Martin Pool - rewrite diff using compare_trees() | 80 | |
| 1092.1.50
by Robert Collins make diff lsdiff/filterdiff friendly | 81 | ud = list(ud) | 
| 475
by Martin Pool - rewrite diff using compare_trees() | 82 |     # work-around for difflib being too smart for its own good
 | 
| 83 |     # if /dev/null is "1,0", patch won't recognize it as /dev/null
 | |
| 84 | if not oldlines: | |
| 85 | ud[2] = ud[2].replace('-1,0', '-0,0') | |
| 86 | elif not newlines: | |
| 87 | ud[2] = ud[2].replace('+1,0', '+0,0') | |
| 1092.1.50
by Robert Collins make diff lsdiff/filterdiff friendly | 88 |     # work around for difflib emitting random spaces after the label
 | 
| 89 | ud[0] = ud[0][:-2] + '\n' | |
| 90 | ud[1] = ud[1][:-2] + '\n' | |
| 475
by Martin Pool - rewrite diff using compare_trees() | 91 | |
| 804
by Martin Pool Patch from John: | 92 | for line in ud: | 
| 93 | to_file.write(line) | |
| 974.1.5
by Aaron Bentley Fixed handling of missing newlines in udiffs | 94 | if not line.endswith('\n'): | 
| 95 | to_file.write("\n\\ No newline at end of file\n") | |
| 475
by Martin Pool - rewrite diff using compare_trees() | 96 | print >>to_file | 
| 97 | ||
| 98 | ||
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 99 | def _spawn_external_diff(diffcmd, capture_errors=True): | 
| 100 | """Spawn the externall diff process, and return the child handle. | |
| 101 | ||
| 102 |     :param diffcmd: The command list to spawn
 | |
| 2138.1.1
by Wouter van Heyst Robuster external diff output handling. | 103 |     :param capture_errors: Capture stderr as well as setting LANG=C
 | 
| 104 |         and LC_ALL=C. This lets us read and understand the output of diff,
 | |
| 105 |         and respond to any errors.
 | |
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 106 |     :return: A Popen object.
 | 
| 107 |     """
 | |
| 108 | if capture_errors: | |
| 2321.2.2
by Alexander Belchenko win32 fixes for test_external_diff_binary (gettext on win32 rely on $LANGUAGE) | 109 |         # construct minimal environment
 | 
| 110 | env = {} | |
| 111 | path = os.environ.get('PATH') | |
| 112 | if path is not None: | |
| 113 | env['PATH'] = path | |
| 2321.2.5
by Alexander Belchenko external diff: no need for special code path for win32 (suggested by John Meinel) | 114 | env['LANGUAGE'] = 'C' # on win32 only LANGUAGE has effect | 
| 115 | env['LANG'] = 'C' | |
| 116 | env['LC_ALL'] = 'C' | |
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 117 | stderr = subprocess.PIPE | 
| 118 | else: | |
| 2321.2.2
by Alexander Belchenko win32 fixes for test_external_diff_binary (gettext on win32 rely on $LANGUAGE) | 119 | env = None | 
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 120 | stderr = None | 
| 121 | ||
| 122 | try: | |
| 123 | pipe = subprocess.Popen(diffcmd, | |
| 124 | stdin=subprocess.PIPE, | |
| 125 | stdout=subprocess.PIPE, | |
| 126 | stderr=stderr, | |
| 2321.2.2
by Alexander Belchenko win32 fixes for test_external_diff_binary (gettext on win32 rely on $LANGUAGE) | 127 | env=env) | 
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 128 | except OSError, e: | 
| 129 | if e.errno == errno.ENOENT: | |
| 130 | raise errors.NoDiff(str(e)) | |
| 131 |         raise
 | |
| 132 | ||
| 133 | return pipe | |
| 134 | ||
| 135 | ||
| 1185.35.29
by Aaron Bentley Support whitespace in diff filenames | 136 | def external_diff(old_filename, oldlines, new_filename, newlines, to_file, | 
| 571
by Martin Pool - new --diff-options to pass options through to external | 137 | diff_opts): | 
| 568
by Martin Pool - start adding support for showing diffs by calling out to | 138 | """Display a diff by calling out to the external diff program.""" | 
| 581
by Martin Pool - make sure any bzr output is flushed before | 139 |     # make sure our own output is properly ordered before the diff
 | 
| 140 | to_file.flush() | |
| 141 | ||
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 142 | oldtmp_fd, old_abspath = tempfile.mkstemp(prefix='bzr-diff-old-') | 
| 143 | newtmp_fd, new_abspath = tempfile.mkstemp(prefix='bzr-diff-new-') | |
| 144 | oldtmpf = os.fdopen(oldtmp_fd, 'wb') | |
| 145 | newtmpf = os.fdopen(newtmp_fd, 'wb') | |
| 568
by Martin Pool - start adding support for showing diffs by calling out to | 146 | |
| 147 | try: | |
| 148 |         # TODO: perhaps a special case for comparing to or from the empty
 | |
| 149 |         # sequence; can just use /dev/null on Unix
 | |
| 150 | ||
| 151 |         # TODO: if either of the files being compared already exists as a
 | |
| 152 |         # regular named file (e.g. in the working directory) then we can
 | |
| 153 |         # compare directly to that, rather than copying it.
 | |
| 154 | ||
| 155 | oldtmpf.writelines(oldlines) | |
| 156 | newtmpf.writelines(newlines) | |
| 157 | ||
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 158 | oldtmpf.close() | 
| 159 | newtmpf.close() | |
| 568
by Martin Pool - start adding support for showing diffs by calling out to | 160 | |
| 571
by Martin Pool - new --diff-options to pass options through to external | 161 | if not diff_opts: | 
| 162 | diff_opts = [] | |
| 163 | diffcmd = ['diff', | |
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 164 | '--label', old_filename, | 
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 165 | old_abspath, | 
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 166 | '--label', new_filename, | 
| 1711.2.56
by John Arbash Meinel Raise NoDiff if 'diff' not present. | 167 | new_abspath, | 
| 168 | '--binary', | |
| 169 |                   ]
 | |
| 571
by Martin Pool - new --diff-options to pass options through to external | 170 | |
| 171 |         # diff only allows one style to be specified; they don't override.
 | |
| 172 |         # note that some of these take optargs, and the optargs can be
 | |
| 173 |         # directly appended to the options.
 | |
| 174 |         # this is only an approximate parser; it doesn't properly understand
 | |
| 175 |         # the grammar.
 | |
| 176 | for s in ['-c', '-u', '-C', '-U', | |
| 177 | '-e', '--ed', | |
| 178 | '-q', '--brief', | |
| 179 | '--normal', | |
| 180 | '-n', '--rcs', | |
| 181 | '-y', '--side-by-side', | |
| 182 | '-D', '--ifdef']: | |
| 183 | for j in diff_opts: | |
| 184 | if j.startswith(s): | |
| 185 |                     break
 | |
| 186 | else: | |
| 187 |                 continue
 | |
| 188 |             break
 | |
| 189 | else: | |
| 190 | diffcmd.append('-u') | |
| 191 | ||
| 192 | if diff_opts: | |
| 193 | diffcmd.extend(diff_opts) | |
| 194 | ||
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 195 | pipe = _spawn_external_diff(diffcmd, capture_errors=True) | 
| 196 | out,err = pipe.communicate() | |
| 197 | rc = pipe.returncode | |
| 571
by Martin Pool - new --diff-options to pass options through to external | 198 | |
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 199 |         # internal_diff() adds a trailing newline, add one here for consistency
 | 
| 200 | out += '\n' | |
| 1899.1.5
by John Arbash Meinel Always buffer the output of diff, so we can check if retcode==2 is because of Binary files | 201 | if rc == 2: | 
| 202 |             # 'diff' gives retcode == 2 for all sorts of errors
 | |
| 203 |             # one of those is 'Binary files differ'.
 | |
| 204 |             # Bad options could also be the problem.
 | |
| 1904.1.4
by Marien Zwart Make external diff in binary mode work with recent versions of diffutils. | 205 |             # 'Binary files' is not a real error, so we suppress that error.
 | 
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 206 | lang_c_out = out | 
| 207 | ||
| 208 |             # Since we got here, we want to make sure to give an i18n error
 | |
| 209 | pipe = _spawn_external_diff(diffcmd, capture_errors=False) | |
| 210 | out, err = pipe.communicate() | |
| 211 | ||
| 212 |             # Write out the new i18n diff response
 | |
| 213 | to_file.write(out+'\n') | |
| 214 | if pipe.returncode != 2: | |
| 1996.3.9
by John Arbash Meinel lazy_import diff.py | 215 | raise errors.BzrError( | 
| 216 |                                'external diff failed with exit code 2'
 | |
| 2138.1.1
by Wouter van Heyst Robuster external diff output handling. | 217 |                                ' when run with LANG=C and LC_ALL=C,'
 | 
| 218 | ' but not when run natively: %r' % (diffcmd,)) | |
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 219 | |
| 220 | first_line = lang_c_out.split('\n', 1)[0] | |
| 1904.1.4
by Marien Zwart Make external diff in binary mode work with recent versions of diffutils. | 221 |             # Starting with diffutils 2.8.4 the word "binary" was dropped.
 | 
| 222 | m = re.match('^(binary )?files.*differ$', first_line, re.I) | |
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 223 | if m is None: | 
| 1996.3.9
by John Arbash Meinel lazy_import diff.py | 224 | raise errors.BzrError('external diff failed with exit code 2;' | 
| 225 | ' command: %r' % (diffcmd,)) | |
| 1920.1.1
by John Arbash Meinel fix bug #56307, handle binary files even when LANG is not english | 226 | else: | 
| 227 |                 # Binary files differ, just return
 | |
| 228 |                 return
 | |
| 229 | ||
| 230 |         # If we got to here, we haven't written out the output of diff
 | |
| 231 |         # do so now
 | |
| 232 | to_file.write(out) | |
| 233 | if rc not in (0, 1): | |
| 571
by Martin Pool - new --diff-options to pass options through to external | 234 |             # returns 1 if files differ; that's OK
 | 
| 235 | if rc < 0: | |
| 236 | msg = 'signal %d' % (-rc) | |
| 237 | else: | |
| 238 | msg = 'exit code %d' % rc | |
| 239 | ||
| 1996.3.9
by John Arbash Meinel lazy_import diff.py | 240 | raise errors.BzrError('external diff failed with %s; command: %r' | 
| 241 | % (rc, diffcmd)) | |
| 1899.1.6
by John Arbash Meinel internal_diff always adds a trailing \n, make sure external_diff does too | 242 | |
| 243 | ||
| 568
by Martin Pool - start adding support for showing diffs by calling out to | 244 | finally: | 
| 245 | oldtmpf.close() # and delete | |
| 246 | newtmpf.close() | |
| 1711.2.54
by John Arbash Meinel Use mkstemp instead of NamedTemporary file for external diff. | 247 |         # Clean up. Warn in case the files couldn't be deleted
 | 
| 248 |         # (in case windows still holds the file open, but not
 | |
| 249 |         # if the files have already been deleted)
 | |
| 250 | try: | |
| 251 | os.remove(old_abspath) | |
| 252 | except OSError, e: | |
| 253 | if e.errno not in (errno.ENOENT,): | |
| 254 | warning('Failed to delete temporary file: %s %s', | |
| 255 | old_abspath, e) | |
| 256 | try: | |
| 257 | os.remove(new_abspath) | |
| 258 | except OSError: | |
| 259 | if e.errno not in (errno.ENOENT,): | |
| 260 | warning('Failed to delete temporary file: %s %s', | |
| 261 | new_abspath, e) | |
| 568
by Martin Pool - start adding support for showing diffs by calling out to | 262 | |
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 263 | |
| 264 | @deprecated_function(zero_eight) | |
| 1432
by Robert Collins branch: namespace | 265 | def show_diff(b, from_spec, specific_files, external_diff_options=None, | 
| 1185.35.28
by Aaron Bentley Support diff with two branches as input. | 266 | revision2=None, output=None, b2=None): | 
| 619
by Martin Pool doc | 267 | """Shortcut for showing the diff to the working tree. | 
| 268 | ||
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 269 |     Please use show_diff_trees instead.
 | 
| 270 | ||
| 619
by Martin Pool doc | 271 |     b
 | 
| 272 |         Branch.
 | |
| 273 | ||
| 274 |     revision
 | |
| 1432
by Robert Collins branch: namespace | 275 |         None for 'basis tree', or otherwise the old revision to compare against.
 | 
| 619
by Martin Pool doc | 276 |     
 | 
| 277 |     The more general form is show_diff_trees(), where the caller
 | |
| 278 |     supplies any two trees.
 | |
| 279 |     """
 | |
| 1092.1.47
by Robert Collins make show_diff redirectable | 280 | if output is None: | 
| 281 | output = sys.stdout | |
| 475
by Martin Pool - rewrite diff using compare_trees() | 282 | |
| 1432
by Robert Collins branch: namespace | 283 | if from_spec is None: | 
| 1508.1.19
by Robert Collins Give format3 working trees their own last-revision marker. | 284 | old_tree = b.bzrdir.open_workingtree() | 
| 1185.35.28
by Aaron Bentley Support diff with two branches as input. | 285 | if b2 is None: | 
| 1534.4.35
by Robert Collins Give branch its own basis tree and last_revision methods; deprecated branch.working_tree() | 286 | old_tree = old_tree = old_tree.basis_tree() | 
| 329
by Martin Pool - refactor command functions into command classes | 287 | else: | 
| 1185.67.2
by Aaron Bentley Renamed Branch.storage to Branch.repository | 288 | old_tree = b.repository.revision_tree(from_spec.in_history(b).rev_id) | 
| 974.1.26
by aaron.bentley at utoronto merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 | 289 | |
| 1185.11.5
by John Arbash Meinel Merged up-to-date against mainline, still broken. | 290 | if revision2 is None: | 
| 1185.35.28
by Aaron Bentley Support diff with two branches as input. | 291 | if b2 is None: | 
| 1508.1.19
by Robert Collins Give format3 working trees their own last-revision marker. | 292 | new_tree = b.bzrdir.open_workingtree() | 
| 1185.35.28
by Aaron Bentley Support diff with two branches as input. | 293 | else: | 
| 1508.1.19
by Robert Collins Give format3 working trees their own last-revision marker. | 294 | new_tree = b2.bzrdir.open_workingtree() | 
| 974.1.26
by aaron.bentley at utoronto merged mbp@sourcefrog.net-20050817233101-0939da1cf91f2472 | 295 | else: | 
| 1185.67.2
by Aaron Bentley Renamed Branch.storage to Branch.repository | 296 | new_tree = b.repository.revision_tree(revision2.in_history(b).rev_id) | 
| 329
by Martin Pool - refactor command functions into command classes | 297 | |
| 1490
by Robert Collins Implement a 'bzr push' command, with saved locations; update diff to return 1. | 298 | return show_diff_trees(old_tree, new_tree, output, specific_files, | 
| 299 | external_diff_options) | |
| 571
by Martin Pool - new --diff-options to pass options through to external | 300 | |
| 301 | ||
| 1551.2.15
by Aaron Bentley Rename cmd_show_diff to diff_cmd_helper | 302 | def diff_cmd_helper(tree, specific_files, external_diff_options, | 
| 1684.1.6
by Martin Pool (patch) --diff-prefix option (goffredo, alexander) | 303 | old_revision_spec=None, new_revision_spec=None, | 
| 2197.2.1
by Martin Pool Refactor cmd_diff | 304 | revision_specs=None, | 
| 1684.1.6
by Martin Pool (patch) --diff-prefix option (goffredo, alexander) | 305 | old_label='a/', new_label='b/'): | 
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 306 | """Helper for cmd_diff. | 
| 307 | ||
| 2197.2.1
by Martin Pool Refactor cmd_diff | 308 |     :param tree:
 | 
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 309 |         A WorkingTree
 | 
| 310 | ||
| 2197.2.1
by Martin Pool Refactor cmd_diff | 311 |     :param specific_files:
 | 
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 312 |         The specific files to compare, or None
 | 
| 313 | ||
| 2197.2.1
by Martin Pool Refactor cmd_diff | 314 |     :param external_diff_options:
 | 
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 315 |         If non-None, run an external diff, and pass it these options
 | 
| 316 | ||
| 2197.2.1
by Martin Pool Refactor cmd_diff | 317 |     :param old_revision_spec:
 | 
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 318 |         If None, use basis tree as old revision, otherwise use the tree for
 | 
| 319 |         the specified revision. 
 | |
| 320 | ||
| 2197.2.1
by Martin Pool Refactor cmd_diff | 321 |     :param new_revision_spec:
 | 
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 322 |         If None, use working tree as new revision, otherwise use the tree for
 | 
| 323 |         the specified revision.
 | |
| 324 |     
 | |
| 2197.2.1
by Martin Pool Refactor cmd_diff | 325 |     :param revision_specs: 
 | 
| 326 |         Zero, one or two RevisionSpecs from the command line, saying what revisions 
 | |
| 327 |         to compare.  This can be passed as an alternative to the old_revision_spec 
 | |
| 328 |         and new_revision_spec parameters.
 | |
| 329 | ||
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 330 |     The more general form is show_diff_trees(), where the caller
 | 
| 331 |     supplies any two trees.
 | |
| 332 |     """
 | |
| 2197.2.2
by Martin Pool merge bzr.dev, reconcile with option changes | 333 | |
| 334 |     # TODO: perhaps remove the old parameters old_revision_spec and
 | |
| 335 |     # new_revision_spec, since this is only really for use from cmd_diff and
 | |
| 336 |     # it now always passes through a sequence of revision_specs -- mbp
 | |
| 337 |     # 20061221
 | |
| 338 | ||
| 1551.2.14
by Aaron Bentley Updated argument names, DRY fixes. | 339 | def spec_tree(spec): | 
| 1732.3.1
by Matthieu Moy Implementation of -r revno:N:/path/to/branch | 340 | if tree: | 
| 341 | revision = spec.in_store(tree.branch) | |
| 342 | else: | |
| 343 | revision = spec.in_store(None) | |
| 344 | revision_id = revision.rev_id | |
| 345 | branch = revision.branch | |
| 346 | return branch.repository.revision_tree(revision_id) | |
| 2197.2.1
by Martin Pool Refactor cmd_diff | 347 | |
| 348 | if revision_specs is not None: | |
| 349 | assert (old_revision_spec is None | |
| 350 | and new_revision_spec is None) | |
| 351 | if len(revision_specs) > 0: | |
| 352 | old_revision_spec = revision_specs[0] | |
| 353 | if len(revision_specs) > 1: | |
| 354 | new_revision_spec = revision_specs[1] | |
| 355 | ||
| 1551.2.14
by Aaron Bentley Updated argument names, DRY fixes. | 356 | if old_revision_spec is None: | 
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 357 | old_tree = tree.basis_tree() | 
| 358 | else: | |
| 1551.2.14
by Aaron Bentley Updated argument names, DRY fixes. | 359 | old_tree = spec_tree(old_revision_spec) | 
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 360 | |
| 2197.2.1
by Martin Pool Refactor cmd_diff | 361 | if (new_revision_spec is None | 
| 362 | or new_revision_spec.spec is None): | |
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 363 | new_tree = tree | 
| 364 | else: | |
| 1551.2.14
by Aaron Bentley Updated argument names, DRY fixes. | 365 | new_tree = spec_tree(new_revision_spec) | 
| 2197.2.1
by Martin Pool Refactor cmd_diff | 366 | |
| 1551.7.19
by Aaron Bentley Always include working tree when calculating file ids for diff | 367 | if new_tree is not tree: | 
| 368 | extra_trees = (tree,) | |
| 369 | else: | |
| 370 | extra_trees = None | |
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 371 | |
| 1551.2.14
by Aaron Bentley Updated argument names, DRY fixes. | 372 | return show_diff_trees(old_tree, new_tree, sys.stdout, specific_files, | 
| 1684.1.6
by Martin Pool (patch) --diff-prefix option (goffredo, alexander) | 373 | external_diff_options, | 
| 1551.7.19
by Aaron Bentley Always include working tree when calculating file ids for diff | 374 | old_label=old_label, new_label=new_label, | 
| 375 | extra_trees=extra_trees) | |
| 1551.2.13
by Aaron Bentley Got diff working properly with checkouts | 376 | |
| 571
by Martin Pool - new --diff-options to pass options through to external | 377 | |
| 378 | def show_diff_trees(old_tree, new_tree, to_file, specific_files=None, | |
| 1684.1.6
by Martin Pool (patch) --diff-prefix option (goffredo, alexander) | 379 | external_diff_options=None, | 
| 1551.7.17
by Aaron Bentley Switch to PathsNotVersioned, accept extra_trees | 380 | old_label='a/', new_label='b/', | 
| 381 | extra_trees=None): | |
| 550
by Martin Pool - Refactor diff code into one that works purely on | 382 | """Show in text form the changes from one tree to another. | 
| 383 | ||
| 384 |     to_files
 | |
| 385 |         If set, include only changes to these files.
 | |
| 571
by Martin Pool - new --diff-options to pass options through to external | 386 | |
| 387 |     external_diff_options
 | |
| 388 |         If set, use an external GNU diff and pass these options.
 | |
| 1551.7.18
by Aaron Bentley Indentation and documentation fixes | 389 | |
| 390 |     extra_trees
 | |
| 391 |         If set, more Trees to use for looking up file ids
 | |
| 550
by Martin Pool - Refactor diff code into one that works purely on | 392 |     """
 | 
| 1543.1.1
by Denys Duchier lock operations for trees - use them for diff | 393 | old_tree.lock_read() | 
| 394 | try: | |
| 2255.7.38
by John Arbash Meinel show_diff_trees() should lock any extra trees it is passed. | 395 | if extra_trees is not None: | 
| 396 | for tree in extra_trees: | |
| 397 | tree.lock_read() | |
| 1543.1.1
by Denys Duchier lock operations for trees - use them for diff | 398 | new_tree.lock_read() | 
| 399 | try: | |
| 400 | return _show_diff_trees(old_tree, new_tree, to_file, | |
| 1684.1.6
by Martin Pool (patch) --diff-prefix option (goffredo, alexander) | 401 | specific_files, external_diff_options, | 
| 1551.7.17
by Aaron Bentley Switch to PathsNotVersioned, accept extra_trees | 402 | old_label=old_label, new_label=new_label, | 
| 403 | extra_trees=extra_trees) | |
| 1543.1.1
by Denys Duchier lock operations for trees - use them for diff | 404 | finally: | 
| 405 | new_tree.unlock() | |
| 2255.7.38
by John Arbash Meinel show_diff_trees() should lock any extra trees it is passed. | 406 | if extra_trees is not None: | 
| 407 | for tree in extra_trees: | |
| 408 | tree.unlock() | |
| 1543.1.1
by Denys Duchier lock operations for trees - use them for diff | 409 | finally: | 
| 410 | old_tree.unlock() | |
| 411 | ||
| 412 | ||
| 413 | def _show_diff_trees(old_tree, new_tree, to_file, | |
| 1684.1.6
by Martin Pool (patch) --diff-prefix option (goffredo, alexander) | 414 | specific_files, external_diff_options, | 
| 1551.7.17
by Aaron Bentley Switch to PathsNotVersioned, accept extra_trees | 415 | old_label='a/', new_label='b/', extra_trees=None): | 
| 329
by Martin Pool - refactor command functions into command classes | 416 | |
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 417 |     # GNU Patch uses the epoch date to detect files that are being added
 | 
| 418 |     # or removed in a diff.
 | |
| 419 | EPOCH_DATE = '1970-01-01 00:00:00 +0000' | |
| 1706.1.2
by James Henstridge Simplify logic a little | 420 | |
| 329
by Martin Pool - refactor command functions into command classes | 421 |     # TODO: Generation of pseudo-diffs for added/deleted files could
 | 
| 422 |     # be usefully made into a much faster special case.
 | |
| 423 | ||
| 571
by Martin Pool - new --diff-options to pass options through to external | 424 | if external_diff_options: | 
| 425 | assert isinstance(external_diff_options, basestring) | |
| 426 | opts = external_diff_options.split() | |
| 427 | def diff_file(olab, olines, nlab, nlines, to_file): | |
| 428 | external_diff(olab, olines, nlab, nlines, to_file, opts) | |
| 429 | else: | |
| 430 | diff_file = internal_diff | |
| 431 | ||
| 1852.10.3
by Robert Collins Remove all uses of compare_trees and replace with Tree.changes_from throughout bzrlib. | 432 | delta = new_tree.changes_from(old_tree, | 
| 433 | specific_files=specific_files, | |
| 434 | extra_trees=extra_trees, require_versioned=True) | |
| 475
by Martin Pool - rewrite diff using compare_trees() | 435 | |
| 1490
by Robert Collins Implement a 'bzr push' command, with saved locations; update diff to return 1. | 436 | has_changes = 0 | 
| 475
by Martin Pool - rewrite diff using compare_trees() | 437 | for path, file_id, kind in delta.removed: | 
| 1490
by Robert Collins Implement a 'bzr push' command, with saved locations; update diff to return 1. | 438 | has_changes = 1 | 
| 1732.1.7
by John Arbash Meinel Instead of iterating randomly in both trees, _compare_trees now iterates in order on both trees simultaneously. | 439 | print >>to_file, '=== removed %s %r' % (kind, path.encode('utf8')) | 
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 440 | old_name = '%s%s\t%s' % (old_label, path, | 
| 441 | _patch_header_date(old_tree, file_id, path)) | |
| 442 | new_name = '%s%s\t%s' % (new_label, path, EPOCH_DATE) | |
| 443 | old_tree.inventory[file_id].diff(diff_file, old_name, old_tree, | |
| 444 | new_name, None, None, to_file) | |
| 475
by Martin Pool - rewrite diff using compare_trees() | 445 | for path, file_id, kind in delta.added: | 
| 1490
by Robert Collins Implement a 'bzr push' command, with saved locations; update diff to return 1. | 446 | has_changes = 1 | 
| 1732.1.7
by John Arbash Meinel Instead of iterating randomly in both trees, _compare_trees now iterates in order on both trees simultaneously. | 447 | print >>to_file, '=== added %s %r' % (kind, path.encode('utf8')) | 
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 448 | old_name = '%s%s\t%s' % (old_label, path, EPOCH_DATE) | 
| 449 | new_name = '%s%s\t%s' % (new_label, path, | |
| 450 | _patch_header_date(new_tree, file_id, path)) | |
| 451 | new_tree.inventory[file_id].diff(diff_file, new_name, new_tree, | |
| 452 | old_name, None, None, to_file, | |
| 1399.1.4
by Robert Collins move diff and symlink conditionals into inventory.py from diff.py | 453 | reverse=True) | 
| 1398
by Robert Collins integrate in Gustavos x-bit patch | 454 | for (old_path, new_path, file_id, kind, | 
| 455 | text_modified, meta_modified) in delta.renamed: | |
| 1490
by Robert Collins Implement a 'bzr push' command, with saved locations; update diff to return 1. | 456 | has_changes = 1 | 
| 1398
by Robert Collins integrate in Gustavos x-bit patch | 457 | prop_str = get_prop_change(meta_modified) | 
| 458 | print >>to_file, '=== renamed %s %r => %r%s' % ( | |
| 1711.2.28
by John Arbash Meinel Fix up bzrlib.diff now that compare_trees is returning unicode paths. | 459 | kind, old_path.encode('utf8'), | 
| 460 | new_path.encode('utf8'), prop_str) | |
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 461 | old_name = '%s%s\t%s' % (old_label, old_path, | 
| 462 | _patch_header_date(old_tree, file_id, | |
| 463 | old_path)) | |
| 464 | new_name = '%s%s\t%s' % (new_label, new_path, | |
| 465 | _patch_header_date(new_tree, file_id, | |
| 466 | new_path)) | |
| 467 | _maybe_diff_file_or_symlink(old_name, old_tree, file_id, | |
| 468 | new_name, new_tree, | |
| 1092.2.9
by Robert Collins bugfix _maybe_diff, the test was not catching the error | 469 | text_modified, kind, to_file, diff_file) | 
| 1398
by Robert Collins integrate in Gustavos x-bit patch | 470 | for path, file_id, kind, text_modified, meta_modified in delta.modified: | 
| 1490
by Robert Collins Implement a 'bzr push' command, with saved locations; update diff to return 1. | 471 | has_changes = 1 | 
| 1398
by Robert Collins integrate in Gustavos x-bit patch | 472 | prop_str = get_prop_change(meta_modified) | 
| 1732.1.7
by John Arbash Meinel Instead of iterating randomly in both trees, _compare_trees now iterates in order on both trees simultaneously. | 473 | print >>to_file, '=== modified %s %r%s' % (kind, path.encode('utf8'), prop_str) | 
| 2405.1.3
by John Arbash Meinel Do a better fix, which recognizes that we should pass the correct old path. | 474 |         # The file may be in a different location in the old tree (because
 | 
| 475 |         # the containing dir was renamed, but the file itself was not)
 | |
| 476 | old_path = old_tree.id2path(file_id) | |
| 477 | old_name = '%s%s\t%s' % (old_label, old_path, | |
| 478 | _patch_header_date(old_tree, file_id, old_path)) | |
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 479 | new_name = '%s%s\t%s' % (new_label, path, | 
| 480 | _patch_header_date(new_tree, file_id, path)) | |
| 1398
by Robert Collins integrate in Gustavos x-bit patch | 481 | if text_modified: | 
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 482 | _maybe_diff_file_or_symlink(old_name, old_tree, file_id, | 
| 483 | new_name, new_tree, | |
| 1398
by Robert Collins integrate in Gustavos x-bit patch | 484 | True, kind, to_file, diff_file) | 
| 1658.1.9
by Martin Pool Give an error for bzr diff on an nonexistent file (Malone #3619) | 485 | |
| 1490
by Robert Collins Implement a 'bzr push' command, with saved locations; update diff to return 1. | 486 | return has_changes | 
| 1658.1.9
by Martin Pool Give an error for bzr diff on an nonexistent file (Malone #3619) | 487 | |
| 488 | ||
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 489 | def _patch_header_date(tree, file_id, path): | 
| 490 | """Returns a timestamp suitable for use in a patch header.""" | |
| 2405.1.2
by John Arbash Meinel Fix bug #103870 by passing None instead of a (sometimes wrong) path | 491 | mtime = tree.get_file_mtime(file_id, path) | 
| 492 | assert mtime is not None, \ | |
| 493 | "got an mtime of None for file-id %s, path %s in tree %s" % ( | |
| 494 | file_id, path, tree) | |
| 495 | return timestamp.format_patch_date(mtime) | |
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 496 | |
| 497 | ||
| 1662.1.9
by Martin Pool Give a clear error for bzr status of an unversioned, nonexistent file. (Malone #3619) | 498 | def _raise_if_nonexistent(paths, old_tree, new_tree): | 
| 499 | """Complain if paths are not in either inventory or tree. | |
| 500 | ||
| 501 |     It's OK with the files exist in either tree's inventory, or 
 | |
| 502 |     if they exist in the tree but are not versioned.
 | |
| 503 |     
 | |
| 504 |     This can be used by operations such as bzr status that can accept
 | |
| 505 |     unknown or ignored files.
 | |
| 506 |     """
 | |
| 507 | mutter("check paths: %r", paths) | |
| 508 | if not paths: | |
| 509 |         return
 | |
| 510 | s = old_tree.filter_unversioned_files(paths) | |
| 511 | s = new_tree.filter_unversioned_files(s) | |
| 512 | s = [path for path in s if not new_tree.has_filename(path)] | |
| 513 | if s: | |
| 514 | raise errors.PathsDoNotExist(sorted(s)) | |
| 515 | ||
| 516 | ||
| 1398
by Robert Collins integrate in Gustavos x-bit patch | 517 | def get_prop_change(meta_modified): | 
| 518 | if meta_modified: | |
| 519 | return " (properties changed)" | |
| 520 | else: | |
| 521 | return "" | |
| 522 | ||
| 523 | ||
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 524 | def _maybe_diff_file_or_symlink(old_path, old_tree, file_id, | 
| 525 | new_path, new_tree, text_modified, | |
| 1092.2.9
by Robert Collins bugfix _maybe_diff, the test was not catching the error | 526 | kind, to_file, diff_file): | 
| 1092.2.6
by Robert Collins symlink support updated to work | 527 | if text_modified: | 
| 1399.1.4
by Robert Collins move diff and symlink conditionals into inventory.py from diff.py | 528 | new_entry = new_tree.inventory[file_id] | 
| 529 | old_tree.inventory[file_id].diff(diff_file, | |
| 1740.2.5
by Aaron Bentley Merge from bzr.dev | 530 | old_path, old_tree, | 
| 531 | new_path, new_entry, | |
| 1399.1.4
by Robert Collins move diff and symlink conditionals into inventory.py from diff.py | 532 | new_tree, to_file) |