Re: [ccache] Support for color diagnostics

2014-08-07 Thread Joel Rosdahl
Attached is an updated patch that passes also all the tests. Applied on master, thanks! (Sorry for the massive delay.) -- Joel On 26 June 2014 18:44, Lubos Lunak l.lu...@centrum.cz wrote: On Saturday 14 of June 2014, Joel Rosdahl wrote: Hi Lubos, Sorry about the ping delay. I've

Re: [ccache] Support for color diagnostics

2014-06-26 Thread Paul Smith
On Thu, 2014-06-26 at 18:44 +0200, Lubos Lunak wrote: Caveats: - Compiles with and without colors are considered different from each other (so they are duplicated). This doesn't seem ideal, does it? If I'm understanding this correctly won't this cause rebuilds based on whether you're

Re: [ccache] Support for color diagnostics

2014-06-26 Thread Lubos Lunak
On Thursday 26 of June 2014, Paul Smith wrote: On Thu, 2014-06-26 at 18:44 +0200, Lubos Lunak wrote: Caveats: - Compiles with and without colors are considered different from each other (so they are duplicated). This doesn't seem ideal, does it? No, it doesn't seem ideal. It doesn't

Re: [ccache] Support for color diagnostics

2014-06-14 Thread Joel Rosdahl
it will work on Windows as well. -- Joel On 1 June 2014 10:17, Lubos Lunak l.lu...@centrum.cz wrote: On Friday 29 of November 2013, Lubos Lunak wrote: Hello, the attached patch adds ccache support for compiler color diagnostics (also reported by somebody as #10075). Ping? Any official

Re: [ccache] Support for color diagnostics

2014-06-01 Thread Lubos Lunak
On Friday 29 of November 2013, Lubos Lunak wrote: Hello, the attached patch adds ccache support for compiler color diagnostics (also reported by somebody as #10075). Ping? Any official comments on the patch? I've been using the patch for half a year by now without problems. Clang

Re: [ccache] Support for color diagnostics

2014-02-18 Thread interfaSys Sàrl
, November 29, 2013 12:39:25 PM UTC+1, Lubos Lunak wrote: Hello, the attached patch adds ccache support for compiler color diagnostics (also reported by somebody as #10075). Clang automatically uses colors for output automatically if used in terminal. Ccache's redirecting to a file

Re: [ccache] Support for color diagnostics

2014-02-11 Thread Lubos Lunak
Ping? On Friday 29 of November 2013, Lubos Lunak wrote: Hello, the attached patch adds ccache support for compiler color diagnostics (also reported by somebody as #10075). Clang automatically uses colors for output automatically if used in terminal. Ccache's redirecting to a file

Re: [ccache] Support for color diagnostics

2014-02-11 Thread Ryan Hill
On Fri, 29 Nov 2013 12:39:25 +0100 Lubos Lunak l.lu...@centrum.cz wrote: Clang automatically uses colors for output automatically if used in terminal. Ccache's redirecting to a file disables this. GCC 4.8 has got a similar support, except that it apparently requires also $GCC_COLORS or an

Re: [ccache] Support for color diagnostics

2013-12-12 Thread Eitan Adler
On Fri, Nov 29, 2013 at 6:39 AM, Lubos Lunak l.lu...@centrum.cz wrote: 100% reliable anyway), the code merely guesses from the binary name. If the compiler used will be e.g. the 'cc' symlink, there'll be no colors. No big deal. Most users should be calling their compiler via the name 'cc' and

Re: [ccache] Support for color diagnostics

2013-12-04 Thread Lubos Lunak
support for compiler color diagnostics (also reported by somebody as #10075). ... I think you didn't understand GCC documentation correctly. Actually I think I did. I've now tried with a chroot (openSUSE build service really is a useful tool) and it pretty much matches my

Re: [ccache] Support for color diagnostics

2013-11-29 Thread Lubos Lunak
On Friday 29 of November 2013, Lubos Lunak wrote: Hello, the attached patch adds ccache support for compiler color diagnostics (also reported by somebody as #10075). ... Caveats: I forgot one: - The function color_output_possible() may look simplistic, but as far as I can tell it works

Re: [ccache] Support for color diagnostics

2013-11-29 Thread Loïc Yhuel
Le 29/11/2013 14:08, Lubos Lunak a écrit : On Friday 29 of November 2013, Lubos Lunak wrote: Hello, the attached patch adds ccache support for compiler color diagnostics (also reported by somebody as #10075). ... Caveats: I forgot one: - The function color_output_possible() may look