Review Request 129318: Add an option to context menu to launch the application on a discrete graphics card

[Jan Grulich]

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

(Updated Lis. 3, 2016, 9:21 dop.)


Review request for Plasma and Eike Hein.


Changes
-------

Remove trailing spaces, somehow this auto-remove trailing spaces was disabled in my KDevelop.


Repository: plasma-desktop


Description
-------

In Fedora 25 there have been many improvements regarding dual graphics support and they recently added an option to Gnome to launch the application using a dicrete graphics card, by just overriding DRI_PRIME environment variable, see http://www.hadess.net/2016/10/dual-gpu-integration-in-gnome.html. This option is available based on simple dbus service implemented here https://github.com/hadess/switcheroo-control/ providing just a single property whether the user has dual GPU. This dbus thingie is enabled in Fedora 25 by default. This patch tries to add this option the same way as Gnome does. I'm just not sure about using qputenv(), but I couldn't find a better way as using QProcess, which allows to set environment variables, won't have that nice mouse effect of opening application, at least I think it won't.


Diffs (updated)
-----

  applets/kicker/plugin/actionlist.h 869354c 
  applets/kicker/plugin/actionlist.cpp 2aa97d9 
  applets/kicker/plugin/appentry.cpp 80290e1 
  applets/kicker/plugin/recentusagemodel.cpp 24874ff 
  applets/kicker/plugin/runnermatchesmodel.cpp 4e4845c 

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


Testing
-------


Thanks,

Jan Grulich

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161103/adc03f90/attachment.html>