Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-25 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117644/#review56571 --- This review has been submitted with commit

Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-24 Thread Wolfgang Bauer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117644/ --- (Updated April 24, 2014, 3:42 p.m.) Review request for kde-workspace and

Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-24 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117644/#review56416 --- and this as well then. - Thomas Lübking On April 24, 2014,

Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-24 Thread Wolfgang Bauer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117644/ --- (Updated April 24, 2014, 10:28 p.m.) Status -- This change has been

Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-24 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117644/#review56437 --- This review has been submitted with commit

Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-23 Thread Wolfgang Bauer
On April 23, 2014, 7:36 a.m., Martin Gräßlin wrote: would you please also adapt that for plasma-workspace repo (new master)? Yes, I will. Should I create a new review request for that, or should I just submit it? - Wolfgang ---

Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-23 Thread Martin Gräßlin
On April 23, 2014, 7:36 a.m., Martin Gräßlin wrote: would you please also adapt that for plasma-workspace repo (new master)? Wolfgang Bauer wrote: Yes, I will. Should I create a new review request for that, or should I just submit it? The code looks pretty straight forward

Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-22 Thread Wolfgang Bauer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117644/ --- Review request for kde-workspace and Plasma. Bugs: 224200

Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-22 Thread Wolfgang Bauer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117644/ --- (Updated April 22, 2014, 10:41 p.m.) Review request for kde-workspace

Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-22 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117644/#review56219 --- Ship it! from here - see veto condition in the other RR (i

Re: Review Request 117644: screenlocker: don't leave behind screensaver processes

2014-04-22 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117644/#review56240 --- would you please also adapt that for plasma-workspace repo