Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 16924 - colordiff doesnt add escape sequences to turn off colors in output
Summary: colordiff doesnt add escape sequences to turn off colors in output
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Seth Chandler
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-03-05 17:26 UTC by gatto
Modified: 2003-03-07 19:21 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
patch to fix colordiff.pl (colordiff.pl.diff,1.65 KB, patch)
2003-03-05 17:29 UTC, gatto
Details | Diff
patch to update BUGS file (BUGS.diff,634 bytes, patch)
2003-03-05 17:33 UTC, gatto
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description gatto 2003-03-05 17:26:43 UTC
colordiff doesn't add ANSI escape sequences to turn off colors, corrupting any
output (by colorizing it) after it finishes running. also, colorized context
diffs arent supported and would be nice.

Reproducible: Always
Steps to Reproduce:
1. set your command prompt to something without colors
2. run colordiff on two different files

Actual Results:  
command prompt will be same color as the color of the last line of colordiff output.

Expected Results:  
command prompt should be color you have it normally set to.

Creating an attachment that seemingly fixes both problems.
Comment 1 gatto 2003-03-05 17:29:15 UTC
Created attachment 8998 [details, diff]
patch to fix colordiff.pl
Comment 2 gatto 2003-03-05 17:33:15 UTC
Created attachment 8999 [details, diff]
patch to update BUGS file
Comment 3 gatto 2003-03-07 18:32:08 UTC
The author of colordiff incorporated the changes and fixed an error in the
previous patch, and bumped the version, and it is available at: http://colordiff.sourceforge.net/colordiff-1.0.1.tar.gz

So this can be closed. I will submit a new ebuild in a separate bug.
Comment 4 Seth Chandler 2003-03-07 19:21:51 UTC
fixed in cvs...