Review Request 123444: [ksmserver] Do not try to start X11 window manager if Wayland env variables are set

2015-04-21 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123444/ --- Review request for Plasma. Repository: plasma-workspace Description

Build failed in Jenkins: polkit-kde-agent-1_stable_qt5 #5

2015-04-21 Thread KDE CI System
See http://build.kde.org/job/polkit-kde-agent-1_stable_qt5/5/changes Changes: [scripty] SVN_SILENT made messages (.desktop file) [scripty] SVN_SILENT made messages (.desktop file) [jr] Update version number for 5.2.90 [scripty] SVN_SILENT made messages (.desktop file) [scripty] SVN_SILENT

Review Request 123448: Message dialog should be shown before we change a category in the Desktop Settings

2015-04-21 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123448/ --- Review request for Plasma. Bugs: 346431

Build failed in Jenkins: plasma-workspace_stable_qt5 #113

2015-04-21 Thread KDE CI System
See http://build.kde.org/job/plasma-workspace_stable_qt5/113/changes Changes: [lukas] [containmentactions] Verify that list returned from kglobalaccel is not empty -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building

Re: Review Request 123448: Message dialog should be shown before we change a category in the Desktop Settings

2015-04-21 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123448/#review79298 --- Note that currently if you press Apply in the applet's config

Build failed in Jenkins: plasma-desktop_stable_qt5 #122

2015-04-21 Thread KDE CI System
See http://build.kde.org/job/plasma-desktop_stable_qt5/122/changes Changes: [lukas] unbreak Revert function -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3 (PACKAGER LINBUILDER) in

Review Request 123459: Lockscreen: It shouldn't show the battery information on system which they don't have a battery

2015-04-21 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123459/ --- Review request for Plasma. Bugs: 346441

Re: Review Request 123457: Plasma-Workspace: we don't need QtWebkit as a depedency.

2015-04-21 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123457/#review79308 --- Ship it! Ship It! - David Edmundson On April 21, 2015,

Re: Review Request 123448: Message dialog should be shown before we change a category in the Desktop Settings

2015-04-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123448/#review79307 --- Ship it! Ship It! - Marco Martin On April 21, 2015, 3:04

Re: Review Request 123455: Plasma-Desktop: Desktop Settings apply button should become disabled after it has been pressed.

2015-04-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123455/#review79306 --- Ship it! Ship It! - Marco Martin On April 21, 2015, 7:43

Re: Review Request 123455: Plasma-Desktop: Desktop Settings apply button should become disabled after it has been pressed.

2015-04-21 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123455/ --- (Updated April 21, 2015, 8:32 p.m.) Status -- This change has been

Review Request 123457: Plasma-Workspace: we don't need QtWebkit as a depedency.

2015-04-21 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123457/ --- Review request for Plasma. Repository: plasma-workspace Description

Re: Review Request 123448: Message dialog should be shown before we change a category in the Desktop Settings

2015-04-21 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123448/ --- (Updated April 21, 2015, 8:30 p.m.) Status -- This change has been

Re: Review Request 123420: Timer: added tooltip for compact mode (e.g. in panel) and full mode

2015-04-21 Thread Bernhard Friedreich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123420/ --- (Updated April 21, 2015, 7:37 p.m.) Status -- This change has been

Re: Review Request 123411: Protect X11 calls in the Pager plasmoid to prevent plasmashell from crashing in Wayland

2015-04-21 Thread Nerdopolis Turfwalker
On April 18, 2015, 7:50 a.m., Martin Gräßlin wrote: applets/pager/plugin/pager.h, line 144 https://git.reviewboard.kde.org/r/123411/diff/1/?file=361959#file361959line144 m_rows seems not to be initialized if m_isX11 is false I tried to change this in patch revision #2. I hope I'm

Re: Review Request 123411: Protect X11 calls in the Pager plasmoid to prevent plasmashell from crashing in Wayland

2015-04-21 Thread Nerdopolis Turfwalker
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123411/ --- (Updated April 22, 2015, 12:05 a.m.) Review request for Plasma.

Re: Review Request 123459: Lockscreen: It shouldn't show the battery information on system which they don't have a battery

2015-04-21 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123459/#review79321 --- -1 that's what the visible: pmSource.data[Battery][Has

[Powerdevil] [Bug 346456] New: When screen turn off automatically, it will turn black twice.

2015-04-21 Thread Weng Xuetian
https://bugs.kde.org/show_bug.cgi?id=346456 Bug ID: 346456 Summary: When screen turn off automatically, it will turn black twice. Product: Powerdevil Version: 5.2.95 Platform: Chakra OS: Linux

Re: Review Request 123444: [ksmserver] Do not try to start X11 window manager if Wayland env variables are set

2015-04-21 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123444/ --- (Updated April 21, 2015, 10:58 a.m.) Status -- This change has been

Re: Review Request 123444: [ksmserver] Do not try to start X11 window manager if Wayland env variables are set

2015-04-21 Thread Martin Gräßlin
On April 21, 2015, 12:45 p.m., David Edmundson wrote: related to this, on logout ksmserver kills kwin. If you're up first, I assume you also need to be down last? oh I wasn't aware the ksmserver kills kwin. I assumed it's the X connection going away that kills kwin. Will search for it

Re: Review Request 123444: [ksmserver] Do not try to start X11 window manager if Wayland env variables are set

2015-04-21 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123444/#review79278 --- related to this, on logout ksmserver kills kwin. If you're

Re: Review Request 123444: [ksmserver] Do not try to start X11 window manager if Wayland env variables are set

2015-04-21 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123444/#review79279 --- Ship it! Ship It! - David Edmundson On April 21, 2015,

[Plasma Workspace Wallpapers] [Bug 346425] New: crash when you want to change the time of image sequences

2015-04-21 Thread Marc Senn
https://bugs.kde.org/show_bug.cgi?id=346425 Bug ID: 346425 Summary: crash when you want to change the time of image sequences Product: Plasma Workspace Wallpapers Version: 5.2.95 Platform: openSUSE RPMs OS:

Re: Review Request 123448: Message dialog should be shown before we change a category in the Desktop Settings

2015-04-21 Thread Antonis Tsiapaliokas
On April 21, 2015, 3:31 p.m., Martin Klapetek wrote: Note that currently if you press Apply in the applet's config dialogs, the Apply will not become disabled but will stay enabled forever. Antonis Tsiapaliokas wrote: Oh, i see. Then i guess i can fix this one too. I will open

Review Request 123455: Plasma-Desktop: Desktop Settings apply button should become disabled after it has been pressed.

2015-04-21 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123455/ --- Review request for Plasma. Repository: plasma-desktop Description

Re: Review Request 123459: Lockscreen: It shouldn't show the battery information on system which they don't have a battery

2015-04-21 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123459/#review79311 --- lookandfeel/contents/components/InfoPane.qml (line 47)

Re: Review Request 123459: Lockscreen: It shouldn't show the battery information on system which they don't have a battery

2015-04-21 Thread Lukáš Tinkl
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123459/#review79312 --- Ship it! Ship It! - Lukáš Tinkl On Dub. 21, 2015, 11:18