Review Request 126007: Fix ksmserver path in startkde

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126007/ --- Review request for Plasma and David Faure. Repository: plasma-workspace

Re: Review Request 125997: Catch other openGL error gracefully

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125997/#review88188 --- Ship it! Ship It! - Martin Gräßlin On Nov. 8, 2015, 10:07

Re: Review Request 125997: Catch other openGL error gracefully

> On Nov. 9, 2015, 7:57 a.m., Martin Gräßlin wrote: > > Could you please explain the change? Reduce string comparison? > > > > Anyway: we should probably extend the checks to also include the EGL/XCB > > error messages an EGL/Wayland error messages > > David Edmundson wrote: > there's a

Re: Review Request 125976: add an update() method

> On Nov. 7, 2015, 2:03 p.m., Sebastian Kügler wrote: > > src/kpackage/private/packagejobthread_p.h, line 58 > > > > > > As you note yourself, perhaps just make it an enum? It's not private > > API, OK, but the

Re: Review Request 125984: kcm_formats: Do not write out an unusable value for the "C" locale

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125984/ --- (Updated Nov. 9, 2015, 1:29 p.m.) Status -- This change has been

Re: Common purpose for the Plasma team, and VDG

as I start the discussion I have to say a few words 1. I understand you. I understand that big changes are a problem and collaboration is not that easy. 2. I start this discussion to have in the end an better cooperation. I am not happy with the process now cause I don't know how it works. an

Re: Review Request 125976: add an update() method

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125976/ --- (Updated Nov. 9, 2015, 2:51 p.m.) Review request for KDE Frameworks,

Re: Review Request 126007: Fix ksmserver path in startkde

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126007/#review88201 --- this is not the only usage of the path. We have the same for

Re: Review Request 125976: add an update() method

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125976/ --- (Updated Nov. 9, 2015, 2:40 p.m.) Review request for KDE Frameworks,

Re: Review Request 125976: add an update() method

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125976/#review88197 --- Ship it! Not sure if we can use the Package::JobError::Foo

Re: Review Request 125977: Remove unused dependencies in breeze-gtk

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125977/ --- (Updated Nov. 9, 2015, 2:35 p.m.) Status -- This change has been

Re: Review Request 125976: add an update() method

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125976/ --- (Updated Nov. 9, 2015, 1:15 p.m.) Review request for KDE Frameworks,

Re: Review Request 125976: add an update() method

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125976/#review88195 --- Ship it! Nice. Few issues with the wording in the comments,

Re: Review Request 126007: Fix ksmserver path in startkde

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126007/#review88198 --- Ship it! Ship It! - Sebastian Kügler On Nov. 9, 2015,

Re: Review Request 125976: add an update() method

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125976/ --- (Updated Nov. 9, 2015, 1:18 p.m.) Review request for KDE Frameworks,

Re: Common purpose for the Plasma team, and VDG

> > the thing is that we are depanded to an programmer but the programmer > isn't related and when we make something by alone like the plasma theme or > icons we get the message you have to ask first. > Yes. All of us ask permission for all changes we make. You've seen us do that. You can

Re: Plasma 5.5 Release Anouncements

On Mon, Nov 9, 2015 at 5:08 PM, Martin Klapetek wrote: > On Sun, Nov 8, 2015 at 8:16 AM, David Edmundson < > da...@davidedmundson.co.uk> wrote: > >> With the proposed no-delay beta, that means we need to start on our >> release anouncement already. >> >> Can everyone

Re: Review Request 126008: [wayland] use Qt::AA_DisableHighDpiScaling in Qt 5.6

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126008/#review88204 --- Ship it! Ship It! - David Edmundson On Nov. 9, 2015, 5:48

Re: Review Request 125951: [calendar] Move the plugins handling to a separate class

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125951/#review88206 --- src/declarativeimports/calendar/eventpluginsmanager.cpp

Re: Review Request 126013: fix: kacess should not be built if xkb is not present

> On Nov. 10, 2015, 1:26 a.m., David Edmundson wrote: > > Ship It! Forgot to mention that the patch can be pulled from the fix-cmake-xkb-dep branch at: g...@github.com:cmacq2/plasma-desktop.git - Johan --- This is an automatically

Re: Plasma 5.5 Release Anouncements

On Mon, Nov 9, 2015 at 4:39 PM, David Edmundson wrote: > >> Absolutely. Which Monday do you mean? Today? >> > > I did. > (unless you read this tomorrow, in which case I meant yesterday) > > Though the release has slipped by a day too, so it's all good. > Isn't the

Review Request 126008: [wayland] use Qt::AA_DisableHighDpiScaling in Qt 5.6

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126008/ --- Review request for Plasma. Repository: kwin Description ---

Re: Review Request 125976: add an update() method

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125976/ --- (Updated Nov. 9, 2015, 3:15 p.m.) Status -- This change has been

Re: Common purpose for the Plasma team, and VDG

Hi, On Monday, November 09, 2015 01:10:41 AM Valorie Zimmerman wrote: > Hello folks, this afternoon my time, a discussion started in #plasma > with some rather upset designers, who said that they felt they are > treated by the Plasma devels as children. > > I waded into that discussion, and am

Re: Plasma 5.5 Release Anouncements

On Sun, Nov 8, 2015 at 8:16 AM, David Edmundson wrote: > With the proposed no-delay beta, that means we need to start on our > release anouncement already. > > Can everyone write down their big changes that are coming in 5.5 that they > made that are worthy of being

Re: Review Request 125951: [calendar] Move the plugins handling to a separate class

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125951/ --- (Updated Nov. 9, 2015, 8:40 p.m.) Review request for Plasma. Changes

Review Request 126010: [digital-clock] Split the Calendar config stuff to it's own category + load the plugins configs

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126010/ --- Review request for Plasma. Repository: plasma-workspace Description

Re: Review Request 125951: [calendar] Move the plugins handling to a separate class

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125951/ --- (Updated Nov. 9, 2015, 8:56 p.m.) Review request for KDE Frameworks and

Re: Plasma 5.5 Release Anouncements

On Monday, November 9, 2015 9:39:44 PM CET David Edmundson wrote: > On Mon, Nov 9, 2015 at 5:08 PM, Martin Klapetek > > wrote: > > On Sun, Nov 8, 2015 at 8:16 AM, David Edmundson < > > > > da...@davidedmundson.co.uk> wrote: > >> With the proposed no-delay beta, that

Re: Common purpose for the Plasma team, and VDG

On Monday, November 9, 2015 11:38:42 PM CET kainz.a wrote: > Yes different communication systems are one of the main problem. Different > tasks are the second problem. When I read the VDG forum threads or the > Telegram thread there was discussion about some stuff but less than 5% are > bugfixing

Re: Review Request 126008: [wayland] use Qt::AA_DisableHighDpiScaling in Qt 5.6

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126008/#review88212 --- *sigh* now I have to update my Qt... Thanks for fixing! -

[Powerdevil] [Bug 354623] Full charge not trigger notification

https://bugs.kde.org/show_bug.cgi?id=354623 Sebastian Kügler changed: What|Removed |Added CC||se...@kde.org --- Comment

Review Request 125972: Don't list systray containment actions in systray context menu.

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125972/ --- Review request for Plasma. Repository: plasma-workspace Description

Review Request 125973: Remove completely duplicated function

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125973/ --- Review request for Plasma. Repository: plasma-workspace Description

Review Request 126012: Fix crash if context gets unrefd

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126012/ --- Review request for Plasma and Harald Sitter. Repository: plasma-pa

Re: Review Request 125997: Catch other openGL error gracefully

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125997/ --- (Updated Nov. 9, 2015, 10:30 p.m.) Status -- This change has been

Re: Common purpose for the Plasma team, and VDG

Hi, 2015-11-09 22:52 GMT+01:00 David Edmundson : > the thing is that we are depanded to an programmer but the programmer >> isn't related and when we make something by alone like the plasma theme or >> icons we get the message you have to ask first. >> > > Yes. All

Re: Common purpose for the Plasma team, and VDG

On Monday 09 November 2015, Valorie Zimmerman wrote: > Hello folks, this afternoon my time, a discussion started in #plasma > with some rather upset designers, who said that they felt they are > treated by the Plasma devels as children. I'm sorry if i sounded like that, wasn't the intention :/

Re: Common purpose for the Plasma team, and VDG

On Monday 09 November 2015, Valorie Zimmerman wrote: > Hello folks, this afternoon my time, a discussion started in #plasma > with some rather upset designers, who said that they felt they are > treated by the Plasma devels as children. I'm sorry if i sounded like that, wasn't the intention :/

Re: Review Request 125984: kcm_formats: Do not write out an unusable value for the "C" locale

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125984/#review88180 --- Ship it! Ship It! - Sebastian Kügler On Nov. 9, 2015,

Review Request 126013: fix: kacess should not be built if xkb is not present

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126013/ --- Review request for Plasma. Repository: plasma-desktop Description

Re: Review Request 126008: [wayland] use Qt::AA_DisableHighDpiScaling in Qt 5.6

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126008/ --- (Updated Nov. 9, 2015, 10:48 p.m.) Status -- This change has been

Re: Common purpose for the Plasma team, and VDG

On Monday 09 November 2015 01:10:41 Valorie Zimmerman wrote: > First, are all the designers on this list? If not, it's difficult to > have a dialog. Please make sure that at least andreas_k and uriherrera > (irc nicks) become part of this conversation. After all, they are > Plasma devels. Just

Re: Review Request 125972: Don't list systray containment actions in systray context menu.

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125972/#review88207 --- Why is that? How is the systray different from other widgets

Re: Common purpose for the Plasma team, and VDG

Hi all, Before I chime in: 'developer' is the term commonly used for 'programmers' - at least in the IT world. Other professions have other meanings for the word like in civil engineering). And, even if we start using the word to mean something else, nobody else will. The account is for that

Re: Review Request 125975: X11 build system cleanup

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125975/ --- (Updated Nov. 9, 2015, 10:11 a.m.) Status -- This change has been

Re: Review Request 125984: kcm_formats: Do not write out an unusable value for the "C" locale

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125984/ --- (Updated Nov. 9, 2015, 8:28 a.m.) Review request for Plasma. Changes

Common purpose for the Plasma team, and VDG

Hello folks, this afternoon my time, a discussion started in #plasma with some rather upset designers, who said that they felt they are treated by the Plasma devels as children. I waded into that discussion, and am reluctant to write to the list about it, but I think there are some issues here

Re: Common purpose for the Plasma team, and VDG

My thoughts on this are two-fold: (a) Non-VDG devs generally shouldn't question VDG dev decisions in detail and avoid push-back on small issues. The VDG is still a relatively new institution and needs to be given the latitude to try things, build up institutional knowledge (and learn

Minutes Monday Plasma Hangout

Present: apol, Riddell, bshah, Kai Uwe, notmart, mgraesslin, sebas Date: 2 Nov, 2015 General, discussed collaboration problems with VDG * Nobody meant to piss anybody off, in case that happened, sincere apologies * Seems to be mostly friction in normal work, lots of stuff getting done, much of

Re: Review Request 125997: Catch other openGL error gracefully

> On Nov. 9, 2015, 6:57 a.m., Martin Gräßlin wrote: > > Could you please explain the change? Reduce string comparison? > > > > Anyway: we should probably extend the checks to also include the EGL/XCB > > error messages an EGL/Wayland error messages there's a second if message == "blah"

Re: Review Request 126013: fix: kacess should not be built if xkb is not present

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126013/#review88208 --- Ship it! Ship It! - David Edmundson On Nov. 10, 2015,