It's also may be possible to repeat unmount request from solid a few times with some delays between requests to try waiting until preview jobs are stopped when unmount is requested from 'mounted devices' widget. I've tried 5 retries with 2 seconds between each one, and it worked for me. But waiting for replies to unmount notifications in solid might be preferable to retries.
Fri, Mar 27
Thu, Mar 12
Tue, Mar 10
Fri, Mar 6
Could you please merge this change? Or is something stopping from merging it?
Mar 4 2020
There are other pending requests that were pending uncommitted. Would you like to push them into the repo first?
Mar 2 2020
Feb 28 2020
Updated change to obtain information only about last device mounted over specific mount path. At least filesystem may be different in such case.
Feb 27 2020
Feb 26 2020
Got rid of two disconnects and updated summary formatting.
And this used QWeakPointer leads to premature destruction of object m_layout points to.
Do you have commit access?
Fix consists of following line:
Uploaded updated change
This function takes a plain pointer and wraps it into weak shared pointer.
QPointer is something else that doesn't have an equivalent in stdlib.
It takes a QObject, then QObject has a special hook to unset watching QPointers to nullptr in QObject::~QObject()
Feb 25 2020
Feb 18 2020
Could you please take a look at this change as well?
Feb 14 2020
Here's screenshot of empty Science and Development groups being visible before this change is applied:
Here's how empty "Other Settings Modules" entry looks when selected:
Feb 13 2020
Neither the XSync nor the windows backend explicitly call resumingFromIdle in simulateUserActivity, so I think this should be fine.
However, I don't know if that won't indirectly trigger it if the session was idle? Maybe we need to check if the session was idle and only then emit the signal.
I think I found source of infinite recursion.
Feb 12 2020
I've managed to reproduce it and take a backtrace. It's an infinite recursion. Eventually stack's end is reached, although it takes some time to do it. Here's a piece of backtrace I got:
I've tested this patch in it's current form, and powerdevil no longer crashes for me on launch too. How can I reproduce crash in resume on idle? Can it be done in virtual machine?
Feb 10 2020
Thanks, I'll try reworking KIdleTime.
Updated remaining conditions
Makes sense, this can go into 5.18 branch.
Do I need to land this for you?
There seem to be a bunch of additional m_helper.isNull() that could be ported to be consistent?
Fixed superfluous parenthesis in one of conditions.
I've hit this crash trying to use KDE via x2go (https://wiki.x2go.org/doku.php) with both x2go client and server running in two separate VMs.