Don't use -Werror in Stack
authorGreg Hurrell <greg@hurrell.net>
Thu, 16 Jun 2016 14:41:55 +0000 (07:41 -0700)
committerGreg Hurrell <greg@hurrell.net>
Thu, 16 Jun 2016 14:41:55 +0000 (07:41 -0700)
I intend this to mostly be a local feedback tool, so I *think* that defaulting
this to off is right call. I fixed the specific build failure cited below anyway
in c9efe9d7b010adbd1, but I think it's ok to turn off `-Werror` in Stack because
other "real" errors will still be errors; these are just warnings.

I've added explicit `-Werror` to all my wrapper scripts except `bin/repl`, which
is not "build-like" and I want to be lenient. (I actually meant to make it more
lenient in the future by deferring type errors with `-fdefer-type-errors`.

See:

- https://github.com/wincent/docvim/issues/24
- https://travis-ci.org/wincent/docvim/jobs/138094062
- https://www.reddit.com/r/haskell/comments/4np0l2/a_ghci_annoyance_wondering_if_anyones_else_also/

bin/accept
bin/build
bin/golden
bin/haddock
bin/lint
bin/tasty
bin/test
stack.yaml

index bc4567a18c63a32c1e166006dfdcf2d73c46b9f2..f21b372bb6d92b6c3f374e2e745f826dc5910580 100755 (executable)
@@ -1,3 +1,3 @@
 #!/bin/sh
 
-stack test :tasty --test-arguments=--accept
+stack test :tasty --test-arguments=--accept --ghc-options -Werror
index 0b8610f589b009cd65e9f79f75651660d2c93ec7..a29aa453fa562a93dcd7fc2a23586fe4c7b99384 100755 (executable)
--- a/bin/build
+++ b/bin/build
@@ -1,3 +1,3 @@
 #!/bin/sh
 
-stack build --file-watch
+stack build --file-watch --ghc-options -Werror
index 081971674b2b4b5ec0b4577a296103b8ba3a54fa..06270237b37a4e4080b59ce057a9f8632826023c 100755 (executable)
@@ -1,4 +1,4 @@
 #!/bin/sh
 
 # Cheating here: for now we just run all the tests, not just the golden ones.
-stack test :tasty
+stack test :tasty --ghc-options -Werror
index 59305e8d997e48d50e700a636f3bc9415d7c7581..429f52756649275dc20cec1bd25c54ecd5d67624 100755 (executable)
@@ -3,4 +3,4 @@
 # BUG: grep will exit with non-zero exit code if haddock target is already
 # up-to-date (no output)
 # TODO: capture output and echo it only on error
-stack haddock | grep docvim/index.html
+stack haddock --ghc-options -Werror
index 9dfda009f3c78274383d9bdd4cae5e21a70747d5..a878991710aae1ba9a4a8b5b03da1b0eab458433 100755 (executable)
--- a/bin/lint
+++ b/bin/lint
@@ -1,3 +1,3 @@
 #!/bin/sh
 
-stack test :hlint
+stack test :hlint --ghc-options -Werror
index d200fe5d22ee82bcc0a2e0fb071fe9dbb662eae2..c8f4cfb6bd3846424ecd508897d5df0a9e69c68b 100755 (executable)
--- a/bin/tasty
+++ b/bin/tasty
@@ -1,3 +1,3 @@
 #!/bin/sh
 
-stack test :tasty
+stack test :tasty --ghc-options -Werror
index b9385f9d6db0e8ae6e465a4c3f2b7258b72012d4..7616c874eccb7de406602ddbae47a373a3a1271c 100755 (executable)
--- a/bin/test
+++ b/bin/test
@@ -1,3 +1,3 @@
 #!/bin/sh
 
-stack test
+stack test --ghc-options -Werror
index ca469ef10b334fc940f537dda8f35d3b068175ff..b1510f8b68a9c26f8c35aea186f5518e6e203927 100644 (file)
@@ -16,8 +16,8 @@ flags: {}
 # Extra package databases containing global packages
 extra-package-dbs: []
 
-ghc-options:
-    "*": -Werror
+ghc-options:
+    "*": -Werror
 
 # Control whether we use the GHC we find on the path
 # system-ghc: true