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