----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120287/ -----------------------------------------------------------
(Updated Oct. 14, 2014, 6:06 p.m.) Review request for KDE Software on Mac OS X and kde-workspace. Changes ------- A new revision of the patchset. This revision includes the plasma subdirectory in the build, in order to provide the possibility to run plasmoids via plasma-windowed. In an initial step, te CMake files under /plasma have been changed to exclude the same components on OS X as on MS Windows, i.e. checks are now for `NOT WIN32 AND NOT APPLE` instead of for `NOT WIN32` only. Further exclusions were then made for the components that gave build errors due to X11 dependencies. Plasma-windowed's build process was modified to add an Info.plist that makes the application an agent, i.e. a GUI application without menubar or presence in the Dock or AppSwitcher. Changes had to be made to the plasma-windowed code to prevent crashing when exiting from a successfully loaded plasmoid through the standard Command-Q quit instead of by closing the window. Replacing all `delete` operations of QObject-derived class instances with `->deleteLater()` was enough to prevent crashing (suggesting that a nested eventloop was the culprit, or at the least an immediate delete rather than a deferred delete compatible with Objective-C's `release` (which takes effect when exitting the event-loop in which the release was done)). I tried making PlasmaApp inherit KApplication instead of KUniqueApplication so that multiple instances can be run concurrently. This has the effect that plasmoids that usually work are no longer rendered. I'd appreciate pointers on how to circumvent this. 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 (updated) ----- CMakeLists.txt df8a1f7 kcontrol/CMakeLists.txt fc666b1 kcontrol/krdb/krdb.cpp 36fc99c kcontrol/style/CMakeLists.txt d832b20 libs/CMakeLists.txt c0576fe plasma/CMakeLists.txt 199dbb0 plasma/generic/shells/plasma-windowed/plasmaapp.cpp dbdff47 plasma/netbook/CMakeLists.txt 1eff685 plasma/netbook/containments/CMakeLists.txt c96a688 plasma/desktop/CMakeLists.txt 2de78dd plasma/desktop/applets/CMakeLists.txt 6f80cec plasma/generic/CMakeLists.txt cfaf14f plasma/generic/applets/CMakeLists.txt 2b888ee plasma/generic/dataengines/CMakeLists.txt d240683 plasma/generic/runners/CMakeLists.txt 6831ac0 plasma/generic/shells/plasma-windowed/CMakeLists.txt 86b7770 plasma/generic/shells/plasma-windowed/Info.plist.template PRE-CREATION 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
