Fix exported symbols on Mac

This PR is a follow-up to duckdb#164 change, it fixes the exported symbols for
MacOS and adds listing of symbols on CI runs.

Author: staticlibs

.github/workflows/Java.yml

@@ -72,6 +72,10 @@ jobs:
             make -C /duckdb release
           "
 
+      - name: List Symbols
+        run: |
+          nm -gU ./build/release/libduckdb_java.so_linux_amd64
+
       - name: JDBC Tests EL8
         if: ${{ inputs.skip_tests != 'true' }}
         run: |
@@ -218,6 +222,9 @@ jobs:
             source /opt/rh/gcc-toolset-12/enable
             make -C /duckdb release
           "
+      - name: List Symbols
+        run: |
+          nm -gU ./build/release/libduckdb_java.so_linux_aarch64
 
       - name: JDBC Tests EL8
         shell: bash
@@ -283,6 +290,10 @@ jobs:
           ${{ env.ALPINE_IMAGE }}                  \
           sh -c 'apk add ${{ env.ALPINE_PACKAGES }} && make -C /duckdb release'
 
+      - name: List Symbols
+        run: |
+          nm -gU ./build/release/libduckdb_java.so_linux_amd64
+
       - name: JDBC Tests
         shell: bash
         if: ${{ inputs.skip_tests != 'true' }}
@@ -328,6 +339,10 @@ jobs:
           ${{ env.ALPINE_IMAGE }}                  \
           sh -c 'apk add ${{ env.ALPINE_PACKAGES }} && make -C /duckdb release'
 
+      - name: List Symbols
+        run: |
+          nm -gU ./build/release/libduckdb_java.so_linux_aarch64
+
       # Test runs are failing because of linux_arm64_musl extensions missing
       - name: JDBC Tests
         shell: bash
@@ -370,6 +385,11 @@ jobs:
         shell: bash
         run: make release
 
+      - name: List Symbols
+        shell: bash
+        run: |
+          dumpbin /exports ./build/Release/libduckdb_java.so_windows_amd64
+
       - name: Java Tests
         if: ${{ inputs.skip_tests != 'true' }}
         shell: bash
@@ -404,6 +424,10 @@ jobs:
         shell: bash
         run: make release
 
+      - name: List Symbols
+        run: |
+          nm -gU ./build/release/libduckdb_java.so_osx_universal
+
       - name: Java Tests
         if: ${{ inputs.skip_tests != 'true' }}
         shell: bash

CMakeLists.txt

@@ -638,7 +638,7 @@ if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
     -Bsymbolic-functions
     -fvisibility=hidden 
     -Wl,--version-script=${CMAKE_CURRENT_LIST_DIR}/duckdb_java.map)
-elseif(CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
+elseif(CMAKE_CXX_COMPILER_ID STREQUAL "AppleClang")
   target_link_options(duckdb_java PRIVATE
     -fvisibility=hidden
     -Wl,-exported_symbols_list,${CMAKE_CURRENT_LIST_DIR}/duckdb_java.exp)

CMakeLists.txt.in

@@ -164,7 +164,7 @@ if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
     -Bsymbolic-functions
     -fvisibility=hidden 
     -Wl,--version-script=${CMAKE_CURRENT_LIST_DIR}/duckdb_java.map)
-elseif(CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
+elseif(CMAKE_CXX_COMPILER_ID STREQUAL "AppleClang")
   target_link_options(duckdb_java PRIVATE
     -fvisibility=hidden
     -Wl,-exported_symbols_list,${CMAKE_CURRENT_LIST_DIR}/duckdb_java.exp)