D29445: [KOpenWithDialog] When pointing at a non-executable file print more meaningful error
Kai Uwe Broulik
noreply at phabricator.kde.org
Tue May 5 14:07:17 BST 2020
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200505/da2e4365/attachment.html>
More information about the Kde-frameworks-devel
mailing list