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?
> 
> Andreas Pakulat wrote:
>     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.
> 
> Steffen Ohrendorf wrote:
>     InfoZIP is the one that comes with the emerge tool of the winkde project, and also the "zip" on the linux machine I'm using (OpenSUSE 12.1) is InfoZIP, so I guessed that would be the standard tool.

Can someone check what the kde-runtime binary package from the windows installer provides, i.e. does it even install (itself or via dependencies) a zip utility? I was mostly concerned about that case, i.e. someone wanting to use kdevelop installs it via binary package and has some other zip-tool installed hits problems because the commandline needs to be different.

FWIW my zip on a non-KDE windows box is also info-zip. So if someone can confirm that info-zip is installed via the binary packages then I think the change can go in.


- 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/20120813/a701b543/attachment.html>