minutes meeting week 20

[Bhushan Shah]

On Mon, May 14, 2018 at 04:18:07PM +0530, Bhushan Shah wrote:
> (See logs for more discussion)

and ofcourse logs I forgot to attach, attached now

-- 
Bhushan Shah
http://blog.bshah.in
IRC Nick : bshah on Freenode
GPG key fingerprint : 0AAC 775B B643 7A8D 9AF7 A3AC FE07 8411 7FBC E11D
-------------- next part --------------
15:31 <bshah> alright Sho_ starts
15:31 <Sho_> 
15:31 <Sho_> * [PBI] Finished David's browser integration extension installation reminder for him, since went on vacation
15:31 <Sho_> * [SysSe] Finished Language KCM port (w/ DND coming in 5.14)
15:31 <Sho_> * [SysSe] Spent Sunday working with my GSoC student on our keyboard/input KCM rewrite, very productive day
15:31 <Sho_> ** He's getting set up with a git account and a blog, first blog report soonish
15:31 <Sho_> * [FV] Always set applet title even when the label is disabled in settings, so FV always gets a nice tab title in the grouping apple
15:31 <Sho_> * [Kicker] Fixed recent regression (by dfaure! :) breaking submenu updating when switching between categories of the same size
15:31 <Sho_> * [Kicker] Looking into bug 394202
15:31 <sKreamer> KDE bug 394202 in plasmashell (Application Menu (Kicker)) "Single letter upper case search hides Application and System Settings categories" [normal,] https://bugs.kde.org/show_bug.cgi?id=394202
15:31 <Sho_> * [Plasma] Spent some time butting heads with the org.kde.plasma:force-create stuff and wrote bug 394034 about it
15:31 <sKreamer> KDE bug 394034 in plasmashell (general) "Convention of passing "org.kde.plasma:force-create" as argument to Containment::createApplet is crap" [normal,] https://bugs.kde.org/show_bug.cgi?id=394034
15:31 <Sho_> * [Other] Thursday was a holiday
15:31 <Sho_> next()
15:32 <bshah> kbroulik: your turn
15:32 <kbroulik> been sick all week, not much from me :(
15:32 <kbroulik> ++meetingIt;
15:32 <bshah> aww
15:32 <bshah> notmart: your turn
15:32 <notmart> ok
15:32 <notmart>  * fixed a bug in the base qml kcm classes that broke the language kcm rewrite
15:33 <notmart>  * Continued working primarly on the virtual desktop wayland protocol, it's on Phab D12820, now will be review period in which will change a lot
15:33 <sKreamer> Diff 12820 "Add KWayland virtual desktop protocol" [Needs Review] https://phabricator.kde.org/D12820
15:33 <notmart>  ** this week added the plasmawindow part, windows can enter and exit desktops
15:33 <notmart>  ** an attempt to make the manager to keep internally coherence in the desktop grid. It will probably have to be reworked further. 
15:33 <notmart>  * Kirigami: fixing a bug in the DelegateRecycler, which involved a minimal refactoring
15:33 <notmart>  * Kirigami: working on a component to reorder ListViews by drag and drop, will be used in the language kcm (post-5.13) 
15:33 <notmart> questions/next
15:33 <bshah> romangg: your turn
15:33 <romangg> * pushed D12739, fix for kdesrc-build
15:33 <sKreamer> Diff 12739 "Ignore system standard path when prepending environment variables" [Closed] https://phabricator.kde.org/D12739
15:33 <romangg> * reviewed patches for Martin and notmart
15:34 <romangg> Still blocked otherwise on my patches for KWin/KWayland through missing reviews. There seems to be a problem in general at the moment. Should we discuss this now or later?
15:34 <romangg> (maybe also because d_ed was away last week)
15:35 <Sho_> I think we got review-backlogged because of David/Kai absence, notmart digging into VDs (sorely needed), generally people wanting to get back to their work after sprint travel
15:35 <Sho_> It's good to note that we need to get back to reviews now, but I don't think it's a fundamental problem just yet
15:35 <notmart> yeah, will review more stuff this week
15:35 <Sho_> same
15:35 <notmart> except.. forgot one thing, tomorrow i'll be away :)
15:36 <Sho_> I actually have a half day off tmrw too, wedding suit fitting
15:36 <notmart> but apart from that.. yeah
15:36 <romangg> Well, I see the "fundamental" problem in KWin/KWayland more in the sense that often it's dependent on Martin, who can't be so active anymore than in the past.
15:36 <Sho_> (scared of post-sprint fatness ...)
15:36 <Sho_> romangg: Not sure,  Igot the EGL context priority patch in without him ever looking at it
15:36 <Sho_> There's precedent for this now, we can do stuff ourselves
15:36 <Sho_> In fact he explicitly said we'll have to because he has a kid on the way
15:36 <Sho_> So the lesson is
15:37 <Sho_> Don't wait for Martin to review things: Review things
15:37 <romangg> Sho_: Ok, but on some reviews he is also really needed.
15:37 <notmart> Sho_: sure you'll look like a male catwalk model :p
15:37 <Sho_> notmart: With the price of this suit, I better ...
15:37 <Sho_> (maybe I should but a corset)
15:37 <Sho_> *buy
15:38 <kbroulik> Sho_: so how did your fitting go?
15:38 <romangg> Ok, then we agree that we don't always wait on Martin to review and push. Only if we are sure it needs review by Martin write a mail to the mailing list for example.
15:39 <Sho_> kbroulik: nono, it's tomorrow
15:39 <romangg> Of course this could sometimes mean that we push something which is wrong and would have been spottet by Martin who knows most about KWin/KWayland. But we need to risk this then.
15:39 <Sho_> romangg: we need to, because the only way to build up the skills we need to be less dependent on Martin is to do work and learn by doing
15:40 <Sho_> Martin's review skills don't come out of a vacuum either
15:40 <notmart> http://i.dailymail.co.uk/i/pix/2017/11/30/01/46D29A2C00000578-5127635-Inspiration_Zoolander_is_about_two_outrageous_models_who_are_lur-m-71_1512003871477.jpg
15:40 <romangg> yea
15:40 <romangg> ok, next?
15:40 <Sho_> romangg: Honestly even if Martin were around fulltime this would be a good idea, KWin needs a higher bus number
15:40 <Sho_> It's a big app and needs a team to handle
15:41 <romangg> Sho_: totally agree
15:41 <romangg> and it's very central to lots of stuff in Plasma , with wayland even more.
15:41 <Sho_> And it's also an important skill-generator for Plasma in general, the things one learns working on KWin are hugely relevant for shell dev
15:42 <notmart> yeah, we must learn more of it
15:42 <Sho_> but yeah, let's move the meeting along before bshah hits us with a bat
15:42 <bshah> I am not going to do that
15:42 <bshah> :P
15:42 <notmart> right now probably david is the one who knows more, but we must catch up as well
15:42 <Riddell> * Plasma 5.13 beta on Thursday! announcement being written today! Get your features in pronto!
15:43 <Riddell> although I will be looking out for bug 394134
15:43 <sKreamer> KDE bug 394134 in plasmashell (general) "cursor theme KCM has labels on wrong side" [normal,WAITINGFORINFO] https://bugs.kde.org/show_bug.cgi?id=394134
15:45 <Sho_> kcm: your labels are on the wrong side ... of history *damdamdamdam*
15:46 <bshah> Riddell: done?
15:46 <krop> ← can reproduce
15:46 <Riddell> yep
15:46 <bshah> so assuming no-one is missing..
15:46 <bshah> it's my turn
15:46 <kbroulik> can reproduce on qt 5.10
15:46 <bshah> which is not interesting notes
15:46 <bshah> - Not much stuff done last week related to plasma
15:46 <bshah> - Was sick for almost half of week (and vacation on monday which turned out to be sickness leave), bloody summer!
15:48 <Riddell> so much sickness :(
15:48 <bshah> anyway, meeting is done