<table><tr><td style="">mdlubakowski created this revision.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />mdlubakowski requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22510">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Currently if you try to run executable file for which you don't have execute permission, KRun will either open it in some default program, or prompt user to choose one. It would make sense that, if a file can be normally run with +x permission, user should be asked if he likes to automatically make it executable and run it. If user chooses "Open" option, KRun falls back to default behavior.<br />
This QoL change makes it easier to just download & run things like AppImage files.<br />
Obviously, running executables downloaded from the internet may not be safe, so a warning is included in the prompt.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ol class="remarkup-list">
<li class="remarkup-list-item">Obtain two executable files (e.g. AppImage), set +x on one and -x on another.</li>
<li class="remarkup-list-item">Try to run both files. The one with +x will run normally, the other one will show the prompt.</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>make-executable-prompt</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22510">https://phabricator.kde.org/D22510</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/widgets/krun.cpp<br />
src/widgets/krun.h</div></div></div><br /><div><strong>To: </strong>mdlubakowski<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns<br /></div>