darcs

Issue 2220 Color darcs diff output

Title Color darcs diff output
Priority feature Status needs-diagnosis/design
Milestone Resolved in
Superseder Nosy List mail2
Assigned To
Topics UI

Created on 2012-08-03.16:37:23 by mail2, last changed 2012-08-04.05:34:07 by kowey.

Messages
msg15940 (view) Author: mail2 Date: 2012-08-03.16:37:21
Feature request:

<nh2> kowey: DARCS_ALWAYS_COLOR=1 DARCS_DO_COLOR_LINES=1 darcs diff -u
is still completely colorless

<kowey> nh2: if the latter, coloured diffs may be worth adding as a
feature to darcs (riding on the same option)

Thanks!
msg15941 (view) Author: kowey Date: 2012-08-03.17:11:16
Sounds like a good thing to have for tty output
msg15942 (view) Author: kowey Date: 2012-08-03.17:11:58
We already have whatsnew from Nicolas' work; so maybe not so hard to do?   
You just need to be careful that people trying to create diff+patch style 
files don't get weird garbage
msg15943 (view) Author: mndrix Date: 2012-08-03.20:18:52
`darcs diff` uses external diff tools, so coloring is done there instead.  
I have the following line in ~/.darcs/defaults so that I see color diffs:

  diff diff-command colordiff -rN -u %1 %2
msg15944 (view) Author: mail2 Date: 2012-08-03.20:21:35
That should go into clearly visible documentation. Colored diffs make 
life so much easier.

On Fri 03 Aug 2012 21:18:53 BST, Michael Hendricks wrote:
>
> Michael Hendricks <michael@ndrix.org> added the comment:
>
> `darcs diff` uses external diff tools, so coloring is done there instead.
> I have the following line in ~/.darcs/defaults so that I see color diffs:
>
>   diff diff-command colordiff -rN -u %1 %2
>
> __________________________________
> Darcs bug tracker <bugs@darcs.net>
> <http://bugs.darcs.net/issue2220>
> __________________________________
msg15945 (view) Author: mndrix Date: 2012-08-03.21:45:16
In darcs 2.8.0, `darcs help diff` says "The command always uses an 
external diff utility."  I think that documentation was absent in earlier 
versions.
msg15946 (view) Author: kowey Date: 2012-08-04.05:34:06
FAQ updated: http://darcs.net/FAQ#how-do-i-get-coloured-diffs

Anything more we can do with this? Should Darcs try to be clever about 
using colordiff if it's available?
History
Date User Action Args
2012-08-03 16:37:23mail2create
2012-08-03 17:11:19koweysetpriority: feature
status: unknown -> needs-implementation
topic: + UI
messages: + msg15941
2012-08-03 17:11:59koweysetmessages: + msg15942
2012-08-03 20:18:53mndrixsetmessages: + msg15943
2012-08-03 20:21:36mail2setmessages: + msg15944
2012-08-03 21:45:16mndrixsetmessages: + msg15945
2012-08-04 05:34:07koweysetstatus: needs-implementation -> needs-diagnosis/design
messages: + msg15946