[TxMt] FileMerge revisited

[Trevor Harmon]

On Apr 27, 2006, at 10:35 PM, Allan Odgaard wrote:

>> [...] The problem, though, is that TextMate is hard-coded for diff  
>> [...]
>
> That’s intentional, since some user had his svn diff set to  
> something generating tty coloring of the diff output, useful in  
> Terminal, but not TextMate.

Okay, I think I've come up with a solution that will make everyone  
happy. My version of svn_diff now works like this:

If a custom diff is specified in ~/.subversion/config, and the  
TM_SVN_DIFF_CMD variable is not set, TextMate will use the custom  
diff (and assume it's a GUI app). Otherwise, if the TM_SVN_DIFF_CMD  
variable is set, TextMate will use that instead (and assume it's a  
console app). And if not, it will force svn to use plain old diff.

So, all the cases are handled. The guy who needed to override his  
custom Subversion diff command for TextMate can do so; those who  
simply want TextMate to use their graphical diff command (specified  
in Subversion's config file) will see that happen automatically. And  
as before, those who haven't specified a custom command won't see any  
difference in the way TextMate does diffs.

I'd like to submit the patch to the repo, but I don't have commit  
rights. Can someone take a look at this and approve it for me?

Trevor

P.S. You may have noticed that my last two posts to this list (which  
were identical to the post above; I was trying to resend) were  
corrupted. I don't know what's going on. For some reason, the mail  
server has recently decided to chew up my mail on occasion.  I assume  
it has something to do with my digital signature, but I've been using  
the same certificate ever since I started on this list and didn't  
have problems until recently. Does anyone know if there have been  
changes to this list server?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: svn_diff.rb.patch
Type: application/octet-stream
Size: 2723 bytes
Desc: not available
URL: <http://lists.macromates.com/textmate/attachments/20060502/e50be257/attachment.patch>