From 867ba9d7fe58858815b9650b69348b86d841e906 Mon Sep 17 00:00:00 2001 From: Oliver Breitwieser Date: Fri, 26 Jul 2019 02:50:23 +0200 Subject: googletest: Fix reversed +pthreads option (#12128) If +pthreads is specified, googletest actually gets build without pthreads support and vice versa. If pthreads is not available, googletest should detect it itself which is why "enabling" pthreads in case of ~pthreads does not seem to cause any error. Change-Id: I29de8c4a23dc3901124b27a1ead3354949b7e65b --- var/spack/repos/builtin/packages/googletest/package.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/var/spack/repos/builtin/packages/googletest/package.py b/var/spack/repos/builtin/packages/googletest/package.py index 4bf0724058..806ecf01ff 100644 --- a/var/spack/repos/builtin/packages/googletest/package.py +++ b/var/spack/repos/builtin/packages/googletest/package.py @@ -38,7 +38,7 @@ class Googletest(CMakePackage): options = [] options.append('-Dgtest_disable_pthreads={0}'.format( - 'ON' if '+pthreads' in spec else 'OFF')) + 'OFF' if '+pthreads' in spec else 'ON')) options.append('-DBUILD_SHARED_LIBS={0}'.format( 'ON' if '+shared' in spec else 'OFF')) return options -- cgit v1.2.3-70-g09d2