Use Qt to create the autostart entry instead of system calls

Authored by ltoscano on Apr 28 2017, 7:27 AM.



Probably the code predates QFile?

Test Plan

The symlinks is created and removed as expected (and seen
by kcm_autostart).

Diff Detail

R429 KMouseTool
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
ltoscano created this revision.Apr 28 2017, 7:27 AM

Please note that I wrote this against the frameworks version, but I suspect that it would work against the Qt4 version.
If we are going to ship the Qt5 version of kmousetool in Applications 17.08 (which I would recommend, as it seems to work), then this can go directly to frameworks, otherwise let's re-target to master,

whiting edited edge metadata.EditedJun 25 2017, 2:10 AM

Yes, this should go in the frameworks branch, I'll see if I can get the frameworks branch merged to master this week.

ltoscano added a comment.EditedJun 25 2017, 8:35 AM

And what about this review? I mean, if it's fine, can you please select "Approve"? :)

whiting accepted this revision.Jul 5 2017, 10:39 PM
This revision is now accepted and ready to land.Jul 5 2017, 10:39 PM
This revision was automatically updated to reflect the committed changes.