Bug 1319166 - Don't automatically enable color in iTerm2; r?glandium
iTerm2 interprets the byte sequence "ESC [ 2 i" as a request to open
a print dialog. This sequence can get emitted when printing colorized
compiler output.
Until we figure out a workaround, disable automatic coloring on iTerm2
so developers aren't annoyed.
MozReview-Commit-ID: 3Be4YgArg4k
--- a/config/config.mk
+++ b/config/config.mk
@@ -356,19 +356,24 @@ color_flags_vars := \
COMPILE_CMMFLAGS \
HOST_CFLAGS \
HOST_CXXFLAGS \
LDFLAGS \
$(NULL)
ifdef MACH_STDOUT_ISATTY
ifdef COLOR_CFLAGS
+# TODO Bug 1319166 - iTerm2 interprets some bytes sequences as a
+# request to show a print dialog. Don't enable color on iTerm2 until
+# a workaround is in place.
+ifneq ($(TERM_PROGRAM),iTerm.app)
$(foreach var,$(color_flags_vars),$(eval $(call colorize_flags,$(var))))
endif
endif
+endif
#
# Name of the binary code directories
#
# Override defaults
SDK_LIB_DIR = $(DIST)/sdk/lib
SDK_BIN_DIR = $(DIST)/sdk/bin