Review Request: Fix zip invocation on Windows

[Andreas Pakulat]

> On Aug. 12, 2012, 9:02 p.m., Milian Wolff wrote:
> > not knowing anything about windows, is InfoZIP the default? Are there other ZIP implementations? Do they continue to work?

There is nothing like a standard zip tool on Windows, windows doesn't ship one so everybody downloads one from somewhere. And yes this change can break the commandline for other zip tools if they have different requirements. So basically someone has to write find-modules for one or multiple zip-tools that should be supported and setup the commandline accordingly. Alternatively one could write a small utility using the KDE classes that are also used for extraction which is built as part of kdevplatform/internal or so and then used for packaging the templates.


- Andreas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105985/#review17281
-----------------------------------------------------------


On Aug. 11, 2012, 9:18 p.m., Steffen Ohrendorf wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105985/
> -----------------------------------------------------------
> 
> (Updated Aug. 11, 2012, 9:18 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> This fixes the invocation of InfoZIP on Windows which requires the exclusion parameter to be the last one on the command line.
> 
> 
> Diffs
> -----
> 
>   cmake/modules/KDevPlatformMacros.cmake f960e82 
> 
> Diff: http://git.reviewboard.kde.org/r/105985/diff/
> 
> 
> Testing
> -------
> 
> KDevPlatform successfully zips the app templates.
> 
> 
> Thanks,
> 
> Steffen Ohrendorf
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120812/5138c37a/attachment.html>