---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124289/#review82221
---
Ship it!
It's what I would have expected for private
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124288/#review82220
---
Ship it!
It's what I would have expected for private
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124281/#review82234
---
src/knotificationmanager.cpp (line 92)
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124282/
---
(Updated heinä 8, 2015, 5:10 ip)
Review request for KDE Frameworks and
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124220/#review82237
---
src/pluginwrapper.cpp (line 68)
On jul 7, 2015, 7:38 p.m., Martin Klapetek wrote:
I'm looking at the KNotifications dependency graph here [1] and see that
KWindowSystem is only required for KCrash.
So err, can't that one go as well since you removed KService which required
KCrash which then required KWindowSystem?
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124288/
---
Review request for KDE Frameworks.
Repository: kwindowsystem
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124289/
---
Review request for KDE Frameworks.
Repository: kidletime
Description