[Powerdevil] [Bug 423131] powerdevil does not release pending Inhibit cookie if source application exits too quickly

2020-06-22 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=423131

Kai Uwe Broulik  changed:

   What|Removed |Added

   Version Fixed In||5.18.6
 Resolution|--- |FIXED
  Latest Commit||https://invent.kde.org/plas
   ||ma/powerdevil/commit/d21102
   ||cc6c7a4db204a29f376ce5eb316
   ||ef57a6e
 Status|ASSIGNED|RESOLVED

--- Comment #3 from Kai Uwe Broulik  ---
Git commit d21102cc6c7a4db204a29f376ce5eb316ef57a6e by Kai Uwe Broulik.
Committed on 19/06/2020 at 17:51.
Pushed by broulik into branch 'Plasma/5.18'.

Watch DBus service right away to discard pending inhibitions reliably

PowerDevil delays applying a power management inhibition by 5 seconds to avoid
brief inhibitions
to e.g. wake up the screen and similar.
However, when an inhibition was posted and the process then quit, we would then
still
enforce the inhibition as it wasn't explicitly revoked.
Fix this by always watching the inhibition sender and remove it from pending
when the
service goes away.
FIXED-IN: 5.18.6

M  +12   -14   daemon/powerdevilpolicyagent.cpp

https://invent.kde.org/plasma/powerdevil/commit/d21102cc6c7a4db204a29f376ce5eb316ef57a6e

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 423131] powerdevil does not release pending Inhibit cookie if source application exits too quickly

2020-06-19 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=423131

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #2 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/plasma/powerdevil/-/merge_requests/8

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 423131] powerdevil does not release pending Inhibit cookie if source application exits too quickly

2020-06-19 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=423131

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org
 Status|REPORTED|CONFIRMED
 Ever confirmed|0   |1

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 423131] powerdevil does not release pending Inhibit cookie if source application exits too quickly

2020-06-18 Thread Gabriel Marcano
https://bugs.kde.org/show_bug.cgi?id=423131

--- Comment #1 from Gabriel Marcano  ---
I think the solution is to move the m_busWatcher parts in
addInhibitionWithExplicitDBusService and ReleaseInhibition further up, so that
they are processed before the functions are short-circuited by the pending
checks. This way a service will be registered as soon as we have even a pending
request, and thus the pending cookie will be released properly when that
service dies or goes away.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 423131] powerdevil does not release pending Inhibit cookie if source application exits too quickly

2020-06-17 Thread Gabriel Marcano
https://bugs.kde.org/show_bug.cgi?id=423131

Gabriel Marcano  changed:

   What|Removed |Added

 CC||gabemarc...@yahoo.com

-- 
You are receiving this mail because:
You are watching all bug changes.