[Mac] xcodebuild color output is suppressed when using make
authormitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 11 Jan 2014 19:48:08 +0000 (19:48 +0000)
committermitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 11 Jan 2014 19:48:08 +0000 (19:48 +0000)
https://bugs.webkit.org/show_bug.cgi?id=126815

Reviewed by Anders Carlsson.

WHen using make, xcodebuild’s output is piped through a filter, which causes it not to
format its output with color and emphasis.

* Makefile.shared: Pass COLOR_DIAGNOSTICS=YES to xcodebuild to force color output on. Due to
xcodebuild limitations, this restores some of the formatting, but not all of it.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@161772 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
Makefile.shared

index 9f07f868810df357ce47ffe50cb0d869896e4a0f..a53f0e8b1d6686689ff7f700dc8d3854645b3ff1 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2014-01-11  Dan Bernstein  <mitz@apple.com>
+
+        [Mac] xcodebuild color output is suppressed when using make
+        https://bugs.webkit.org/show_bug.cgi?id=126815
+
+        Reviewed by Anders Carlsson.
+
+        WHen using make, xcodebuild’s output is piped through a filter, which causes it not to
+        format its output with color and emphasis.
+
+        * Makefile.shared: Pass COLOR_DIAGNOSTICS=YES to xcodebuild to force color output on. Due to
+        xcodebuild limitations, this restores some of the formatting, but not all of it.
+
 2014-01-10  Benjamin Poulain  <bpoulain@apple.com>
 
         Remove the BlackBerry port from trunk
index ee723d830dea51d1ce9e2d1ad8c985eeca2d4f3f..cfb05af87985fc5908c034cf50c5f46ee5435c51 100644 (file)
@@ -1,6 +1,6 @@
 SCRIPTS_PATH ?= ../Tools/Scripts
 
-XCODE_OPTIONS = `perl -I$(SCRIPTS_PATH) -Mwebkitdirs -e 'print XcodeOptionString()' -- $(BUILD_WEBKIT_OPTIONS)` $(ARGS)
+XCODE_OPTIONS = `perl -I$(SCRIPTS_PATH) -Mwebkitdirs -e 'print XcodeOptionString()' -- $(BUILD_WEBKIT_OPTIONS)` COLOR_DIAGNOSTICS=YES $(ARGS)
 
 ifneq (,$(SDKROOT))
        XCODE_OPTIONS += SDKROOT=$(SDKROOT)