Skip to content

Commit 8b87faa

Browse files
Still trying to get libzip found (+ name fix)
1 parent f3e5bce commit 8b87faa

File tree

2 files changed

+45
-7
lines changed

2 files changed

+45
-7
lines changed

.github/workflows/build_wheels.yml

Lines changed: 41 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,7 @@ jobs:
8282
mingw-w64-clang-x86_64-boost \
8383
mingw-w64-clang-x86_64-hdf5 \
8484
mingw-w64-clang-x86_64-zlib \
85+
mingw-w64-clang-x86_64-libzip \
8586
mingw-w64-clang-x86_64-libaec"
8687
echo "C:\tools\msys64\mingw64\bin" | Out-File -Append -Encoding ascii $env:GITHUB_PATH
8788
echo "C:\tools\msys64\clang64\bin" | Out-File -Append -Encoding ascii $env:GITHUB_PATH
@@ -110,6 +111,7 @@ jobs:
110111
CIBW_BUILD: ${{ matrix.python_version }}
111112
CIBW_ENVIRONMENT: >
112113
FC=/usr/local/bin/gfortran
114+
MACOSX_DEPLOYMENT_TARGET="10.13"
113115
# with:
114116
# package-dir: .
115117
# output-dir: wheelhouse
@@ -133,7 +135,7 @@ jobs:
133135
if: matrix.os == 'ubuntu-24.04-arm'
134136
env:
135137
CIBW_BUILD: ${{ matrix.python_version }}
136-
CIBW_BEFORE_BUILD: "pip install h5py && export LIBZIP_INCLUDE_DIR=/usr/local/include && export LIBZIP_LIBRARY=/usr/local/lib/libzip.so"
138+
CIBW_BEFORE_BUILD: "apt install cmake ninja -y pip install h5py"
137139
# with:
138140
# package-dir: .
139141
# output-dir: wheelhouse
@@ -144,15 +146,50 @@ jobs:
144146
if: matrix.os == 'ubuntu-latest'
145147
env:
146148
CIBW_BUILD: ${{ matrix.python_version }}
147-
CIBW_BEFORE_BUILD: "pip install h5py && export LIBZIP_INCLUDE_DIR=/usr/local/include && export LIBZIP_LIBRARY=/usr/local/lib/libzip.so"
149+
CIBW_BEFORE_BUILD: "apt install cmake ninja -y && pip install h5py \
150+
"
148151
# with:
149152
# package-dir: .
150153
# output-dir: wheelhouse
151154
# config-file: "{package}/pyproject.toml"
155+
# && mkdir /libzip_source \
156+
# &&
157+
# && export LIBZIP_INCLUDE_DIR=/usr/local/include \
158+
# && export LIBZIP_LIBRARY=/usr/local/lib/libzip.so
152159

153-
- uses: actions/upload-artifact@v4
160+
- name: Upload Artifact (Python 3.9)
161+
uses: actions/upload-artifact@v4
162+
if: matrix.python_version == 'cp39-*'
154163
with:
155-
name: pyvcell-fv-${{ matrix.os }}-${{ matrix.python_version }}-cibw-wheels
164+
name: pyvcell-fv-${{ matrix.os }}-cp39-cibw-wheels
165+
path: ./wheelhouse/*.whl
166+
167+
- name: Upload Artifact (Python 3.10)
168+
uses: actions/upload-artifact@v4
169+
if: matrix.python_version == 'cp310-*'
170+
with:
171+
name: pyvcell-fv-${{ matrix.os }}-cp310-cibw-wheels
172+
path: ./wheelhouse/*.whl
173+
174+
- name: Upload Artifact (Python 3.11)
175+
uses: actions/upload-artifact@v4
176+
if: matrix.python_version == 'cp311-*'
177+
with:
178+
name: pyvcell-fv-${{ matrix.os }}-cp311-cibw-wheels
179+
path: ./wheelhouse/*.whl
180+
181+
- name: Upload Artifact (Python 3.12)
182+
uses: actions/upload-artifact@v4
183+
if: matrix.python_version == 'cp312-*'
184+
with:
185+
name: pyvcell-fv-${{ matrix.os }}-cp312-cibw-wheels
186+
path: ./wheelhouse/*.whl
187+
188+
- name: Upload Artifact (Python 3.13)
189+
uses: actions/upload-artifact@v4
190+
if: matrix.python_version == 'cp313-*'
191+
with:
192+
name: pyvcell-fv-${{ matrix.os }}-cp313-cibw-wheels
156193
path: ./wheelhouse/*.whl
157194

158195
- name: Setup tmate session on failure

CMakeLists.txt

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -132,14 +132,15 @@ set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_SOURCE_DIR}/libzippp/cmake/"
132132
include(GetGitRevisionDescription)
133133
git_describe(GIT_DESCRIBE ALLOW_LOOKING_ABOVE_CMAKE_SOURCE_DIR)
134134

135-
include (FindZLIB)
136-
include (FindLIBZIP)
137-
138135
# help CMake find libzip
139136
if (APPLE)
140137
execute_process(COMMAND brew --prefix libzip OUTPUT_VARIABLE LIBZIP_PREFIX OUTPUT_STRIP_TRAILING_WHITESPACE)
141138
set(CMAKE_PREFIX_PATH ${CMAKE_PREFIX_PATH} ${LIBZIP_PREFIX})
142139
endif()
140+
141+
include (FindZLIB)
142+
include (FindLIBZIP)
143+
143144
find_package(LIBZIP 1.10 REQUIRED)
144145

145146

0 commit comments

Comments
 (0)