Fix DuckDB build for macOS ARM and 32-bit (#3115)

kuzudb · Mar 22, 2024 · 6d39076 · 6d39076
1 parent 3f817f2
commit 6d39076
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/.github/workflows/build-extensions.yml b/.github/workflows/build-extensions.yml
@@ -108,7 +108,7 @@ jobs:
 
       - name: Install OpenSSL
         run: |
-          brew install openssl@3
+          brew install openssl@3 duckdb
           OPENSSL_ROOT=$(readlink -f /opt/homebrew/Cellar/openssl@3/*/)
           echo "OPENSSL_ROOT_DIR=$OPENSSL_ROOT" >> $GITHUB_ENV
 

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -46,6 +46,7 @@ if(CMAKE_SIZEOF_VOID_P EQUAL 8)
 elseif(CMAKE_SIZEOF_VOID_P EQUAL 4)
     message(STATUS "32-bit architecture detected")
     add_compile_definitions(__32BIT__)
+    set(__32BIT__ TRUE)
 endif()
 
 if(NOT CMAKE_BUILD_TYPE)

diff --git a/extension/CMakeLists.txt b/extension/CMakeLists.txt
@@ -3,7 +3,11 @@ if("httpfs" IN_LIST BUILD_EXTENSIONS)
 endif()
 
 if ("duckdb_scanner" IN_LIST BUILD_EXTENSIONS)
-    add_subdirectory(duckdb_scanner)
+    if(NOT __32BIT__)
+        # DuckDB does not officially support 32-bit builds, so we disable the 
+        # extension for 32-bit builds
+        add_subdirectory(duckdb_scanner)
+    endif()
 endif()
 
 if (${BUILD_EXTENSION_TESTS})