Skip to content

Commit 5e8725c

Browse files
committed
add libzip submodule to source build it as static lib on all platforms
1 parent 52a4e9e commit 5e8725c

File tree

6 files changed

+15
-18
lines changed

6 files changed

+15
-18
lines changed

.github/workflows/cd.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,6 @@ jobs:
7171
run: |
7272
brew install boost
7373
brew install hdf5
74-
brew install libzip
7574
brew install ninja
7675
7776
brew install llvm
@@ -91,7 +90,6 @@ jobs:
9190
run: |
9291
brew install boost
9392
brew install hdf5
94-
brew install libzip
9593
brew install ninja
9694
9795
brew install llvm
@@ -119,7 +117,6 @@ jobs:
119117
mingw-w64-clang-x86_64-cmake
120118
mingw-w64-clang-x86_64-boost
121119
mingw-w64-clang-x86_64-hdf5
122-
mingw-w64-clang-x86_64-libzip
123120
mingw-w64-clang-x86_64-zlib
124121
mingw-w64-clang-x86_64-libaec
125122
@@ -129,7 +126,6 @@ jobs:
129126
sudo apt-get update
130127
sudo apt-get install -y libboost-all-dev
131128
sudo apt-get install -y libhdf5-dev
132-
sudo apt-get install -y libzip-dev
133129
sudo apt-get install -y ninja-build
134130
135131
gcc --version

.github/workflows/pip.yml

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ jobs:
3737
run: |
3838
brew install boost
3939
brew install hdf5
40-
brew install libzip
4140
brew install ninja
4241
4342
brew install llvm
@@ -57,7 +56,6 @@ jobs:
5756
run: |
5857
brew install boost
5958
brew install hdf5
60-
brew install libzip
6159
brew install ninja
6260
6361
brew install llvm
@@ -85,7 +83,6 @@ jobs:
8583
mingw-w64-clang-x86_64-cmake
8684
mingw-w64-clang-x86_64-boost
8785
mingw-w64-clang-x86_64-hdf5
88-
mingw-w64-clang-x86_64-libzip
8986
mingw-w64-clang-x86_64-zlib
9087
mingw-w64-clang-x86_64-libaec
9188
mingw-w64-clang-x86_64-python-pip-tools
@@ -136,9 +133,9 @@ jobs:
136133

137134
- name: Build and install manylinux for python 3.9
138135
run: |
139-
docker run --rm -v $(pwd):/io quay.io/pypa/manylinux2014_x86_64 \
136+
docker run --rm -v $(pwd):/io quay.io/pypa/manylinux_2_28_x86_64 \
140137
/bin/bash -c \
141-
"yum install -y boost-devel hdf5-devel libzip-devel ninja-build && \
138+
"yum install -y boost-devel hdf5-devel ninja-build && \
142139
/opt/python/cp39-cp39/bin/pip wheel /io/ -w /io/dist && \
143140
auditwheel repair /io/dist/*.whl -w /io/dist/"
144141
echo "keeping only the manylinux wheels, remove those with -linux_x86_64.whl in the name"
@@ -148,9 +145,9 @@ jobs:
148145

149146
- name: Build and install manylinux for python 3.10
150147
run: |
151-
docker run --rm -v $(pwd):/io quay.io/pypa/manylinux2014_x86_64 \
148+
docker run --rm -v $(pwd):/io quay.io/pypa/manylinux_2_28_x86_64 \
152149
/bin/bash -c \
153-
"yum install -y boost-devel hdf5-devel libzip-devel ninja-build && \
150+
"yum install -y boost-devel hdf5-devel ninja-build && \
154151
/opt/python/cp310-cp310/bin/pip wheel /io/ -w /io/dist && \
155152
auditwheel repair /io/dist/*.whl -w /io/dist/"
156153
echo "keeping only the manylinux wheels, remove those with -linux_x86_64.whl in the name"
@@ -160,9 +157,9 @@ jobs:
160157

161158
- name: Build and install manylinux for python 3.11
162159
run: |
163-
docker run --rm -v $(pwd):/io quay.io/pypa/manylinux2014_x86_64 \
160+
docker run --rm -v $(pwd):/io quay.io/pypa/manylinux_2_28_x86_64 \
164161
/bin/bash -c \
165-
"yum install -y boost-devel hdf5-devel libzip-devel ninja-build && \
162+
"yum install -y boost-devel hdf5-devel ninja-build && \
166163
/opt/python/cp311-cp311/bin/pip wheel /io/ -w /io/dist && \
167164
auditwheel repair /io/dist/*.whl -w /io/dist/"
168165
echo "keeping only the manylinux wheels, remove those with -linux_x86_64.whl in the name"
@@ -172,9 +169,9 @@ jobs:
172169

173170
- name: Build and install manylinux for python 3.12
174171
run: |
175-
docker run --rm -v $(pwd):/io quay.io/pypa/manylinux2014_x86_64 \
172+
docker run --rm -v $(pwd):/io quay.io/pypa/manylinux_2_28_x86_64 \
176173
/bin/bash -c \
177-
"yum install -y boost-devel hdf5-devel libzip-devel ninja-build && \
174+
"yum install -y boost-devel hdf5-devel ninja-build && \
178175
/opt/python/cp312-cp312/bin/pip wheel /io/ -w /io/dist && \
179176
auditwheel repair /io/dist/*.whl -w /io/dist/"
180177
echo "keeping only the manylinux wheels, remove those with -linux_x86_64.whl in the name"

.gitmodules

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,6 @@
55
[submodule "libzippp"]
66
path = libzippp
77
url = https://github.com/ctabin/libzippp.git
8+
[submodule "extern/libzip"]
9+
path = extern/libzip
10+
url = https://github.com/nih-at/libzip.git

CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,8 +129,6 @@ include(GetGitRevisionDescription)
129129
git_describe(GIT_DESCRIBE ALLOW_LOOKING_ABOVE_CMAKE_SOURCE_DIR)
130130

131131
include (FindZLIB)
132-
include (FindLIBZIP)
133-
134132

135133
if (NOT OPTION_TARGET_PYTHON_BINDING)
136134
set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
@@ -201,6 +199,8 @@ add_subdirectory(VCellMessaging)
201199

202200
add_subdirectory(VCellZipUtils)
203201

202+
add_subdirectory(extern/libzip)
203+
204204
add_subdirectory(libzippp)
205205

206206
add_subdirectory(ExpressionParser)

bridgeVCellSmoldyn/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ file (GLOB HEADER_FILES *.h)
55
file (GLOB SOURCE_FILES ${CMAKE_CURRENT_SOURCE_DIR}/*.cpp)
66
list(REMOVE_ITEM SOURCE_FILES ${VCELL_SMOL_MAIN})
77

8-
add_library(vcellsmoldynbridge ${HEADER_FILES} ${SOURCE_FILES})
8+
add_library(vcellsmoldynbridge STATIC ${HEADER_FILES} ${SOURCE_FILES})
99
target_link_libraries(vcellsmoldynbridge vcell smoldyn_static)
1010
target_include_directories(vcellsmoldynbridge PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})
1111
target_compile_definitions(vcellsmoldynbridge PUBLIC -DVCELL_HYBRID -DVCELL -D_CRT_SECURE_NO_WARNINGS)

extern/libzip

Submodule libzip added at 6593c06

0 commit comments

Comments
 (0)