broulik created this revision. broulik added reviewers: Frameworks, VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. broulik requested review of this revision.
REVISION SUMMARY `QStandardPaths::findExecutable` only finds executable binaries, so when explicitly pointing it at an existing file, it would complain about it not existing, rather than telling you it's not executable. TEST PLAN - Tried to associate a non-executable shell script with a file: it now told me what was up Before it would tell me file doesn't exist REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29445 AFFECTED FILES src/widgets/kopenwithdialog.cpp To: broulik, #frameworks, #vdg Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns