256 Color Patch

Jonas Kramer

unread,

Oct 29, 2008, 7:10:05 AM10/29/08

to ack-...@googlegroups.com

Ahoy,

I've written a small patch that checks if the environment variables ACK_COLOR_{MATCH,FILENAME} do contain color codes (like "0;35" or "38;5;208"). If so, it doesn't use Term::ANSIColor for coloring the string but colors the string itself. This way ack can support 256 colors (for terminals that support it).

Regards,
Jonas

ack-env-color.diff

Andy Lester

unread,

Oct 29, 2008, 10:16:59 AM10/29/08

to ack-...@googlegroups.com

Sounds good to me, but can you please resubmit the patch with some
addition to the docs for ACK_COLOR_{MATCH,FILENAME} that explain these
two options?

--
Andy Lester => an...@petdance.com => www.petdance.com => AIM:petdance

Jonas Kramer

unread,

Oct 30, 2008, 7:54:01 AM10/30/08

to ack-...@googlegroups.com

I attached another patch that also adds information about the new
functionality to the manuals/PODs of ack and Ack.pm.