##// END OF EJS Templates
fix: warn when a fixer doesn't have a configured command...
fix: warn when a fixer doesn't have a configured command It seems we currently produce an empty command line and then decide to not run it, but it seems better to skip that part too. Differential Revision: https://phab.mercurial-scm.org/D7085

File last commit:

r43347:687b865b default
r43494:d3d1a3af default
Show More
extdiff.py
721 lines | 23.6 KiB | text/x-python | PythonLexer
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333 # extdiff.py - external diff program support for mercurial
#
# Copyright 2006 Vadim Gelfer <vadim.gelfer@gmail.com>
#
Martin Geisler
updated license to be explicit about GPL version 2
r8225 # This software may be used and distributed according to the terms of the
Matt Mackall
Update license to GPLv2+
r10263 # GNU General Public License version 2 or any later version.
Brendan Cully
Improve extdiff configuration....
r5245
Dirkjan Ochtman
extensions: change descriptions for extensions providing a few commands
r8934 '''command to allow external programs to compare revisions
Dirkjan Ochtman
help: add/fix docstrings for a bunch of extensions
r8873
Cédric Duval
doc: fix quotes mismatches affecting rst
r9286 The extdiff Mercurial extension allows you to use external programs
Martin Geisler
extdiff: wrap docstrings at 70 characters
r9257 to compare revisions, or revision with working directory. The external
diff programs are called with a configurable set of options and two
Brendan Cully
Improve extdiff configuration....
r5245 non-option arguments: paths to directories containing snapshots of
files to compare.
Kyle Lippincott
extdiff: document that it copies modified files back to working directory...
r37226 If there is more than one file being compared and the "child" revision
is the working directory, any modifications made in the external diff
program will be copied back to the working directory from the temporary
directory.
Javi Merino
extdiff: grammar "allows to" -> "allows one to"...
r14327 The extdiff extension also allows you to configure new diff commands, so
Martin Geisler
extdiff: fix reST syntax in module docstring
r11191 you do not need to type :hg:`extdiff -p kdiff3` always. ::
Mathieu Clabaut
Update [extdiff] configuration sample for vimdiff,...
r3127
Brendan Cully
Improve extdiff configuration....
r5245 [extdiff]
# add new command that runs GNU diff(1) in 'context diff' mode
cdiff = gdiff -Nprc5
## or the old way:
#cmd.cdiff = gdiff
#opts.cdiff = -Nprc5
Mathieu Clabaut
Update [extdiff] configuration sample for vimdiff,...
r3127
Matt Harbison
extdiff: allow a preconfigured merge-tool to be invoked...
r23150 # add new command called meld, runs meld (no need to name twice). If
# the meld executable is not available, the meld tool in [merge-tools]
# will be used, if available
Brendan Cully
Improve extdiff configuration....
r5245 meld =
# add new command called vimdiff, runs gvimdiff with DirDiff plugin
Martin Geisler
extdiff: wrap docstrings at 70 characters
r9257 # (see http://www.vim.org/scripts/script.php?script_id=102) Non
# English user, be sure to put "let g:DirDiffDynamicDiffText = 1" in
Brendan Cully
Improve extdiff configuration....
r5245 # your .vimrc
Thomas Arendsen Hein
extdiff: escape filenames with vim/DirDiff and make quoting work with Windows...
r16242 vimdiff = gvim -f "+next" \\
"+execute 'DirDiff' fnameescape(argv(0)) fnameescape(argv(1))"
Brendan Cully
Improve extdiff configuration....
r5245
Martin Geisler
extdiff: fix reST syntax in module docstring
r11191 Tool arguments can include variables that are expanded at runtime::
Steve Borho
extdiff: add labels, read diff arguments from [merge-tools]...
r11184
$parent1, $plabel1 - filename, descriptive label of first parent
$child, $clabel - filename, descriptive label of child revision
$parent2, $plabel2 - filename, descriptive label of second parent
Steven Stallion
extdiff: add repository root as a variable...
r14045 $root - repository root
Steve Borho
extdiff: add labels, read diff arguments from [merge-tools]...
r11184 $parent is an alias for $parent1.
The extdiff extension will look in your [diff-tools] and [merge-tools]
sections for diff tool arguments, when none are specified in [extdiff].
Martin Geisler
extdiff: fix reST syntax in module docstring
r11191 ::
Steve Borho
extdiff: add labels, read diff arguments from [merge-tools]...
r11184 [extdiff]
Martin Geisler
extdiff: fix reST syntax in module docstring
r11191 kdiff3 =
Steve Borho
extdiff: add labels, read diff arguments from [merge-tools]...
r11184
[diff-tools]
kdiff3.diffargs=--L1 '$plabel1' --L2 '$clabel' $parent $child
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 If a program has a graphical interface, it might be interesting to tell
Mercurial about it. It will prevent the program from being mistakenly
used in a terminal-only environment (such as an SSH terminal session),
and will make :hg:`extdiff --per-file` open multiple file diffs at once
instead of one by one (if you still want to open file diffs one by one,
you can use the --confirm option).
Declaring that a tool has a graphical interface can be done with the
``gui`` flag next to where ``diffargs`` are specified:
::
[diff-tools]
kdiff3.diffargs=--L1 '$plabel1' --L2 '$clabel' $parent $child
kdiff3.gui = true
Martin Geisler
extdiff: fix reST syntax in module docstring
r11191 You can use -I/-X and list of file or directory names like normal
:hg:`diff` command. The extdiff extension makes snapshots of only
needed files, so running the external diff program will actually be
pretty fast (at least faster than having to compare the entire tree).
Brendan Cully
Improve extdiff configuration....
r5245 '''
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333
Pulkit Goyal
py3: make extdiff use absolute_import
r28970 from __future__ import absolute_import
import os
import re
import shutil
Augie Fackler
cleanup: use stat_result[stat.ST_MTIME] instead of stat_result.st_mtime...
r36799 import stat
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 import subprocess
Yuya Nishihara
py3: wrap tempfile.mkdtemp() to use bytes path...
r38183
Matt Mackall
Simplify i18n imports
r3891 from mercurial.i18n import _
Pulkit Goyal
py3: make extdiff use absolute_import
r28970 from mercurial.node import (
nullid,
short,
)
from mercurial import (
archival,
cmdutil,
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 encoding,
Pulkit Goyal
py3: make extdiff use absolute_import
r28970 error,
filemerge,
Yuya Nishihara
export: enable formatter support (API)...
r37622 formatter,
Pulkit Goyal
py3: have a bytes version of shlex.split()...
r30678 pycompat,
Yuya Nishihara
registrar: move cmdutil.command to registrar module (API)...
r32337 registrar,
Pulkit Goyal
py3: make extdiff use absolute_import
r28970 scmutil,
util,
)
Yuya Nishihara
stringutil: bulk-replace call sites to point to new module...
r37102 from mercurial.utils import (
Yuya Nishihara
procutil: bulk-replace function calls to point to new module
r37138 procutil,
Yuya Nishihara
stringutil: bulk-replace call sites to point to new module...
r37102 stringutil,
)
Brad Schick
extdiff: un-nested two functions...
r5135
Gregory Szorc
extdiff: declare command using decorator
r21246 cmdtable = {}
Yuya Nishihara
registrar: move cmdutil.command to registrar module (API)...
r32337 command = registrar.command(cmdtable)
Boris Feld
configitems: register the 'exdiff.opts.*' config
r34778
configtable = {}
configitem = registrar.configitem(configtable)
Augie Fackler
formatting: blacken the codebase...
r43346 configitem(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'extdiff', br'opts\..*', default=b'', generic=True,
Boris Feld
configitems: register the 'exdiff.opts.*' config
r34778 )
Augie Fackler
formatting: blacken the codebase...
r43346 configitem(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'extdiff', br'gui\..*', generic=True,
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 )
Augie Fackler
formatting: blacken the codebase...
r43346 configitem(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'diff-tools', br'.*\.diffargs$', default=None, generic=True,
Boris Feld
configitems: register the 'extdata.*.diffargs' config
r34779 )
Augie Fackler
formatting: blacken the codebase...
r43346 configitem(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'diff-tools', br'.*\.gui$', generic=True,
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 )
Augie Fackler
extensions: change magic "shipped with hg" string...
r29841 # Note for extension authors: ONLY specify testedwith = 'ships-with-hg-core' for
Augie Fackler
extensions: document that `testedwith = 'internal'` is special...
r25186 # extensions which SHIP WITH MERCURIAL. Non-mainline extensions should
# be specifying the version(s) of Mercurial they are tested with, or
# leave the attribute unspecified.
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 testedwith = b'ships-with-hg-core'
Augie Fackler
hgext: mark all first-party extensions as such
r16743
Augie Fackler
formatting: blacken the codebase...
r43346
Matt Harbison
extdiff: add support for subrepos...
r25813 def snapshot(ui, repo, files, node, tmproot, listsubrepos):
Patrick Mezard
extdiff: merge node and working dir snapshot modes
r8064 '''snapshot files as of some revision
if not using snapshot, -I/-X does not work and recursive diff
in tools like kdiff3 and meld displays too many files.'''
Brad Schick
extdiff: un-nested two functions...
r5135 dirname = os.path.basename(repo.root)
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 if dirname == b"":
dirname = b"root"
Patrick Mezard
extdiff: merge node and working dir snapshot modes
r8064 if node is not None:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 dirname = b'%s.%s' % (dirname, short(node))
Brad Schick
extdiff: un-nested two functions...
r5135 base = os.path.join(tmproot, dirname)
os.mkdir(base)
Matt Harbison
extdiff: copy back files to the working directory if the size changed...
r32212 fnsandstat = []
Matt Harbison
extdiff: use archiver to take snapshots of committed revisions...
r25812
Patrick Mezard
extdiff: merge node and working dir snapshot modes
r8064 if node is not None:
Augie Fackler
formatting: blacken the codebase...
r43346 ui.note(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 _(b'making snapshot of %d files from rev %s\n')
Augie Fackler
formatting: blacken the codebase...
r43346 % (len(files), short(node))
)
Patrick Mezard
extdiff: merge node and working dir snapshot modes
r8064 else:
Augie Fackler
formatting: blacken the codebase...
r43346 ui.note(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 _(b'making snapshot of %d files from working directory\n')
Augie Fackler
formatting: blacken the codebase...
r43346 % (len(files))
)
Matt Harbison
extdiff: use archiver to take snapshots of committed revisions...
r25812
if files:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 repo.ui.setconfig(b"ui", b"archivemeta", False)
Matt Harbison
extdiff: use archiver to take snapshots of committed revisions...
r25812
Augie Fackler
formatting: blacken the codebase...
r43346 archival.archive(
repo,
base,
node,
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'files',
Augie Fackler
formatting: blacken the codebase...
r43346 match=scmutil.matchfiles(repo, files),
subrepos=listsubrepos,
)
Matt Harbison
extdiff: use archiver to take snapshots of committed revisions...
r25812
for fn in sorted(files):
wfn = util.pconvert(fn)
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 ui.note(b' %s\n' % wfn)
Matt Harbison
extdiff: use archiver to take snapshots of committed revisions...
r25812
if node is None:
dest = os.path.join(base, wfn)
Matt Harbison
extdiff: copy back files to the working directory if the size changed...
r32212 fnsandstat.append((dest, repo.wjoin(fn), os.lstat(dest)))
return dirname, fnsandstat
Thomas Arendsen Hein
Remove trailing spaces, fix indentation
r5143
Augie Fackler
formatting: blacken the codebase...
r43346
def formatcmdline(
cmdline,
repo_root,
do3way,
parent1,
plabel1,
parent2,
plabel2,
child,
clabel,
):
Ludovic Chabant
extdiff: move external tool command line building into separate function
r41232 # Function to quote file/dir names in the argument string.
# When not operating in 3-way mode, an empty string is
# returned for parent2
Augie Fackler
formatting: blacken the codebase...
r43346 replace = {
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'parent': parent1,
b'parent1': parent1,
b'parent2': parent2,
b'plabel1': plabel1,
b'plabel2': plabel2,
b'child': child,
b'clabel': clabel,
b'root': repo_root,
Augie Fackler
formatting: blacken the codebase...
r43346 }
Ludovic Chabant
extdiff: move external tool command line building into separate function
r41232 def quote(match):
pre = match.group(2)
key = match.group(3)
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 if not do3way and key == b'parent2':
Ludovic Chabant
extdiff: move external tool command line building into separate function
r41232 return pre
return pre + procutil.shellquote(replace[key])
# Match parent2 first, so 'parent1?' will match both parent1 and parent
Augie Fackler
formatting: blacken the codebase...
r43346 regex = (
br'''(['"]?)([^\s'"$]*)'''
br'\$(parent2|parent1?|child|plabel1|plabel2|clabel|root)\1'
)
Ludovic Chabant
extdiff: move external tool command line building into separate function
r41232 if not do3way and not re.search(regex, cmdline):
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 cmdline += b' $parent1 $child'
Ludovic Chabant
extdiff: move external tool command line building into separate function
r41232 return re.sub(regex, quote, cmdline)
Augie Fackler
formatting: blacken the codebase...
r43346
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 def _systembackground(cmd, environ=None, cwd=None):
''' like 'procutil.system', but returns the Popen object directly
so we don't have to wait on it.
'''
cmd = procutil.quotecommand(cmd)
env = procutil.shellenviron(environ)
Augie Fackler
formatting: blacken the codebase...
r43346 proc = subprocess.Popen(
procutil.tonativestr(cmd),
shell=True,
close_fds=procutil.closefds,
env=procutil.tonativeenv(env),
cwd=pycompat.rapply(procutil.tonativestr, cwd),
)
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 return proc
Augie Fackler
formatting: blacken the codebase...
r43346
def _runperfilediff(
cmdline,
repo_root,
ui,
guitool,
do3way,
confirm,
commonfiles,
tmproot,
dir1a,
dir1b,
dir2root,
dir2,
rev1a,
rev1b,
rev2,
):
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 # Note that we need to sort the list of files because it was
# built in an "unstable" way and it's annoying to get files in a
# random order, especially when "confirm" mode is enabled.
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 waitprocs = []
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 totalfiles = len(commonfiles)
for idx, commonfile in enumerate(sorted(commonfiles)):
path1a = os.path.join(tmproot, dir1a, commonfile)
label1a = commonfile + rev1a
if not os.path.isfile(path1a):
path1a = os.devnull
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 path1b = b''
label1b = b''
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 if do3way:
path1b = os.path.join(tmproot, dir1b, commonfile)
label1b = commonfile + rev1b
if not os.path.isfile(path1b):
path1b = os.devnull
path2 = os.path.join(dir2root, dir2, commonfile)
label2 = commonfile + rev2
if confirm:
# Prompt before showing this diff
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 difffiles = _(b'diff %s (%d of %d)') % (
Augie Fackler
formatting: blacken the codebase...
r43346 commonfile,
idx + 1,
totalfiles,
)
responses = _(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'[Yns?]'
b'$$ &Yes, show diff'
b'$$ &No, skip this diff'
b'$$ &Skip remaining diffs'
b'$$ &? (display help)'
Augie Fackler
formatting: blacken the codebase...
r43346 )
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 r = ui.promptchoice(b'%s %s' % (difffiles, responses))
Augie Fackler
formatting: blacken the codebase...
r43346 if r == 3: # ?
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 while r == 3:
for c, t in ui.extractchoices(responses)[1]:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 ui.write(b'%s - %s\n' % (c, encoding.lower(t)))
r = ui.promptchoice(b'%s %s' % (difffiles, responses))
Augie Fackler
formatting: blacken the codebase...
r43346 if r == 0: # yes
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 pass
Augie Fackler
formatting: blacken the codebase...
r43346 elif r == 1: # no
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 continue
Augie Fackler
formatting: blacken the codebase...
r43346 elif r == 2: # skip
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 break
curcmdline = formatcmdline(
Augie Fackler
formatting: blacken the codebase...
r43346 cmdline,
repo_root,
do3way=do3way,
parent1=path1a,
plabel1=label1a,
parent2=path1b,
plabel2=label1b,
child=path2,
clabel=label2,
)
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 if confirm or not guitool:
# Run the comparison program and wait for it to exit
# before we show the next file.
# This is because either we need to wait for confirmation
# from the user between each invocation, or because, as far
# as we know, the tool doesn't have a GUI, in which case
# we can't run multiple CLI programs at the same time.
Augie Fackler
formatting: blacken the codebase...
r43346 ui.debug(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'running %r in %s\n' % (pycompat.bytestr(curcmdline), tmproot)
Augie Fackler
formatting: blacken the codebase...
r43346 )
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 ui.system(curcmdline, cwd=tmproot, blockedtag=b'extdiff')
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 else:
# Run the comparison program but don't wait, as we're
# going to rapid-fire each file diff and then wait on
# the whole group.
Augie Fackler
formatting: blacken the codebase...
r43346 ui.debug(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'running %r in %s (backgrounded)\n'
Augie Fackler
formatting: blacken the codebase...
r43346 % (pycompat.bytestr(curcmdline), tmproot)
)
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 proc = _systembackground(curcmdline, cwd=tmproot)
waitprocs.append(proc)
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 if waitprocs:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 with ui.timeblockedsection(b'extdiff'):
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 for proc in waitprocs:
proc.wait()
Augie Fackler
formatting: blacken the codebase...
r43346
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 def dodiff(ui, repo, cmdline, pats, opts, guitool=False):
Mads Kiilerich
fix trivial spelling errors
r17424 '''Do the actual diff:
Fabio Zadrozny <fabiofz at gmail dot com>
Propagating changes back to working dirs when changing files in external
r6103
- copy to a temp structure if diffing 2 internal revisions
- copy to a temp structure if diffing working revision with
another one and more than 1 file is changed
- just invoke the diff for a single file in the working dir
'''
Gilles Moris
extdiff: add --change option to display single changeset diff...
r7758
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 revs = opts.get(b'rev')
change = opts.get(b'change')
do3way = b'$parent2' in cmdline
Gilles Moris
extdiff: add --change option to display single changeset diff...
r7758
if revs and change:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 msg = _(b'cannot specify --rev and --change at the same time')
Pierre-Yves David
error: get Abort from 'error' instead of 'util'...
r26587 raise error.Abort(msg)
Gilles Moris
extdiff: add --change option to display single changeset diff...
r7758 elif change:
Martin von Zweigbergk
extdiff: use context-returning revpair()...
r37270 ctx2 = scmutil.revsingle(repo, change, None)
ctx1a, ctx1b = ctx2.p1(), ctx2.p2()
Gilles Moris
extdiff: add --change option to display single changeset diff...
r7758 else:
Martin von Zweigbergk
extdiff: use context-returning revpair()...
r37270 ctx1a, ctx2 = scmutil.revpair(repo, revs)
Sune Foldager
extdiff: add 3-way diff for merge changesets...
r9512 if not revs:
Martin von Zweigbergk
extdiff: use context-returning revpair()...
r37270 ctx1b = repo[None].p2()
Sune Foldager
extdiff: add 3-way diff for merge changesets...
r9512 else:
Martin von Zweigbergk
extdiff: use context-returning revpair()...
r37270 ctx1b = repo[nullid]
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 perfile = opts.get(b'per_file')
confirm = opts.get(b'confirm')
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628
Martin von Zweigbergk
extdiff: use context-returning revpair()...
r37270 node1a = ctx1a.node()
node1b = ctx1b.node()
node2 = ctx2.node()
Sune Foldager
extdiff: add 3-way diff for merge changesets...
r9512
# Disable 3-way merge if there is only one parent
if do3way:
if node1b == nullid:
do3way = False
Gilles Moris
extdiff: add --change option to display single changeset diff...
r7758
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 subrepos = opts.get(b'subrepos')
Matt Harbison
extdiff: add support for subrepos...
r25813
Matt Mackall
scmutil: switch match users to supplying contexts...
r14671 matcher = scmutil.match(repo[node2], pats, opts)
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 if opts.get(b'patch'):
Matt Harbison
extdiff: add a --patch argument for diffing changeset deltas...
r26228 if subrepos:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 raise error.Abort(_(b'--patch cannot be used with --subrepos'))
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 if perfile:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 raise error.Abort(_(b'--patch cannot be used with --per-file'))
Matt Harbison
extdiff: add a --patch argument for diffing changeset deltas...
r26228 if node2 is None:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 raise error.Abort(_(b'--patch requires two revisions'))
Matt Harbison
extdiff: add a --patch argument for diffing changeset deltas...
r26228 else:
Augie Fackler
formatting: blacken the codebase...
r43346 mod_a, add_a, rem_a = map(
set, repo.status(node1a, node2, matcher, listsubrepos=subrepos)[:3]
)
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 if do3way:
Augie Fackler
formatting: blacken the codebase...
r43346 mod_b, add_b, rem_b = map(
set,
repo.status(node1b, node2, matcher, listsubrepos=subrepos)[:3],
)
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 else:
mod_b, add_b, rem_b = set(), set(), set()
modadd = mod_a | add_a | mod_b | add_b
common = modadd | rem_a | rem_b
if not common:
return 0
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 tmproot = pycompat.mkdtemp(prefix=b'extdiff.')
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333 try:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 if not opts.get(b'patch'):
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 # Always make a copy of node1a (and node1b, if applicable)
dir1a_files = mod_a | rem_a | ((mod_b | add_b) - add_a)
Augie Fackler
formatting: blacken the codebase...
r43346 dir1a = snapshot(ui, repo, dir1a_files, node1a, tmproot, subrepos)[
0
]
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 rev1a = b'@%d' % repo[node1a].rev()
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 if do3way:
dir1b_files = mod_b | rem_b | ((mod_a | add_a) - add_b)
Augie Fackler
formatting: blacken the codebase...
r43346 dir1b = snapshot(
ui, repo, dir1b_files, node1b, tmproot, subrepos
)[0]
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 rev1b = b'@%d' % repo[node1b].rev()
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 else:
dir1b = None
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 rev1b = b''
Sune Foldager
extdiff: add 3-way diff for merge changesets...
r9512
Matt Harbison
extdiff: copy back files to the working directory if the size changed...
r32212 fnsandstat = []
Brad Schick
extdiff: do single file diffs from the wc with no copy...
r5137
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 # If node2 in not the wc or there is >1 change, copy it
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 dir2root = b''
rev2 = b''
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 if node2:
dir2 = snapshot(ui, repo, modadd, node2, tmproot, subrepos)[0]
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 rev2 = b'@%d' % repo[node2].rev()
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 elif len(common) > 1:
Augie Fackler
formatting: blacken the codebase...
r43346 # we only actually need to get the files to copy back to
# the working dir in this case (because the other cases
# are: diffing 2 revisions or single file -- in which case
# the file is already directly passed to the diff tool).
dir2, fnsandstat = snapshot(
ui, repo, modadd, None, tmproot, subrepos
)
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 else:
# This lets the diff tool open the changed file directly
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 dir2 = b''
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 dir2root = repo.root
Brad Schick
extdiff: do single file diffs from the wc with no copy...
r5137
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 label1a = rev1a
label1b = rev1b
label2 = rev2
Steve Borho
extdiff: add labels, read diff arguments from [merge-tools]...
r11184
Matt Harbison
extdiff: prepare sections of dodiff() for conditionalizing...
r26227 # If only one change, diff the files instead of the directories
# Handle bogus modifies correctly by checking if the files exist
if len(common) == 1:
common_file = util.localpath(common.pop())
dir1a = os.path.join(tmproot, dir1a, common_file)
label1a = common_file + rev1a
if not os.path.isfile(dir1a):
dir1a = os.devnull
if do3way:
dir1b = os.path.join(tmproot, dir1b, common_file)
label1b = common_file + rev1b
if not os.path.isfile(dir1b):
dir1b = os.devnull
dir2 = os.path.join(dir2root, dir2, common_file)
label2 = common_file + rev2
Matt Harbison
extdiff: add a --patch argument for diffing changeset deltas...
r26228 else:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 template = b'hg-%h.patch'
with formatter.nullformatter(ui, b'extdiff', {}) as fm:
Augie Fackler
formatting: blacken the codebase...
r43346 cmdutil.export(
repo,
[repo[node1a].rev(), repo[node2].rev()],
fm,
fntemplate=repo.vfs.reljoin(tmproot, template),
match=matcher,
)
Yuya Nishihara
cmdutil: pass ctx to makefilename() in place of repo/node pair (API)
r36222 label1a = cmdutil.makefilename(repo[node1a], template)
label2 = cmdutil.makefilename(repo[node2], template)
Matt Harbison
extdiff: add a --patch argument for diffing changeset deltas...
r26228 dir1a = repo.vfs.reljoin(tmproot, label1a)
dir2 = repo.vfs.reljoin(tmproot, label2)
dir1b = None
label1b = None
Matt Harbison
extdiff: copy back files to the working directory if the size changed...
r32212 fnsandstat = []
Thomas Arendsen Hein
Remove trailing spaces, fix indentation
r5143
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 if not perfile:
# Run the external tool on the 2 temp directories or the patches
cmdline = formatcmdline(
Augie Fackler
formatting: blacken the codebase...
r43346 cmdline,
repo.root,
do3way=do3way,
parent1=dir1a,
plabel1=label1a,
parent2=dir1b,
plabel2=label1b,
child=dir2,
clabel=label2,
)
ui.debug(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'running %r in %s\n' % (pycompat.bytestr(cmdline), tmproot)
Augie Fackler
formatting: blacken the codebase...
r43346 )
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 ui.system(cmdline, cwd=tmproot, blockedtag=b'extdiff')
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 else:
# Run the external tool once for each pair of files
_runperfilediff(
Augie Fackler
formatting: blacken the codebase...
r43346 cmdline,
repo.root,
ui,
guitool=guitool,
do3way=do3way,
confirm=confirm,
commonfiles=common,
tmproot=tmproot,
dir1a=dir1a,
dir1b=dir1b,
dir2root=dir2root,
dir2=dir2,
rev1a=rev1a,
rev1b=rev1b,
rev2=rev2,
)
Fabio Zadrozny <fabiofz at gmail dot com>
Propagating changes back to working dirs when changing files in external
r6103
Matt Harbison
extdiff: copy back files to the working directory if the size changed...
r32212 for copy_fn, working_fn, st in fnsandstat:
cpstat = os.lstat(copy_fn)
# Some tools copy the file and attributes, so mtime may not detect
# all changes. A size check will detect more cases, but not all.
# The only certain way to detect every case is to diff all files,
# which could be expensive.
Matt Harbison
extdiff: copy back execbit-only changes to the working directory...
r32283 # copyfile() carries over the permission, so the mode check could
# be in an 'elif' branch, but for the case where the file has
# changed without affecting mtime or size.
Augie Fackler
formatting: blacken the codebase...
r43346 if (
cpstat[stat.ST_MTIME] != st[stat.ST_MTIME]
Augie Fackler
cleanup: use stat_result[stat.ST_MTIME] instead of stat_result.st_mtime...
r36799 or cpstat.st_size != st.st_size
Augie Fackler
formatting: blacken the codebase...
r43346 or (cpstat.st_mode & 0o100) != (st.st_mode & 0o100)
):
ui.debug(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'file changed while diffing. '
b'Overwriting: %s (src: %s)\n' % (working_fn, copy_fn)
Augie Fackler
formatting: blacken the codebase...
r43346 )
Fabio Zadrozny <fabiofz at gmail dot com>
Propagating changes back to working dirs when changing files in external
r6103 util.copyfile(copy_fn, working_fn)
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333 return 1
finally:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 ui.note(_(b'cleaning up temp directory\n'))
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333 shutil.rmtree(tmproot)
Yuya Nishihara
extdiff: factor out list of common options
r27680
Augie Fackler
formatting: blacken the codebase...
r43346 extdiffopts = (
[
(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'o',
b'option',
[],
_(b'pass option to comparison program'),
_(b'OPT'),
),
(b'r', b'rev', [], _(b'revision'), _(b'REV')),
(b'c', b'change', b'', _(b'change made by revision'), _(b'REV')),
(
b'',
b'per-file',
Augie Fackler
formatting: blacken the codebase...
r43346 False,
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 _(b'compare each file instead of revision snapshots'),
Augie Fackler
formatting: blacken the codebase...
r43346 ),
(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'',
b'confirm',
Augie Fackler
formatting: blacken the codebase...
r43346 False,
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 _(b'prompt user before each external program invocation'),
Augie Fackler
formatting: blacken the codebase...
r43346 ),
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 (b'', b'patch', None, _(b'compare patches for two revisions')),
Augie Fackler
formatting: blacken the codebase...
r43346 ]
+ cmdutil.walkopts
+ cmdutil.subrepoopts
)
@command(
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 b'extdiff',
[(b'p', b'program', b'', _(b'comparison program to run'), _(b'CMD')),]
Augie Fackler
formatting: blacken the codebase...
r43346 + extdiffopts,
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 _(b'hg extdiff [OPT]... [FILE]...'),
rdamazio@google.com
help: assigning categories to existing commands...
r40329 helpcategory=command.CATEGORY_FILE_CONTENTS,
Augie Fackler
formatting: blacken the codebase...
r43346 inferrepo=True,
)
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333 def extdiff(ui, repo, *pats, **opts):
'''use external program to diff repository (or selected files)
Show differences between revisions for the specified files, using
Martin Geisler
Change double spaces to single spaces in help texts.
r7983 an external program. The default program used is diff, with
Vadim Gelfer
extdiff: fix bugs. add test.
r2906 default options "-Npru".
Martin Geisler
help texts: write command line switches as -a/--abc
r8076 To select a different program, use the -p/--program option. The
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 program will be passed the names of two directories to compare,
unless the --per-file option is specified (see below). To pass
additional options to the program, use -o/--option. These will be
passed before the names of the directories or files to compare.
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333
Martin Geisler
extdiff: word-wrap help texts at 70 characters
r7990 When two revision arguments are given, then changes are shown
between those revisions. If only one revision is specified then
that revision is compared to the working directory, and, when no
revisions are specified, the working directory files are compared
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628 to its parent.
The --per-file option runs the external program repeatedly on each
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 file to diff, instead of once on two directories. By default,
this happens one by one, where the next file diff is open in the
external program only once the previous external program (for the
previous file diff) has exited. If the external program has a
graphical interface, it can open all the file diffs at once instead
of one by one. See :hg:`help -e extdiff` for information about how
to tell Mercurial that a given program has a graphical interface.
Ludovic Chabant
extdiff: add --per-file and --confirm options...
r41628
The --confirm option will prompt the user before each invocation of
the external program. It is ignored if --per-file isn't specified.
'''
Pulkit Goyal
py3: handle keyword arguments in hgext/extdiff.py...
r34977 opts = pycompat.byteskwargs(opts)
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 program = opts.get(b'program')
option = opts.get(b'option')
Peter Arrenbrecht
extdiff: fix defaulting to "diff" if no --program is given
r9519 if not program:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 program = b'diff'
option = option or [b'-Npru']
cmdline = b' '.join(map(procutil.shellquote, [program] + option))
FUJIWARA Katsunori
extdiff: avoid unexpected quoting arguments for external tools (issue4463)...
r23680 return dodiff(ui, repo, cmdline, pats, opts)
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333
Augie Fackler
formatting: blacken the codebase...
r43346
Yuya Nishihara
extdiff: refactor closure of saved diff command as a top-level class...
r29721 class savedcmd(object):
Yuya Nishihara
extdiff: isolate path variable of saved command to independent paragraph...
r29723 """use external program to diff repository (or selected files)
Yuya Nishihara
extdiff: refactor closure of saved diff command as a top-level class...
r29721
Show differences between revisions for the specified files, using
Yuya Nishihara
extdiff: isolate path variable of saved command to independent paragraph...
r29723 the following program::
%(path)s
Yuya Nishihara
extdiff: refactor closure of saved diff command as a top-level class...
r29721
When two revision arguments are given, then changes are shown
between those revisions. If only one revision is specified then
that revision is compared to the working directory, and, when no
revisions are specified, the working directory files are compared
to its parent.
"""
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 def __init__(self, path, cmdline, isgui):
Yuya Nishihara
extdiff: refactor closure of saved diff command as a top-level class...
r29721 # We can't pass non-ASCII through docstrings (and path is
Matt Harbison
extdiff: avoid double backslashes in the displayed tool path on Windows...
r40842 # in an unknown encoding anyway), but avoid double separators on
# Windows
docpath = stringutil.escapestr(path).replace(b'\\\\', b'\\')
Yuya Nishihara
stringutil: bulk-replace call sites to point to new module...
r37102 self.__doc__ %= {r'path': pycompat.sysstr(stringutil.uirepr(docpath))}
Yuya Nishihara
extdiff: refactor closure of saved diff command as a top-level class...
r29721 self._cmdline = cmdline
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 self._isgui = isgui
Yuya Nishihara
extdiff: refactor closure of saved diff command as a top-level class...
r29721
def __call__(self, ui, repo, *pats, **opts):
Pulkit Goyal
py3: handle keyword arguments in hgext/extdiff.py...
r34977 opts = pycompat.byteskwargs(opts)
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 options = b' '.join(map(procutil.shellquote, opts[b'option']))
Yuya Nishihara
extdiff: refactor closure of saved diff command as a top-level class...
r29721 if options:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 options = b' ' + options
Augie Fackler
formatting: blacken the codebase...
r43346 return dodiff(
ui, repo, self._cmdline + options, pats, opts, guitool=self._isgui
)
Yuya Nishihara
extdiff: refactor closure of saved diff command as a top-level class...
r29721
Vadim Gelfer
new extension: extdiff. allows to use external diff program.
r2333 def uisetup(ui):
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 for cmd, path in ui.configitems(b'extdiff'):
Jordi Gutiérrez Hermoso
extdiff: expand tildes and variables in paths to user-supplied diff programs
r24193 path = util.expandpath(path)
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 if cmd.startswith(b'cmd.'):
Brendan Cully
Improve extdiff configuration....
r5245 cmd = cmd[4:]
Matt Mackall
many, many trivial check-code fixups
r10282 if not path:
Yuya Nishihara
procutil: bulk-replace function calls to point to new module
r37138 path = procutil.findexe(cmd)
Matt Harbison
extdiff: allow a preconfigured merge-tool to be invoked...
r23150 if path is None:
path = filemerge.findexternaltool(ui, cmd) or cmd
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 diffopts = ui.config(b'extdiff', b'opts.' + cmd)
Yuya Nishihara
procutil: bulk-replace function calls to point to new module
r37138 cmdline = procutil.shellquote(path)
FUJIWARA Katsunori
extdiff: avoid unexpected quoting arguments for external tools (issue4463)...
r23680 if diffopts:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 cmdline += b' ' + diffopts
isgui = ui.configbool(b'extdiff', b'gui.' + cmd)
elif cmd.startswith(b'opts.') or cmd.startswith(b'gui.'):
Brendan Cully
Improve extdiff configuration....
r5245 continue
else:
if path:
FUJIWARA Katsunori
extdiff: avoid unexpected quoting arguments for external tools (issue4463)...
r23680 # case "cmd = path opts"
cmdline = path
Pulkit Goyal
py3: have a bytes version of shlex.split()...
r30678 diffopts = len(pycompat.shlexsplit(cmdline)) > 1
Brendan Cully
Improve extdiff configuration....
r5245 else:
FUJIWARA Katsunori
extdiff: avoid unexpected quoting arguments for external tools (issue4463)...
r23680 # case "cmd ="
Yuya Nishihara
procutil: bulk-replace function calls to point to new module
r37138 path = procutil.findexe(cmd)
Matt Harbison
extdiff: allow a preconfigured merge-tool to be invoked...
r23150 if path is None:
path = filemerge.findexternaltool(ui, cmd) or cmd
Yuya Nishihara
procutil: bulk-replace function calls to point to new module
r37138 cmdline = procutil.shellquote(path)
FUJIWARA Katsunori
extdiff: avoid unexpected quoting arguments for external tools (issue4463)...
r23680 diffopts = False
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 isgui = ui.configbool(b'extdiff', b'gui.' + cmd)
Steve Borho
extdiff: add labels, read diff arguments from [merge-tools]...
r11184 # look for diff arguments in [diff-tools] then [merge-tools]
FUJIWARA Katsunori
extdiff: avoid unexpected quoting arguments for external tools (issue4463)...
r23680 if not diffopts:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 key = cmd + b'.diffargs'
for section in (b'diff-tools', b'merge-tools'):
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 args = ui.config(section, key)
if args:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 cmdline += b' ' + args
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 if isgui is None:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 isgui = ui.configbool(section, cmd + b'.gui') or False
Ludovic Chabant
extdiff: support tools that can be run simultaneously
r41724 break
Augie Fackler
formatting: blacken the codebase...
r43346 command(
cmd,
extdiffopts[:],
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 _(b'hg %s [OPTION]... [FILE]...') % cmd,
Augie Fackler
formatting: blacken the codebase...
r43346 helpcategory=command.CATEGORY_FILE_CONTENTS,
inferrepo=True,
)(savedcmd(path, cmdline, isgui))
Yuya Nishihara
extdiff: export __doc__ of saved command for translation
r29722
# tell hggettext to extract docstrings from these functions:
i18nfunctions = [savedcmd]