Review Request: Make the kdewin-installer compile on linux
Commit Hook
null at kde.org
Sat Sep 22 23:38:28 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106534/#review19344
-----------------------------------------------------------
This review has been submitted with commit 58f8943a1967bddc3c6efeb730d623653c142f0f by Aleix Pol to branch master.
- Commit Hook
On Sept. 22, 2012, 8:29 p.m., Aleix Pol Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106534/
> -----------------------------------------------------------
>
> (Updated Sept. 22, 2012, 8:29 p.m.)
>
>
> Review request for kdewin, Patrick Spendrin and Patrick von Reth.
>
>
> Description
> -------
>
> Hi, since I wanted to see what is possible and what not with the kdewin installer I decided to take a look into the code and compile it and all.
>
> Here I'm sending the changes I had to make so that it worked properly.
>
> I understand you might want some of the changes and some not, but I think there are some good changes like proper casing in includes or fixing the non-static compilation that can be useful.
>
> Feel free to reject it anyway :P.
>
>
> Diffs
> -----
>
> 3rdparty/CMakeLists.txt 65cf501
> CMakeLists.txt f62fea8
> shared/CMakeLists.txt 3257b62
> shared/externalinstallercontrol.cpp 018ff96
> shared/installercallconfig.cpp b290298
> shared/installerengine.cpp 181a504
> shared/settings.cpp 419349c
> single-package-installer/CMakeLists.txt 0b0de3d
> single-package-installer/installerdialog.h 1619025
> single-package-installer/installerenginesinglepackage.cpp fc43a26
>
> Diff: http://git.reviewboard.kde.org/r/106534/diff/
>
>
> Testing
> -------
>
> Compiled it on Archlinux, made it run for a bit xD.
>
>
> Thanks,
>
> Aleix Pol Gonzalez
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-windows/attachments/20120922/7cd532cf/attachment-0001.html>
More information about the Kde-windows
mailing list