Avoid setting -Werror=unused-but-set-variable on Windows.
authorGintautas Miliauskas <gintautas.miliauskas@gmail.com>
Thu, 30 Oct 2014 04:13:51 +0000 (23:13 -0500)
committerAustin Seipp <austin@well-typed.com>
Thu, 30 Oct 2014 04:13:51 +0000 (23:13 -0500)
Summary:
The option is not needed (it was only intended to override Debian's default)
and causes an error if the host ghc's mingw is too old (which the script
does not detect).

Fixes T9727

Reviewers: austin

Reviewed By: austin

Subscribers: thomie, carter, simonmar

Differential Revision: https://phabricator.haskell.org/D373

GHC Trac Issues: #9727

mk/validate-settings.mk

index b05b289..52aa648 100644 (file)
@@ -15,7 +15,10 @@ ifneq "$(GccIsClang)" "YES"
 # Debian doesn't turn -Werror=unused-but-set-variable on by default, so
 # we turn it on explicitly for consistency with other users
 ifeq "$(GccLT46)" "NO"
+# Never set the flag on Windows as the host gcc may be too old.
+ifneq "$(HostOS_CPP)" "mingw32"
 SRC_CC_WARNING_OPTS += -Werror=unused-but-set-variable
+endif
 # gcc 4.6 gives 3 warning for giveCapabilityToTask not being inlined
 SRC_CC_WARNING_OPTS += -Wno-error=inline
 endif