Review Request 120287: [OS X] make kde-workspace build

René J.V. Bertin rjvbertin at gmail.com
Tue Oct 14 17:06:10 BST 2014


-----------------------------------------------------------
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20141014/c7e6ff03/attachment.htm>


More information about the kde-core-devel mailing list