]> sigrok.org Git - pulseview.git/commitdiff
Disable unit tests by default
authorSoeren Apel <redacted>
Fri, 8 Jun 2018 07:09:09 +0000 (09:09 +0200)
committerUwe Hermann <redacted>
Fri, 15 Jun 2018 19:41:42 +0000 (21:41 +0200)
The rationale for this change is that except for developers, pretty
much everyone who builds from source does so because they want to
use the latest changes. However, only developers care about the results
of the unit tests, so this doubles the build time for users without
any real benefit.

Even worse, the unit tests are known to fail on several hosts for
reasons that are within the realm of the unit tests themselves, not
because of broken code. This confuses users and even prevents them
from installing PV for no good reason.

Since the unit tests do still run on Jenkins, there's little to no
coverage gap in my opinion.

CMakeLists.txt

index 252f9024d21c4bf089cb74afe7c1cfa14ecff829..e7d0bf5f1f3a1f9ca19eaa7e4ecc61ae66913aed 100644 (file)
@@ -44,7 +44,7 @@ option(DISABLE_WERROR "Build without -Werror" FALSE)
 option(ENABLE_SIGNALS "Build with UNIX signals" TRUE)
 option(ENABLE_STACKTRACE "Enable stack trace when crashing" FALSE)
 option(ENABLE_DECODE "Build with libsigrokdecode" TRUE)
-option(ENABLE_TESTS "Enable unit tests" TRUE)
+option(ENABLE_TESTS "Enable unit tests" FALSE)
 option(STATIC_PKGDEPS_LIBS "Statically link to (pkg-config) libraries" FALSE)
 
 if(WIN32)