Summary: | ApperSentinel inhibits powermanager actions inappropriately | ||
---|---|---|---|
Product: | [Applications] apper | Reporter: | Ryan Rix <ry> |
Component: | general | Assignee: | Daniel Nicoletti <dantti12> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | drf, kevin.kofler, rdieter |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Bug Depends on: | |||
Bug Blocks: | 289760 |
Description
Ryan Rix
2012-02-13 21:20:54 UTC
Well actually it only grabs a cookie when something is running... But really this never really worked reliable, because for some reason power devil seemed not to work properly. I really thing this is a bug in power devil not Apper, since the code is rather simple, grab cookie drop cookie. All tests I did this didn't work well Adding Dario as CC, can you two sort this out? I'm just a peoplemover, not a codemover :) sure I asked Dario about this once, he said he found the bug and fixed but I really never managed to see this working right even after... Maybe my code has a bug but the mechanism is rather simple. I might have a look at this today, now pd is fixed so the problem is on the other side. Can you please point me at the routine where inhibit is triggered? void TransactionWatcher::transactionListChanged(const QStringList &tids) // release any cookie that we might have 93 suppressSleep(true); Maybe you meant false? :) hmm right that might be the problem, tho I think I set that to true because it wasn't suppressing sleep... Can you point me how to make sure it is suppressing sleep? ie not suspend when lid closed or something? That one should be indeed false. The routine is correct apart from that, as your comment also suggests - when no transactions are running, you should release your inhibition. The overall code is fine, it's a mere logical error. The reason why it didn't work for you was probably because inhibition was pretty much broken before 4.8, I suppose. Right, and now that it is working we have bug reports :P I'll fix that and include in the upcoming version... Thanks. Can you CCBUG your commit to this one? I know rdieter is looking to get KDE 4.8 in to Fedora 17 and having this bug addressed before then (even with a manual backport in to the current stable) would probably be a Good Thing for us. @Ryan: if you are in a real hurry, you just s/suppressSleep(true);/suppressSleep(false);/g in the line I pointed you to (93 in the file Daniel mentioned) and be happy :) Bug fixed in: http://commits.kde.org/apper/8e43cddd2634b03ee5a05f5287792bb006e903f8 Thanks for reporting it :D |