Build optimized wheels by default. Don't use -O3 because it fails build

aerospike · Sep 22, 2023 · b34ad9f · b34ad9f
1 parent cc25b5d
commit b34ad9f
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 13 deletions.
diff --git a/.github/workflows/build-wheels.yml b/.github/workflows/build-wheels.yml
@@ -179,10 +179,6 @@ jobs:
       run: echo "CFLAGS=-O0 -g" >> $GITHUB_ENV
       if: ${{ inputs.optimized-or-debug == 'debug' }}
 
-    - name: Set optimized flags
-      run: echo "CFLAGS=-O1" >> $GITHUB_ENV
-      if: ${{ inputs.optimized-or-debug == 'optimized' }}
-
     - name: Build wheel
       uses: pypa/cibuildwheel@v2.11.2
       env:
@@ -248,10 +244,6 @@ jobs:
       run: echo "CFLAGS=-O0 -g" >> $GITHUB_ENV
       if: ${{ inputs.optimized-or-debug == 'debug' }}
 
-    - name: Set optimized flags
-      run: echo "CFLAGS=-O1" >> $GITHUB_ENV
-      if: ${{ inputs.optimized-or-debug == 'optimized' }}
-
     - name: Build wheel
       uses: pypa/cibuildwheel@v2.11.2
       env:
@@ -286,10 +278,6 @@ jobs:
       run: echo "CFLAGS=-O0 -g" >> $GITHUB_ENV
       if: ${{ inputs.optimized-or-debug == 'debug' }}
 
-    - name: Set optimized flags
-      run: echo "CFLAGS=-O1" >> $GITHUB_ENV
-      if: ${{ inputs.optimized-or-debug == 'optimized' }}
-
     - name: Build wheel
       uses: pypa/cibuildwheel@v2.11.2
       env:

diff --git a/setup.py b/setup.py
@@ -58,7 +58,7 @@
     ['/usr/local/opt/openssl/include'] + \
     ['aerospike-client-c/modules/common/src/include']
 extra_compile_args = [
-    '-std=gnu99', '-Wall', '-fPIC', '-DDEBUG',
+    '-std=gnu99', '-Wall', '-fPIC', '-DDEBUG', '-O1',
     '-fno-common', '-fno-strict-aliasing', '-Wno-strict-prototypes',
     '-D_FILE_OFFSET_BITS=64', '-D_REENTRANT',
     '-DMARCH_' + machine,