ifcplusplus icon indicating copy to clipboard operation
ifcplusplus copied to clipboard

Allow miniz to be provided externally

Open antoine-fresse opened this issue 2 years ago • 2 comments

Hello,

I have a project that already uses miniz, I had to comment out the miniz implementation in zippy and replace it with an include to miniz.h in order to not have duplicate symbols errors at link.

It would be nice to have a option in cmake to provide miniz ourselves

Thank you

antoine-fresse avatar Nov 30 '23 15:11 antoine-fresse

Hello,

I understand your point. The thing is, I don't want to change the zippy.h header. I need zippy only for packing and unpacking ifczip files. If there is a better option than zippy, I can replace it. For now, I will keep it in mind.

ifcquery avatar Dec 01 '23 09:12 ifcquery

You know I just realized that one the uses of miniz in our project was to support ifczip files before ifcplusplus supported them.

I'm not familliar with zippy, is it just a wrapper around miniz to have a c++-like API ? It's only a couple extra lines to use miniz directly, hardly seems worth it.

But anyway it's not really a big problem, as I said I just modified the zippy header in our fork to remove the miniz implementation

antoine-fresse avatar Dec 01 '23 11:12 antoine-fresse