D22510: Added dialog to set execute permission for executable file when trying to run it.

[Marcin Dłubakowski]

mdlubakowski marked 6 inline comments as done.
mdlubakowski added a comment.


  Fixed according to comments, hope I haven't forget anything this time.
  Sorry for the API change, I thought it may make sense to add it like this since `isExecutableFile` and `isExecutable` are a part of it.
  I also added showing the errorString from `setExecuteBit` to `makeServiceExecutable`, since it used similar dialog to `KRun::runUrl`

REPOSITORY
  R241 KIO

BRANCH
  arcpatch-D22510

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

To: mdlubakowski, #frameworks, dfaure, cfeck, pino
Cc: broulik, ngraham, probono, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190902/ba3d8c8b/attachment-0001.html>