D22701: [kfileutils] Add isExecutableFile from KRun

David Faure noreply at phabricator.kde.org
Wed Jul 24 22:23:22 BST 2019


dfaure added a comment.


  I started to write a KIO::OpenUrlJob, as a core-only version of KRun. I think this method actually belongs there, because it means "can be executed by KRun" (or soon OpenUrlJob).
  In isolation I don't think this method is useful. I looked at all users of this method on lxr.kde.org and they all check this in connection with KRun.
  With this in mind, is there still a good reason to move it to kfileutils?

INLINE COMMENTS

> kfileutils.h:48
> + * -# The mime type must inherit application/x-executable, application/x-executable-script or application/x-sharedlib.
> + * To allow a script to run when the above rules are satisfied add the entry
> + * @code

Please remove the docu starting from this line to the end (i.e. 48-52), it describes KDE3 mimetype desktop files...

REPOSITORY
  R244 KCoreAddons

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

To: nicolasfella, dfaure
Cc: kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190724/6f3fa953/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list