In D28337#648844, @meven wrote:Solid only sends broadcast notification and doesn't wait for any reply, thus unmounting may fail due to preview jobs not stopping in time.That points to a solid shortcoming worth fixing. Adding a wait for apps to ask solid to wait or to wait by default for instance 20-50ms between tearDownRequested and teardownDone
- Queries
- All Stories
- Search
- Advanced Search
Feed Advanced Search
Advanced Search
Advanced Search
May 22 2020
May 22 2020
Mar 27 2020
Mar 27 2020
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.
Mar 12 2020
Mar 12 2020
Mar 10 2020
Mar 10 2020
In D27703#623741, @sandsmark wrote:lgtm.
Mar 6 2020
Mar 6 2020
Could you please merge this change? Or is something stopping from merging it?
Mar 4 2020
Mar 4 2020
In D27774#621379, @yurchor wrote:There are other pending requests that were pending uncommitted. Would you like to push them into the repo first?
Mar 2 2020
Mar 2 2020
Feb 28 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 27 2020
In D27650#618492, @davidedmundson wrote:Can I have your full name + email address please.
Feb 26 2020
Feb 26 2020
Got rid of two disconnects and updated summary formatting.
In D27650#618207, @davidedmundson wrote: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
In D27650#617550, @davidedmundson wrote:This function takes a plain pointer and wraps it into weak shared pointer.
That's QWeakPointer.
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 25 2020
Feb 18 2020
Feb 18 2020
Could you please take a look at this change as well?
Feb 14 2020
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
Feb 13 2020
In D27357#610605, @broulik wrote: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
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
Feb 10 2020
Thanks, I'll try reworking KIdleTime.
alnikiforov updated the diff for D27278: Prevent powerdevil from calling DPMS extension calls when DPMS extension isn't present.
Updated remaining conditions
alnikiforov added a comment to D27278: Prevent powerdevil from calling DPMS extension calls when DPMS extension isn't present.
In D27278#608788, @broulik wrote:Makes sense, this can go into 5.18 branch.
Do I need to land this for you?
alnikiforov added a comment to D27278: Prevent powerdevil from calling DPMS extension calls when DPMS extension isn't present.
In D27278#608791, @broulik wrote:There seem to be a bunch of additional m_helper.isNull() that could be ported to be consistent?
alnikiforov updated the diff for D27278: Prevent powerdevil from calling DPMS extension calls when DPMS extension isn't present.
Fixed superfluous parenthesis in one of conditions.
alnikiforov added inline comments to D27278: Prevent powerdevil from calling DPMS extension calls when DPMS extension isn't present.
alnikiforov added reviewers for D27276: Powerdevil: allow using qt5widget classes with powerdevil: apol, davidedmundson.
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.
alnikiforov updated the summary of D27276: Powerdevil: allow using qt5widget classes with powerdevil.
Oct 15 2019
Oct 15 2019
alnikiforov added a comment to D24653: discover: check package status before trying to install/update it.
In D24653#547385, @apol wrote:Please submit to master.
alnikiforov updated the diff for D24653: discover: check package status before trying to install/update it.