|
From: | GNU bug Tracking System |
Subject: | [debbugs-tracker] bug#22067: closed ([bug-diffutils] bug#20062: bug#20062: [PATCH] diff: add support for --color) |
Date: | Mon, 07 Mar 2016 16:33:02 +0000 |
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 ---
[Prev in Thread] | Current Thread | [Next in Thread] |