Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

René J.V. Bertin rjvbertin at gmail.com
Wed Nov 25 18:12:28 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126170/
-----------------------------------------------------------

(Updated Nov. 25, 2015, 7:12 p.m.)


Review request for KDE Software on Mac OS X and KDE Frameworks.


Changes
-------

Achieve the same (hopefully!) effect by setting `QT_MAC_DISABLE_FOREGROUND_APPLICATION_TRANSFORM`.

`kded` is supposed to be a true daemon that doesn't even have a "systray" presence, so this mechanism may be suitable or even more appropriate. Time will have to tell.

What bothers me is that feature isn't documented, and also that it works differently from setting `LSUIElement`. If I read `qt_mac_transformProccessToForegroundApplication()` correctly, it is indeed the equivalent of setting `LSBackgrounOnly`, and that is *probably* inappropriate for applications that do need to become foreground for instance to post a dialog.


Repository: kded


Description
-------

There should be no reason to build `kded5` as an app bundle on OS X, and with recent feedback in mind I postulated that marking it "nongui" (`ecm_mark_nongui_application`) would be acceptable on other platforms too.

Additionally, `kded5` doesn't have any more reason to appear in the Dock or app switcher, on OS X, so I have added the code to make it an agent.


Diffs (updated)
-----

  CMakeLists.txt 4b9a5ff 
  src/CMakeLists.txt 5e95df8 
  src/kded.cpp 6929d7d 

Diff: https://git.reviewboard.kde.org/r/126170/diff/


Testing
-------

On OS X 10.9.5 with Qt 5.5.1 and FWs 5.16.0 .


Thanks,

René J.V. Bertin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151125/ddfa4b9e/attachment.html>


More information about the Kde-frameworks-devel mailing list