Review Request 129396: Allow to read X-KDE-RunOnDiscreteGpu property from desktop file + take this option into account when starting service using KToolInvocation

Kai Uwe Broulik kde at privat.broulik.de
Mon Nov 14 09:43:53 UTC 2016


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




src/kdeinit/ktoolinvocation.cpp (line 98)
<https://git.reviewboard.kde.org/r/129396/#comment67669>

    Use left(...) - the file might have ".desktop" inside its filename, unlikely but possible.


- Kai Uwe Broulik


On Nov. 14, 2016, 8:29 vorm., Jan Grulich wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129396/
> -----------------------------------------------------------
> 
> (Updated Nov. 14, 2016, 8:29 vorm.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Repository: kservice
> 
> 
> Description
> -------
> 
> $SUMMARY says it all I guess, this patch allows to read X-KDE-RunOnDiscreteGpu property which can be now set from KPropertiesDialog, see review 129395. Also when running an application using KToolInvocation (e.g. when running an app from panel or desktop) take this property into account and set required variable accordingly.
> 
> 
> Diffs
> -----
> 
>   src/kdeinit/ktoolinvocation.cpp c9eb922 
>   src/services/kservice.h 66525b4 
>   src/services/kservice.cpp 6574367 
> 
> Diff: https://git.reviewboard.kde.org/r/129396/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

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


More information about the Kde-frameworks-devel mailing list