solenv/gbuild/platform/com_GCC_defs.mk |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 9ab2703d5c212746c961dcb238a4c3025d3a2587
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Sun Dec 8 15:18:19 2019 +0100
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Sun Dec 8 16:07:31 2019 +0100

    Improve -fdiagnostics-color handling
    
    ...following up on 13b52f50e52d226c935dcb94fac641c59a77f13f "gbuild: color 
gcc
    output if possible".  Unconditional -fdiagnostics-color=auto overrode the
    -fdiagnostics-color=always I specify in CC/CXX to mitigate using the GNU 
Make -O
    option.  Forcing -fdiagnostics-color=always iff gb_COLOR is set should give 
the
    desired results in all cases (colored output to a terminal when calling make
    without -O but using a GCC that defaults to -fdiagnostics-color=never, as 
well
    as colored output to a terminal when calling make with -O).
    
    Change-Id: I629bad4bceb15af3ed43f42038825bd1481b33dd
    Reviewed-on: https://gerrit.libreoffice.org/84710
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sberg...@redhat.com>

diff --git a/solenv/gbuild/platform/com_GCC_defs.mk 
b/solenv/gbuild/platform/com_GCC_defs.mk
index 12e08ecdb95a..13c770baa79f 100644
--- a/solenv/gbuild/platform/com_GCC_defs.mk
+++ b/solenv/gbuild/platform/com_GCC_defs.mk
@@ -67,7 +67,7 @@ gb_CFLAGS_COMMON := \
        -fno-common \
        -pipe \
        -fstack-protector-strong \
-       -fdiagnostics-color=auto \
+       $(if $(gb_COLOR),-fdiagnostics-color=always) \
 
 gb_CXXFLAGS_COMMON := \
        -Wall \
@@ -83,7 +83,7 @@ gb_CXXFLAGS_COMMON := \
        -fno-common \
        -pipe \
        -fstack-protector-strong \
-       -fdiagnostics-color=auto \
+       $(if $(gb_COLOR),-fdiagnostics-color=always) \
 
 gb_CXXFLAGS_DISABLE_WARNINGS = -w
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to