Review Request 112991: Fix compilation rules of KDE-Workspace under OSX/Macports

[Gilles Caulier]

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


Martin,

KDE 4.11 or master (Qt5) _must_ compile under OSX. My current patch work fine against master and can be applied to git repository.

I currently working to adjust this patch with KDE 4.11 branch.

I know that most of developers work under Linux and X11 based system. But this must not be a portability limitation, especially when people provide a patch on other platform. After all, Windows is also supported by KDE-Workspace, why not OSX. My patch simply reproduce compilation rules defined by Windows to OSX. 

Gilles Caulier

- Gilles Caulier


On Sept. 29, 2013, 5:15 p.m., Gilles Caulier wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112991/
> -----------------------------------------------------------
> 
> (Updated Sept. 29, 2013, 5:15 p.m.)
> 
> 
> Review request for kde-workspace.
> 
> 
> Bugs: https://trac.macports.org/ticket/33780
>     http://bugs.kde.org/show_bug.cgi?id=https://trac.macports.org/ticket/33780
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> This patch fix broken compilation under OSX / macports about kde-workspace.
> 
> Patch do not touch implementation. Only compilation rules are changed in cmake script to follow the way way than Windows rules, where no X11 lib are available.
> 
> By this way, Oxygen is compiled and installed to macport and digiKam has a suitable GUI under OSX.
> 
> See my Macports bug report for details : https://trac.macports.org/ticket/33780 
> 
> Gilles Caulier
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt c37ab8b 
>   kcontrol/CMakeLists.txt a25aaa0 
>   libs/CMakeLists.txt 9d71a03 
> 
> Diff: http://git.reviewboard.kde.org/r/112991/diff/
> 
> 
> Testing
> -------
> 
> I tested this patch under my macbook pro, using a fresh install of Macports (KDE 4.11.1 / Qt 4.8.5)
> 
> As kde-workspace macports package is broken, i checkout code from KDE git/master repository and fixed compilation rules as well. 
> 
> 
> Thanks,
> 
> Gilles Caulier
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20131022/3bc4870c/attachment.htm>