aleksejshilin added a comment.

  In D26582#592469 <https://phabricator.kde.org/D26582#592469>, @meven wrote:
  
  > We may want to change KRun::runUrl when a native binary is to be run but 
with runExecutable is false, around line 400:
  >
  >   } else if (isNativeBinary) {
  >       // Show warning for executables that aren't scripts
  >       noRun = true;
  >   }
  >   
  
  
  KRun currently doesn't seem to consider that there may be a reason to open 
native binaries in an application - its execution confirmation dialog doesn't 
give such option even if some application is associated with the corresponding 
mime type. I believe, they thought that an average user is unlikely to ever 
need this - and they may be right. :)
  
  Given that such use case is probably orders of magnitude less likely than 
sftp: and especially smb:, I guess, it's OK to merge this, and address the 
issue with native binaries in KIO later. @broulik ?

REPOSITORY
  R120 Plasma Workspace

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

To: meven, #plasma, broulik, aleksejshilin
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, 
GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to