Differential D29253

[wayland] Pass command line arguments to launched processes
ClosedPublic
Actions

Authored by zzag on Apr 28 2020, 12:53 PM.

Details

Reviewers
cblack
davidedmundson
apol
Group Reviewers
KWin
Commits
R108:2fa5526f12f4: [wayland] Pass command line arguments to launched processes
Summary

Since QProcess::setProgram() doesn't split the command, we need to do it
ourselves.

Test Plan

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

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
zzag created this revision.Apr 28 2020, 12:53 PM
Restricted Application added a project: KWin. · View Herald TranscriptApr 28 2020, 12:53 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Apr 28 2020, 12:53 PM
zzag edited the test plan for this revision. (Show Details)Apr 28 2020, 12:53 PM
zzag updated this revision to Diff 81429.Apr 28 2020, 12:54 PM

Sort includes alphabetically.

cblack accepted this revision.Apr 28 2020, 2:20 PM
This revision is now accepted and ready to land.Apr 28 2020, 2:20 PM
davidedmundson accepted this revision.Apr 28 2020, 4:11 PM
apol accepted this revision.Apr 28 2020, 5:52 PM
apol added a subscriber: apol.

Thanks!

Closed by commit R108:2fa5526f12f4: [wayland] Pass command line arguments to launched processes (authored by zzag). · Explain WhyApr 28 2020, 6:00 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mmain_wayland.cpp (103 lines)

Diff 81452

View Options

main_wayland.cpp

Loading...