Review Request 123075: do not require X11 on Mac OS X
Christoph Cullmann
cullmann at kde.org
Fri Mar 20 12:37:02 UTC 2015
> On March 20, 2015, 7:07 a.m., Martin Gräßlin wrote:
> > as in other similar requests: -2 from my side
>
> Martin Gräßlin wrote:
> To extend: I think the way is wrong. If it now builds on MacOS the required is wrong. It should be an optional find_package properly ifdefed.
Actually, you don't want that it is optional as you really don't want that it ever is found on MacOS. If you install an XQuartz for legacy apps, it will be found, and you will have a completly mess as result ;=)
- Christoph
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123075/#review77779
-----------------------------------------------------------
On March 19, 2015, 10:59 p.m., Harald Fernengel wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123075/
> -----------------------------------------------------------
>
> (Updated March 19, 2015, 10:59 p.m.)
>
>
> Review request for KDE Frameworks and Michael Palimaka.
>
>
> Repository: kdesu
>
>
> Description
> -------
>
> do not require X11 on Mac OS X
>
>
> Diffs
> -----
>
> CMakeLists.txt 9623483d6f11f9cdb7d7dc19decfd7cf5e86d079
>
> Diff: https://git.reviewboard.kde.org/r/123075/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Harald Fernengel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150320/57d6b686/attachment.html>
More information about the Kde-frameworks-devel
mailing list