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
