User Details
- User Since
- Jul 16 2018, 3:21 PM (309 w, 2 d)
- Availability
- Available
Apr 22 2020
Apr 12 2020
Hi @bruns - did you have a chance to go through this patch again? Am I missing anything to move on with this?
Mar 10 2020
Mar 9 2020
Mar 5 2020
Review comments
Mar 4 2020
@bruns - thanks a lot for the feedback, I have updated the patch with your suggestions.
Addressed review comments
Mar 3 2020
Hi @bruns - I have updated this patch with the changes you requested and would really like your feedback :)
Feb 20 2020
Feb 19 2020
Updated patch with review comments
Feb 4 2020
Jan 19 2020
Jan 18 2020
I have refactored the fstab handling to make supporting fuseiso _much_ simpler - so please take a look at D26600 - once that is merged I will push a new review of this patch.
Jan 12 2020
@meven - I am a little unsure if I have broken the fix you have done in commit c97f0b2a3076731b35435f200bd09a22859f3e03 - could you please check?
Dec 15 2019
Sorry for breaking the build :/
Nov 17 2019
Nov 15 2019
@davidedmundson - ping ;)
@bruns - ping ;)
Nov 3 2019
Note that I haven't tested this patch locally because I don't know how to "run" the locally compiled devicenotifier. I can see that I get a plasma_engine_devicenotifications.so library from building the project, but I don't know how to "run" it, do we have a cli tool to load the shared library for testing? Or how do you guys test?
@bruns - ping :) I have updated this patch with the changes you requested, I hope you are ok with it now.
Oct 27 2019
Oct 22 2019
Review comments
Oct 21 2019
Review comments
Reorder functions to make diff smaller
Oct 20 2019
@bruns - I have now refactored the patch so that it uses the getmntent functions for parsing the mtab file, so I think this patch is pretty much ready for a serious review ;)
Rewrite to use the getmntent function for parsing the mtab file
Oct 19 2019
Oct 18 2019
Currently you have the 'unmount' action if you right click on the device in dolphin, but it cannot unmount, so should we hide it? Or should we fix it so that it can actually unmount?
Implemented parsing of the fuseiso mtab file
@davidedmundson ping :)
Updated to use KListOpenFilesJob
Oct 6 2019
@dfaure - I just found that if you include KListOpenFilesJob from e.g. Dolphin then it fails because it cannot include jobs/kjob.h :(
Oct 5 2019
Oct 3 2019
Oct 1 2019
Sep 30 2019
Sep 29 2019
Sep 27 2019
Review comments
Sep 23 2019
Sep 22 2019
I have been resurrecting this patch again :) and have run into an issue I need some guidance on. To be able to parse the ~/.mtab.fuseiso file I would like to use the KMountPoint class, but this class currently resides in KIO which Solid doesn't depend on. But, KIO actually depends on Solid so would it be an option to move this class from KIO to Solid?
Fix Back/Forward shortcuts
Sep 19 2019
@davidedmundson - ping :)
Restrict the number of navigation entries to 12
Sep 17 2019
Sep 14 2019
Sep 12 2019
Use QSKIP to skip tests
Sep 11 2019
Use QStandardPaths::findExecutable to locate lsof
Updated @since
Sep 9 2019
Sep 7 2019
Rebased
Sep 6 2019
Review comments
Review comments, renamed the files to match the class name