Document BUILD_WERROR
authorJuan Ramos <juan@lunarg.com>
Thu, 28 Dec 2023 22:16:29 +0000 (15:16 -0700)
committerCharles Giessen <46324611+charles-lunarg@users.noreply.github.com>
Fri, 29 Dec 2023 17:10:20 +0000 (11:10 -0600)
BUILD.md

index a3211379a15b9ec65dbcbbfcd4d936b7c36bce1b..6915392204626a351825804df2872a9ea3e1f23f 100644 (file)
--- a/BUILD.md
+++ b/BUILD.md
@@ -127,6 +127,16 @@ cmake ... -D UPDATE_DEPS=ON -D BUILD_TESTS=ON ...
 ```
 This will ensure googletest and detours is downloaded and the appropriate version is used.
 
+### Warnings as errors off by default!
+
+By default `BUILD_WERROR` is `OFF`. The idiom for open source projects is to NOT enable warnings as errors.
+
+System/language package managers have to build on multiple different platforms and compilers.
+
+By defaulting to `ON` we cause issues for package managers since there is no standard way to disable warnings.
+
+Add `-D BUILD_WERROR=ON` to your workflow
+
 ### Build and Install Directory Locations
 
 A common convention is to place the `build` directory in the top directory of