From 257d23581cda1b136912dbcfe3c8cee2ba635aab Mon Sep 17 00:00:00 2001
From: Axel Huebl <axel.huebl@plasma.ninja>
Date: Mon, 8 Jan 2018 19:42:51 +0100
Subject: [PATCH] Cleanup: Remove Dbl Brackets (#6848)

Remove unnecessary double brackets in ADIOS2 and gearshifft packages.
---
 .../repos/builtin/packages/adios2/package.py  | 36 +++++++++----------
 .../builtin/packages/gearshifft/package.py    | 16 ++++-----
 2 files changed, 26 insertions(+), 26 deletions(-)

diff --git a/var/spack/repos/builtin/packages/adios2/package.py b/var/spack/repos/builtin/packages/adios2/package.py
index 8752a18271..935313e6d9 100644
--- a/var/spack/repos/builtin/packages/adios2/package.py
+++ b/var/spack/repos/builtin/packages/adios2/package.py
@@ -97,24 +97,24 @@ def cmake_args(self):
         spec = self.spec
 
         args = [
-            '-DADIOS2_BUILD_SHARED_LIBS:BOOL={0}'.format((
-                'ON' if '+shared' in spec else 'OFF')),
+            '-DADIOS2_BUILD_SHARED_LIBS:BOOL={0}'.format(
+                'ON' if '+shared' in spec else 'OFF'),
             '-DADIOS2_BUILD_TESTING=OFF',
-            '-DADIOS2_USE_MPI={0}'.format((
-                'ON' if '+mpi' in spec else 'OFF')),
-            '-DADIOS2_USE_BZip2={0}'.format((
-                'ON' if '+bzip2' in spec else 'OFF')),
-            '-DADIOS2_USE_ZFP={0}'.format((
-                'ON' if '+zfp' in spec else 'OFF')),
-            '-DADIOS2_USE_DataMan={0}'.format((
-                'ON' if '+dataman' in spec else 'OFF')),
-            '-DADIOS2_USE_ZeroMQ={0}'.format((
-                'ON' if '+dataman' in spec else 'OFF')),
-            '-DADIOS2_USE_HDF5={0}'.format((
-                'ON' if '+hdf5' in spec else 'OFF')),
-            '-DADIOS2_USE_ADIOS1={0}'.format((
-                'ON' if '+adios1' in spec else 'OFF')),
-            '-DADIOS2_USE_Python={0}'.format((
-                'ON' if '+python' in spec else 'OFF'))
+            '-DADIOS2_USE_MPI={0}'.format(
+                'ON' if '+mpi' in spec else 'OFF'),
+            '-DADIOS2_USE_BZip2={0}'.format(
+                'ON' if '+bzip2' in spec else 'OFF'),
+            '-DADIOS2_USE_ZFP={0}'.format(
+                'ON' if '+zfp' in spec else 'OFF'),
+            '-DADIOS2_USE_DataMan={0}'.format(
+                'ON' if '+dataman' in spec else 'OFF'),
+            '-DADIOS2_USE_ZeroMQ={0}'.format(
+                'ON' if '+dataman' in spec else 'OFF'),
+            '-DADIOS2_USE_HDF5={0}'.format(
+                'ON' if '+hdf5' in spec else 'OFF'),
+            '-DADIOS2_USE_ADIOS1={0}'.format(
+                'ON' if '+adios1' in spec else 'OFF'),
+            '-DADIOS2_USE_Python={0}'.format(
+                'ON' if '+python' in spec else 'OFF')
         ]
         return args
diff --git a/var/spack/repos/builtin/packages/gearshifft/package.py b/var/spack/repos/builtin/packages/gearshifft/package.py
index c1a74ce34d..4ea06f6ff4 100644
--- a/var/spack/repos/builtin/packages/gearshifft/package.py
+++ b/var/spack/repos/builtin/packages/gearshifft/package.py
@@ -63,13 +63,13 @@ def cmake_args(self):
             '-DGEARSHIFFT_CLFFT:BOOL=OFF'
         ]
         args.extend([
-            '-DGEARSHIFFT_FFTW:BOOL={0}'.format((
-                'ON' if '+fftw' in spec else 'OFF')),
-            '-DGEARSHIFFT_FFTW_OPENMP:BOOL={0}'.format((
-                'ON' if '+openmp' in spec else 'OFF')),
-            '-DGEARSHIFFT_CUFFT:BOOL={0}'.format((
-                'ON' if '+cufft' in spec else 'OFF')),
-            '-DGEARSHIFFT_CLFFT:BOOL={0}'.format((
-                'ON' if '+clfft' in spec else 'OFF'))
+            '-DGEARSHIFFT_FFTW:BOOL={0}'.format(
+                'ON' if '+fftw' in spec else 'OFF'),
+            '-DGEARSHIFFT_FFTW_OPENMP:BOOL={0}'.format(
+                'ON' if '+openmp' in spec else 'OFF'),
+            '-DGEARSHIFFT_CUFFT:BOOL={0}'.format(
+                'ON' if '+cufft' in spec else 'OFF'),
+            '-DGEARSHIFFT_CLFFT:BOOL={0}'.format(
+                'ON' if '+clfft' in spec else 'OFF')
         ])
         return args
-- 
GitLab