zdl411437734 / svnx

Automatically exported from code.google.com/p/svnx
0 stars 0 forks source link

add support for Textmate as diff app #45

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Please could we have support for sending diffs to TextMate?

If my quick glance over the code hasn't lead me astray, it should be a matter 
of appending @"mate" 
to diffAppNames[]. This won't work for people who haven't installed the 
TextMate command-line 
support, but I'd guess most people using svn will have done that :)

Thanks

Original issue reported on code.google.com by wotnosp...@gmail.com on 21 Jul 2009 at 2:26

GoogleCodeExporter commented 9 years ago
SvnX doesn't actually 'send diffs' to any application.
The 'Diff application' set in the preferences is asked to compare 2 files (and 
it then displays any differences).

I have looked at the TextMate website (before & again today) and I don't 
believe TextMate can do this.
If you can point me to documentation that states otherwise & I can test it with 
TextMate I'd be happy
to add support for TextMate as a diff app.

BTW 'appending @"mate" to diffAppNames' will not make it work.
It's a bit more involved :-).

Original comment by chris...@gmail.com on 22 Jul 2009 at 3:39

GoogleCodeExporter commented 9 years ago
Ah, right — I think we'd probably need 'diff -U3 file1 file2 | mate', 
then… TextMate should spot it's a diff by the 
format of the data.

Thanks

Original comment by wotnosp...@gmail.com on 22 Jul 2009 at 12:35

GoogleCodeExporter commented 9 years ago
I think you are missing the point.
Have you tried this with one of the currently supported diff applications?
None of them show only the diff.  They all show the entire contents of both 
files (side-by-side)
& when used with the working copy they allow copying changes into the WC file.

I don't think you can do this by piping the output of diff into mate.

Original comment by chris...@gmail.com on 22 Jul 2009 at 6:02

GoogleCodeExporter commented 9 years ago

Original comment by chris...@gmail.com on 21 Sep 2009 at 4:39