Page MenuHomePhabricator

Add .zip support to POCO
Closed, ResolvedPublic

Description

As part of scene-serialization, we need a module encapsulating a library for creating and reading .zip files. Seemingly Info-ZIP seems most suited. Info-ZIP is (at least in part) used by WinZip and has a permissive license: http://infozip.sourceforge.net/license.html

A module should be created for the pure purpose of creating and reading .zip files. Info-ZIP is just one option, more shoud be considered.

Related Objects

StatusAssignedTask
Wontfixkislinsk
InvalidNone
ResolvedNone

Event Timeline

I just learned that POCO brings .zip support. I'll just add those files to our POCO utility lib.

Jochen or Hannes, could you please check if the attached patch (applied to mitk/Utilities/Poco) compiles on Windows? If so, you can switch on the advanced CMake option "Poco_BUILD_TESTSUITES", build again, then cd into the Poco/Zip/testsuite/data folder and execute the test driver.

Hope that everything works fine. With Linux it does -- I'll check MacOS.

Attached patch can be applied to {MITK-root}/Utilities/Poco

Building PocoZip produces 165 messages of the following warning:

..\..\..\..\..\mitkSource\mitk\Utilities\Poco\Zip\src\ZipOperation.cpp(45) : warning C4273: 'Poco::Zip::ZipOperation::ZipOperation' : inconsistent dll linkage

added poco zip library in revision #17949.

Daniel confirmed MacOS and Linux compatibility. Closing this bug now.