spatial-model-editor / sme_deps_common

various libraries statically compiled for linux / macOS / windows
MIT License
2 stars 0 forks source link

add libcombine / zipper #21

Closed fbergmann closed 1 year ago

fbergmann commented 1 year ago

I saw, that in the last commit you removed the zlib library. In order to be able to open OMEX archives with SME, i was hoping to add:

to the common deps. It would however also require zlib. So i was wondering why it was removed it in the first place.

lkeegan commented 1 year ago

zlib is still there, just it is now built as part of https://github.com/spatial-model-editor/sme_deps_qt so that qt can link to it as well, as previously qt would build it's own version of zlib and then we ended up with two different static zlib libraries (https://github.com/spatial-model-editor/sme_deps_common/issues/18)

fbergmann commented 1 year ago

I was just wondering where to add combine / zipper. Since it would be useful to have the capability to read combine archives not just from the gui, but from the bindings as well. Basically the plan would be to make it part of the load step to see whether the file loaded is an archive, and if so go through the libcombine.

lkeegan commented 1 year ago

Here would be the place to add them (as well as to https://github.com/spatial-model-editor/sme_manylinux2014_x86_64 for the python wheels)

I have a WIP here: https://github.com/spatial-model-editor/sme_deps_common/pull/22

(ran into some issues with building zipper on msys/mingw, patches here (also a WIP!): https://github.com/lkeegan/zipper/commit/fff200af92c9b203f6b171da70cf3cd1b2d05b92)

fbergmann commented 1 year ago

sorry about the mingw trouble, i'll commit those changes right away.

lkeegan commented 1 year ago

resolved by #22