Skip to content

Commit

Permalink
Build with Zoltan on MinGW CI.
Browse files Browse the repository at this point in the history
Downgrade incompatible-pointer-types to a warning to work around build
error of Zoltan with newer GCC.

Also link to ParMetis libraries when linking to Zoltan with enabled
ParMetis. (It is force-enabled in the bundled version of Zoltan.)
  • Loading branch information
mmuetzel committed Aug 2, 2024
1 parent 8f9dd71 commit 5929964
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/build-windows-mingw.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@ jobs:
- name: checkout repository
uses: actions/checkout@v4
with:
submodules: recursive

- name: install dependencies
uses: msys2/setup-msys2@v2
Expand All @@ -57,6 +59,8 @@ jobs:
mpi: msmpi

- name: configure
# Downgrade incompatible-pointer-types to a warning to work around
# build error of Zoltan with newer GCC.
run: |
mkdir ${GITHUB_WORKSPACE}/build
cd ${GITHUB_WORKSPACE}/build
Expand All @@ -68,7 +72,10 @@ jobs:
-DBLA_VENDOR="OpenBLAS" \
-DWITH_OpenMP=ON \
-DWITH_LUA=ON \
-DWITH_Zoltan=OFF \
-DWITH_Zoltan=ON \
-DCMAKE_C_FLAGS="-Wno-error=incompatible-pointer-types" \
-DParMetis_LIBRARIES="-lparmetis -lmetis" \
-DParMetis_INCLUDE_DIR="${MSYSTEM_PREFIX}/include" \
-DWITH_Mumps=OFF \
-DWITH_ELMERGUI=ON \
-DCREATE_PKGCONFIG_FILE=ON \
Expand Down
3 changes: 3 additions & 0 deletions fem/src/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -171,6 +171,9 @@ ENDIF()

IF(HAVE_ZOLTAN)
TARGET_LINK_LIBRARIES(elmersolver ${ZOLTAN_LIBRARY})
IF(Zoltan_ENABLE_ParMETIS)
TARGET_LINK_LIBRARIES(elmersolver ${ParMetis_LIBRARIES})
ENDIF()
ADD_DEPENDENCIES(elmersolver ${ZOLTAN_LIBRARY})
# SET(ELMERSOLVER_RPATH_STRING_MOD "${ELMERSOLVER_RPATH_STRING_MOD}/:${ZOLTAN_LIBDIR}")
# SET(ELMERLIB_RPATH_STRING "${ELMERLIB_RPATH_STRING}/:${ZOLTAN_LIBDIR}")
Expand Down

0 comments on commit 5929964

Please sign in to comment.