> On Sept. 7, 2015, 7: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.
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 ? - Samuel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125083/#review84937 ----------------------------------------------------------- On Sept. 7, 2015, 7: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, 7: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
