Review Request: Fix zip invocation on Windows

[Steffen Ohrendorf]

> 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.
> 
> Andreas Pakulat wrote:
>     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.

I grep'ed all the archives the installer had downloaded, and zip is not included, but bzip2 is, though tar is unfortunately not included. But AFAIK that would be only a dependency problem, because the emerge tool that actually builds the binary packages downloads infozip, though I haven't found it on http://winkde.org/pub/kde/ports/ -- which either means that there are licensing problems or that the winKDE team simply thought that the binary releases do not need zip. Maybe someone should ask them to include either tar or infozip as an additional dependency.


- Steffen


-----------------------------------------------------------
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/295efbda/attachment.html>