User Details
- User Since
- Jul 11 2019, 7:51 AM (246 w, 14 h)
- Availability
- Available
Feb 9 2020
Sep 17 2019
Scrapping this for now, since I have a few improvements on this, but it needs to be reworked. Ill make a PR with updated version on GitLab instance.
Sep 16 2019
- Incorporated feedback
Sep 3 2019
- Reverted change to KRun::isExecutableFile
Sep 2 2019
Fixed according to comments, hope I haven't forget anything this time.
Sorry for the API change, I thought it may make sense to add it like this since isExecutableFile and isExecutable are a part of it.
I also added showing the errorString from setExecuteBit to makeServiceExecutable, since it used similar dialog to KRun::runUrl
- Incorporated feedback
Thanks for your guidance!
I don't have developer account, so can't land myself.
Should I fix those two other comments before landing?
- Removed leftover kfileitem.h include
Reversed my changes to isExecutableFile method as per D23660, since my changes aren't needed anymore. Also added hasExecuteBit method.
- Reverse change to KRun::isExeutableFile
- Incorporated feedback
Aug 29 2019
First of all, sorry for the mess I made here with Arcanist. My SSD died recently and I lost original repository with those changes, so I used arc patch to pull it back, updated the code and pushed here with arc diff --update. I'm not sure if it affects anything though, I tested on fresh repository and it can be patched with this revision without problem.
In response to the feedback, I ditched my original dialog - it is now shared with that from makeServiceExecutable helper function (the one that shows for .desktop files), and its code is moved to showUntrustedProgramWarning method. Code for setting executable bit is also moved, from makeFileExecutable to setExecuteBit.
Desktop and script files also show another type of dialog, one which ask user whether to open or execute the file, but I decided not to use it here with executables since I don't think anyone needs to open executables with external programs.
Incorporated feedback
Aug 26 2019
Jul 31 2019
Jul 22 2019
Jul 17 2019
One of the commits did not upload
Jul 15 2019
- Added missing header for column with session name