> On Sept. 7, 2015, 9:17 a.m., Martin Gräßlin wrote: > > hey OSX-devs: we need to find a better solution for this things. I didn't > > notice the review before it was pushed, but I would not have given a shipit > > for it. > > > > The solution is sementically wrong. We are now binding finding XCB to > > whehter XLib is found. In the long termn XLib support will become a > > deprecated feature, maybe even completely removed in KWindowSystem. Most is > > already ported to XCB. What then? We go back to finding XCB and it will be > > found again? Then there is nothing to group with. > > > > Please think about a long term strategy how you want to handle the X11 > > dependencies. We have had many such reviews lately and it's always a local > > "workaround"/"fixup" to some OSX build-oddities. Please have a look at the > > bigger picture and think about a general solution on how to disable X11 > > without having to (stupidly) patch all frameworks. > > Samuel Gaist wrote: > Hi, > > Sorry for that mistake. > > What about providing KF5 X11/XCB specifc cmake search modules ? This will > also require patching of the frameworks but will allow to disable it globally > when needed ?
> allow to disable it globally when needed ? But that's already built into CMake. Just pass the right flag at command line and it will build without X11/XLib support. Our CI system does that for every build. Given that I don't understand why OSX devs want again and again (you're not the first and probably not the last) add specific solutions to their system because somehow X11 ends in the build. - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125083/#review84937 ----------------------------------------------------------- On Sept. 7, 2015, 9:25 a.m., Samuel Gaist wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125083/ > ----------------------------------------------------------- > > (Updated Sept. 7, 2015, 9:25 a.m.) > > > Review request for KDE Frameworks. > > > Repository: kwindowsystem > > > Description > ------- > > This avoids trying to detect X11 packages that might be installed on OS X > > > Diffs > ----- > > CMakeLists.txt 3460224eb895b2295b6e7ee66dd8258da5ec2b91 > > Diff: https://git.reviewboard.kde.org/r/125083/diff/ > > > Testing > ------- > > Build on OS X 10.8 > > > Thanks, > > Samuel Gaist > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
