[debbugs-tracker] bug#22067: closed ([bug-diffutils] bug#20062: bug#20062: [PATCH] diff: add support for --color)

[GNU bug Tracking System]

Your message dated Mon, 7 Mar 2016 08:31:42 -0800
with message-id <address@hidden>
and subject line Re: [bug-diffutils] bug#22067: bug#22067: bug#20062: 
bug#20062: [PATCH] diff: add support for --color
has caused the debbugs.gnu.org bug report #22067,
regarding [bug-diffutils] bug#20062: bug#20062: [PATCH] diff: add support for 
--color
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
22067: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=22067
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> [bug-diffutils] bug#20062: bug#20062: [PATCH] diff: add support for --color
>
>
>
>
> Date: 
>
> Tue, 1 Dec 2015 10:57:08 +0100
>
>
>
>
> User-agent: 
>
> Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0
>
>
>
>
> Jim Meyering wrote:
>
> > Good! Worthwhile after all. Thank you.
> > I have just pushed those commits.
>
> Trying this color-patch on Windows, there are a few issues:
>
> 1) ANSI-sequences are no good on Windows.
> 2) Some signals are not in MSVC, MinGW nor in Gnulib; i.e. SIGTSTP,
>    SIGSTOP etc.
> 3) I'm getting a stack-overflow in handling a SIGINT.
>    AFAICS there is a infinite recursion in process_signals().
>    This function calls 'set_color_context (RESET_CONTEXT)'. But
>    'set_color_context()' again calls 'process_signals()'. How can
>    that *not* stack-fault on any platform?
>
> 1+2) I've patched here to get colors. Although hard-coded; ignoring
> the '--palette' option etc.
>
> -- 
> --gv
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: [bug-diffutils] bug#22067: bug#22067: bug#20062: bug#20062: [PATCH] diff: add support for --color
>
>
>
>
> Date: 
>
> Mon, 7 Mar 2016 08:31:42 -0800
>
>
>
>
> pushed.
>
>
> --- End Message ---