> On Sept. 20, 2014, 2:29 p.m., Martin Gräßlin wrote: > > kcontrol/CMakeLists.txt, lines 45-48 > > <https://git.reviewboard.kde.org/r/120287/diff/4/?file=313800#file313800line45> > > > > so what's the point of the if now? > > > > We have: > > * Windows > > * Apple > > * Linux (by fontconfig) > > Thomas Lübking wrote: > The actual check seems to be for Linux w/o fontconfig. > Likely possible but rare and strange setup.
Or APPLE without fontconfig, which might be a less rare setup. So I'd prefer to keep it like this and figure out in KF5 what the status of the fonts kcm is going to be. > On Sept. 20, 2014, 2:29 p.m., Martin Gräßlin wrote: > > libs/CMakeLists.txt, lines 9-13 > > <https://git.reviewboard.kde.org/r/120287/diff/4/?file=313803#file313803line9> > > > > why don't you merge with NOT WIN32? In all other cases it's merged. I just changed the Darwin check to one for APPLE, didn't want to do anything less atomic :P Maybe we could introduce a molecule, pardon, macro, that boils down to NOX11 in the toplevel cmake file? - René J.V. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120287/#review67058 ----------------------------------------------------------- On Sept. 20, 2014, 12:54 p.m., René J.V. Bertin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120287/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2014, 12:54 p.m.) > > > Review request for KDE Software on Mac OS X and kde-workspace. > > > Repository: kde-workspace > > > Description > ------- > > A few rather straightforward patches to make the relevant bits of KDE4's > kde-workspace build and function on OS X. > The main interest is having the systemsettings control panel to control the > various relevant KDE settings among which desktop search, fonts, colours and > even style. > The oxygen style builds and looks good but shows some updating glitches due > to compositing. > > I'm submitting this patch partly in hope it may be useful in bringing > kf5-workspace to OS X, one day. > > > Diffs > ----- > > CMakeLists.txt 195f99c > kcontrol/CMakeLists.txt fc666b1 > kcontrol/krdb/krdb.cpp 36fc99c > kcontrol/style/CMakeLists.txt d832b20 > libs/CMakeLists.txt c0576fe > > Diff: https://git.reviewboard.kde.org/r/120287/diff/ > > > Testing > ------- > > On OS X 10.6.8 and 10.9.4 with KDE/MacPorts (4.12.5 and more recently kdelibs > git/master, 4.14.1). > > > File Attachments > ---------------- > > copy of the diff file saved locally, which had no tabs when I uploaded it. > Checksum: 3989cdd46af3c891e570974d66c330403dcd41c4ee5e17a372fa385080cbabd1 > > https://git.reviewboard.kde.org/media/uploaded/files/2014/09/20/b212730f-6258-4277-851c-226bc0673aa1__patchreview-20140920.patch > > > Thanks, > > René J.V. Bertin > >