D29249: Use deprecated QProcess::start() overload

Vlad Zahorodnii noreply at phabricator.kde.org
Tue Apr 28 09:13:22 BST 2020


zzag created this revision.
zzag added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
zzag requested review of this revision.

REVISION SUMMARY
  In order to use QProcess::setProgram() we need to split the command into
  a list of tokens. Unfortunately, the relevant method had been introduced
  only in Qt 5.15.
  
  In order to avoid regressions, this change partially reverts cca0e15b45 <https://phabricator.kde.org/R108:cca0e15b455ba606043c720479490794d8d86309>.

TEST PLAN
  `dbus-run-session kwin_wayland "kate foo.txt"` works again.

REPOSITORY
  R108 KWin

BRANCH
  showfps-correct-timestamp

REVISION DETAIL
  https://phabricator.kde.org/D29249

AFFECTED FILES
  main_wayland.cpp

To: zzag, #kwin
Cc: kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20200428/36a6a974/attachment.html>


More information about the kwin mailing list