Minutes Monday Plasma Meeting

Mon Jan 16 12:13:33 UTC 2017

The minutes of today's meeting.
-- 
sebas

http://www.kde.org | http://vizZzion.org
-------------- next part --------------
Plasma Meeting minutes 16-01-2017

Present: Sho, d_ed, notmart, bshah, kbroulik, sebas

Sho:
* [General] Was able to clean out my fixes-and-polish queue neatly in 5.9 freeze week, will now spend more time on Yakuake rewrite again which needs to get back on track after falling out of my head during vacation etc.
* [General] Spent time mediating in the CI dispute on k-c-d and privately, making a policy proposal and sending some mails around everywhere
** Wiki page for policy: https://community.kde.org/Policies/Dependency_Changes_and_CI
* [FV] Implemented spring-loading on drag hover (folder preview popup opens when you hover a folder icon with a drag) and dropping into preview popups, waiting on Kai's review
* [FV] Fixed a crash during DND
* [TM] Lowered TM text label collapse threshold based on user feedback
* [TM] Working on critical regression breaking pinning with more than one activity
* [Dash] Fixed going from "All Applications" to "Widgets" not working
* [Dash] Fixed missing "All Widgets" entry
* [Launchers] Fixed a bug in the pagination code in Kicker::RootModel, now everything is ready for a Simple Menu KDE Store release coinciding w/ 5.9
* [Launchers] Reviewing Ivan's KA-based favorites work, other code reviews

d_ed:
* Fixed misc bugs all over the place
* Mostly appmenu
* Improved the way switching containments appears in the config UI - it's more seamless
* this week, I think priority will be continued bug fixing, then hopefully finishing off that kwin scaling patchset so it can be in 5.10

notmart:
* mostly fixed bugs, in particular:
* fixed bug in kpackagetool https://phabricator.kde.org/D4043 over install of packages with stuff in subdirectory
Diff 4043 "fix handling of archives with a folder in them" [Closed] https://phabricator.kde.org/D4043
* fixed bug in look and feel kcm: binding loop with freeze
* fixed a binding loop in plasma components tabbar
* fixed applying color schemes from look and feel kcm of schemes automatically downloaded from the KDE store
* further work on multiscreen fix: https://phabricator.kde.org/D3981 needs review asap, needs to be in the beta
Diff 3981 "better clean up of duplicate containments" [Closed] https://phabricator.kde.org/D3981
* kcoreaddons: parse desktop files 2 times as we must know the servicetypes *before* the actual parsing https://phabricator.kde.org/D4082 that triggered KCoreAddons 5.30.1
Diff 4082 "parse the desktop file 2 times" [Closed] https://phabricator.kde.org/D4082
* fixed wallpaper download bug: https://phabricator.kde.org/D4118 triggered knewstuuf 5.30.1
Diff 4118 "if is not an archive, always copy" [Needs Review] https://phabricator.kde.org/D4118
* further change, that needs review https://phabricator.kde.org/D4118
Diff 4118 "if is not an archive, always copy" [Needs Review] https://phabricator.kde.org/D4118
* support for user friendly name for categories in knewstuff/attica https://phabricator.kde.org/D4120 and https://phabricator.kde.org/D4121
Diff 4120 "add support for display_name in categories" [Closed] https://phabricator.kde.org/D4120
Diff 4121 "add support for display_name in categories" [Closed] https://phabricator.kde.org/D4121
* Kirigami 2.0 release https://dot.kde.org/2017/01/12/kde-releases-kirigami-ui-20
most reviews should already be closed, except one

bshah:
- AOSP based mobile images up
- working on new rootfs images currently

kbroulik:
* OSD (volume, brightness, etc) can now be disabled in settings for people who dislike it
* Renamed open() method in IconApplet to run() due to some MOC bug where it will generate a open64() method instead on 32 bit, breaking compilation, 5.9 beta was respun for this
* Audio indicator in task bar merged, global menu stuff merged \o/ for 5.9 beta, that is
* Various fixes

sebas:
- disabled logging to kscreen.log by default (in 5.8 and 5.9 / master)
- much unrelated stuff

Full log:

[11:59:49] <sebas> meeting!
[11:59:54] <sebas> Who's here for it?
[11:59:54] <Sho_> here
[11:59:56] <-- NicoRod (~nico at 190.189.52.104) has quit (Ping timeout: 240 seconds)
[11:59:56] <maelcum> KurousagiMK2: yep, reproduces on master
[12:00:09] <d_ed> here
[12:00:10] <sebas> http://whatthecommit.com/ << hilarious
[12:00:44] <notmart> tosky: yep, is in 5.8
[12:00:54] <notmart> sebas: yes, here!
[12:01:29] <notmart> " hahahahis branch is so dirty, even your mom can't clean it."
[12:01:57] <bshah> sebas: o/
[12:03:17] <sebas> alright ... we start with Sho!
[12:03:41] <Sho_> before I dump my notes: Everybody should keep in mind we have three open branches now, suitable fixes need to be put into 5.8 and merged up into both 5.9 and master, let's not forget the LTS
[12:03:53] <Sho_> (But I hope release-team remembers to merge 5.8 into 5.9 before making tars, because people sometimes get regrettably lazy and don't merge up their own stuff)
[12:04:06] <Sho_> With that out the way ...
[12:04:11] <sebas> yesyes
[12:04:15] <Sho_> * [General] Was able to clean out my fixes-and-polish queue neatly in 5.9 freeze week, will now spend more time on Yakuake rewrite again which needs to get back on track after falling out of my head during vacation etc.
[12:04:17] <Sho_> * [General] Spent time mediating in the CI dispute on k-c-d and privately, making a policy proposal and sending some mails around everywhere
[12:04:18] <Sho_> ** Wiki page for policy: https://community.kde.org/Policies/Dependency_Changes_and_CI
[12:04:20] <Sho_> * [FV] Implemented spring-loading on drag hover (folder preview popup opens when you hover a folder icon with a drag) and dropping into preview popups, waiting on Kai's review
[12:04:21] <Sho_> * [FV] Fixed a crash during DND
[12:04:23] <Sho_> * [TM] Lowered TM text label collapse threshold based on user feedback
[12:04:24] <Sho_> * [TM] Working on critical regression breaking pinning with more than one activity
[12:04:26] <Sho_> * [Dash] Fixed going from "All Applications" to "Widgets" not working
[12:04:27] <Sho_> * [Dash] Fixed missing "All Widgets" entry
[12:04:29] <Sho_> * [Launchers] Fixed a bug in the pagination code in Kicker::RootModel, now everything is ready for a Simple Menu KDE Store release coinciding w/ 5.9
[12:04:30] <Sho_> * [Launchers] Reviewing Ivan's KA-based favorites work, other code reviews
[12:04:52] <Sho_> next() :)
[12:05:11] <sebas> what's the yakuake rewrite going to bring us? :)
[12:05:24] <sebas> and thanks for fire-fighting the CI discussion!
[12:05:50] <Sho_> sebas: Mostly a new theming system that does stuff the themers wanted to do a long time like more flexible layouts, and fundamentals like hi-dpi support
[12:06:01] <Riddell> Sho_: hang on
[12:06:11] <Sho_> It's in some sense a mini-plasma tech wise, themes are in kpackages etc
[12:06:16] <sebas> aye nice!
[12:06:22] <Riddell> Sho_: I've made no plans to merge before making tars and I don't do that for the normal releases, should I really do that for 5.8?
[12:06:35] <sebas> for 5.9
[12:06:36] <Riddell> that seems likely to cause last minute breakage
[12:06:41] <Sho_> Riddell: IMHO you should always do that, otherwise you release stuff that's missing fixes people expect to be in
[12:06:45] <sebas> merge everything in 5.8 into 5.9 branch
[12:06:56] <Sho_> Riddell: It's not oyur fault because people should really merge up themselves instead of relying on others to do it for them
[12:07:21] <Sho_> Riddell: but in practice people are often too lazy - whenever i merge 5.x into master (or noe 5.8 into 5.9), I see a few fixes come along that other people authored and didn't merge up
[12:07:29] <Riddell> sebas: and for 5.9 into master too?
[12:07:31] <Sho_> noe=now
[12:07:56] <Sho_> basically: people, be nice to your colleagues and merge up so you don't saddle other people with conflicts
[12:07:58] <sebas> yes
[12:08:11] <Sho_> but release-team should still double-check at release time
[12:08:25] -*- Riddell looks around for the rest of the release team
[12:08:29] <Sho_> :)
[12:08:53] <Riddell> Sho_: it makes sense but I'm still very scared the merge would break something which I'd then make tars with
[12:09:29] <Sho_> Riddell: I understand that fear, but I also fear that last-minute fixes go into 5.8 and then don't get merged into 5.9
[12:09:41] <Sho_> so there's some risk both ways
[12:09:43] <Sho_> tricky, not sure
[12:09:50] --> yossarianuk (~morgan at 178.213.73.251) has joined #plasma
[12:10:03] <Riddell> ok, I'll add to notes and give it a try for 5.9.1
[12:10:05] <Sho_> we could do something like "release-team does one merge-up 24h before, if you forget to merge up yourself after it's your fault"
[12:10:18] <Riddell> maybe doing the merge a day early to let kde ci and neon ci build it
[12:10:22] <yossarianuk> hi - running kubuntu 16.10 + Plasma 5.8 PPA - how can I change pidgin notification volume ?
[12:10:23] <sebas> sounds complicated
[12:10:36] <yossarianuk> i.e when I get a notication I can see the volume in the application volume  in 'audio volume' however as soon as the sound finishes I cannot see it any longer and when another sound occurs  it goes to the default volume
[12:10:43] <Sho_> yossarianuk: heya, mind waiting a bit or asking in #kde? we're in the middle of a meeting
[12:10:43] <sebas> yossarianuk: we're amidst a meeting, could you come back in like half an hour?
[12:10:51] <Sho_> sorry~
[12:10:53] <tosky> I was going to suggest the same (ensure that everything is merged one day before)
[12:10:55] <yossarianuk> ok - np
[12:11:04] <yossarianuk> didn't realise...
[12:11:13] <sebas> no problem :)
[12:11:27] <Riddell> tosky: how would it affect translations? if a string from 5.8 which was removed from 5.9 but merged back in
[12:11:46] <tosky> Riddell: you should not have new strings in 5.8 to starts with
[12:11:56] <-- yossarianuk (~morgan at 178.213.73.251) has left #plasma ("Konversation terminated!")
[12:11:58] <Riddell> not new but removed from 5.9
[12:12:02] <Riddell> corner case I suppose but still
[12:12:09] <Sho_> Riddell: merging a new string back in shouldn't happen though
[12:12:26] <Sho_> for code that would be an incorrect resolve, and for .desktops we should always take the newer branch ver
[12:12:26] <Riddell> ok
[12:12:32] <tosky> yep
[12:12:47] <sebas> let's move on though
[12:12:52] <sebas> d_ed's turn!
[12:14:13] <Sho_> that's a really long dramatic pause d_ed
[12:14:15] <Sho_> :P
[12:14:19] <d_ed> building tension
[12:14:28] <d_ed> Fixed misc bugs all over the place
[12:14:32] <d_ed> Mostly appmenu
[12:14:49] <d_ed> Improved the way switching containments appears in the config UI - it's more seamless
[12:14:54] <Riddell> d_ed: does it work in a standalone widget?
[12:15:19] <d_ed> Riddell: it will before 5.9.0
[12:16:22] <d_ed> this week, I think priority will be continued bug fixing, then hopefully finishing off that kwin scaling patchset so it can be in 5.10
[12:16:33] <d_ed> and that's me done
[12:17:50] <sebas> notmart's turn
[12:17:54] <notmart> ok
[12:18:21] <notmart> * mostly fixed bugs, in particular:
[12:18:22] <notmart> * fixed bug in kpackagetool https://phabricator.kde.org/D4043 over install of packages with stuff in subdirectory
[12:18:23] <sKreamer> Diff 4043 "fix handling of archives with a folder in them" [Closed] https://phabricator.kde.org/D4043
[12:18:24] <notmart> * fixed bug in look and feel kcm: binding loop with freeze
[12:18:25] <notmart> * fixed a binding loop in plasma components tabbar
[12:18:27] <notmart> * fixed applying color schemes from look and feel kcm of schemes automatically downloaded from the KDE store
[12:18:28] <notmart> * further work on multiscreen fix: https://phabricator.kde.org/D3981 needs review asap, needs to be in the beta
[12:18:29] <sKreamer> Diff 3981 "better clean up of duplicate containments" [Closed] https://phabricator.kde.org/D3981
[12:18:30] <notmart> * kcoreaddons: parse desktop files 2 times as we must know the servicetypes *before* the actual parsing https://phabricator.kde.org/D4082 that triggered KCoreAddons 5.30.1
[12:18:30] <sKreamer> Diff 4082 "parse the desktop file 2 times" [Closed] https://phabricator.kde.org/D4082
[12:18:31] <notmart> * fixed wallpaper download bug: https://phabricator.kde.org/D4118 triggered knewstuuf 5.30.1
[12:18:32] <sKreamer> Diff 4118 "if is not an archive, always copy" [Needs Review] https://phabricator.kde.org/D4118
[12:18:33] <notmart> * further change, that needs review https://phabricator.kde.org/D4118
[12:18:33] <sKreamer> Diff 4118 "if is not an archive, always copy" [Needs Review] https://phabricator.kde.org/D4118
[12:18:34] <notmart> * support for user friendly name for categories in knewstuff/attica https://phabricator.kde.org/D4120 and https://phabricator.kde.org/D4121
[12:18:35] <sKreamer> Diff 4120 "add support for display_name in categories" [Closed] https://phabricator.kde.org/D4120
[12:18:35] <sKreamer> Diff 4121 "add support for display_name in categories" [Closed] https://phabricator.kde.org/D4121
[12:18:36] <notmart> * Kirigami 2.0 release https://dot.kde.org/2017/01/12/kde-releases-kirigami-ui-20
[12:18:40] <notmart> most reviews should already be closed, except one
[12:19:21] <notmart> that is, D4118
[12:19:21] <sKreamer> Diff 4118 "if is not an archive, always copy" [Needs Review] https://phabricator.kde.org/D4118
[12:19:47] <notmart> about the multiscreen fix, unfortunately there are already reports of things still being broken
[12:19:59] <notmart> so this time the illusion was short lived
[12:20:03] <notmart> but at this point.. no idea
[12:20:12] <notmart> then that's it from me
[12:20:42] <Sho_> what's the chance we can deliver an electric charge over hdmi when people try to connect another monitor?
[12:21:16] <notmart> i would *love* that :)
[12:21:44] <sebas> I'd have implemented that long ago if it were possible
[12:21:52] <sebas> we can't even reliably fry these things
[12:22:03] <einar77_work> (where can I post a BT of a deadlock I'm getting within minutes withotu disrupting the meeting?)
[12:22:15] <Sho_> einar77_work: in the future
[12:22:16] <Sho_> :P
[12:22:18] <bshah> einar77_work: bugs.ko? ;)
[12:22:19] <notmart> an hdmi version of the etherkiller :p http://etherkiller.org/img/etherkiller.jpg
[12:22:32] <einar77_work> Sho_: well, good idea :P
[12:22:50] <-- crazy_ (~crazy at frugalware/developer/crazy) has quit (Quit: Leaving)
[12:24:01] <sebas> bshah's turn
[12:24:07] --> crazy (~crazy at x4e31437f.dyn.telefonica.de) has joined #plasma
[12:24:07] <-- crazy (~crazy at x4e31437f.dyn.telefonica.de) has quit (Changing host)
[12:24:07] --> crazy (~crazy at frugalware/developer/crazy) has joined #plasma
[12:24:08] --> kbroulik (~quassel at kde/broulik) has joined #plasma
[12:24:20] <bshah> okay
[12:24:32] <bshah> not much updates..
[12:24:35] <bshah> - AOSP based mobile images up
[12:24:37] <bshah> - working on new rootfs images currently
[12:24:40] <bshah> done()
[12:24:45] <kbroulik> moin
[12:24:51] <kbroulik> meeting still underway?
[12:24:58] <notmart> kbroulik: yep
[12:25:04] <notmart> (and you can still join :)
[12:25:08] <bshah> (also not to mention.. last week was terrible wrt AOSP v/s CM v/s LineageOS discussion)
[12:25:59] <notmart> aosp is the new chemtrails
[12:26:32] <sebas> :D
[12:26:38] <sebas> kbroulik: you're ON!
[12:26:53] <notmart> we simply can't be bullied into doing stupid things by incredibly uninformed trolls
[12:26:54] <kbroulik> * OSD (volume, brightness, etc) can now be disabled in settings for people who dislike it
[12:26:54] <kbroulik> * Renamed open() method in IconApplet to run() due to some MOC bug where it will generate a open64() method instead on 32 bit, breaking compilation, 5.9 beta was respun for this
[12:26:54] <kbroulik> * Audio indicator in task bar merged, global menu stuff merged \o/ for 5.9 beta, that is
[12:26:54] <kbroulik> * Various fixes
[12:27:28] <[Arfrever]> d_ed: Have you seen my suggestion (in this channel)?
[12:27:31] <sebas> cool stuff, as always!
[12:27:31] <Sho_> kbroulik: you missed ‎[20:04] ‎<‎Sho_‎>‎ * [FV] Implemented spring-loading on drag hover (folder preview popup opens when you hover a folder icon with a drag) and dropping into preview popups, waiting on Kai's review
[12:27:34] <Sho_> :)
[12:28:15] <kbroulik> Sho_: yup, let me just grab a Club Mate and then I'll go through the backlog ;)
[12:28:18] <kbroulik> next()
[12:28:30] <Sho_> thankies
[12:28:46] <sebas> - disabled logging to kscreen.log by default (in 5.8 and 5.9 / master)
[12:28:46] <sebas> - much unrelated stuff
[12:28:56] <sebas> shorty from my side
[12:29:04] <sebas> and with that ... we're done for today
[12:29:12] <sebas> happy hacking and see you next week!
[12:29:19] <d_ed> [Arfrever]: no, i don't think so
[12:29:19] <Sho_> thanks for hosting!
[12:29:22] <[Arfrever]> d_ed: "[2017-01-12 20:08 UTC] <[Arfrever]> d_ed: https://cgit.kde.org/plasma-integration.git/commit/?id=14a546e53bfa48aa165eca65238b4963e47636ee seems to have useless debugging line: message("DAVE ${Qt5Core_VERSION}"). Plasma/5.9 and master branches should be fixed."
[12:29:23] <bshah> thanks for hosting!
[12:29:37] <sebas> you're welcome
[12:29:52] <[Arfrever]> d_ed: CMakeLists.txt, second hunk.
[12:30:21] <d_ed> yeah I see it, thanks
[12:30:21] <pursuivant> plasma-desktop (Plasma/5.8) v5.8.5-12-g4af59a37 * Eike Hein: containments/desktop/package/contents/ui/FolderView.qml
[12:30:22] <pursuivant> Silence warning.
[12:30:22] <pursuivant> https://commits.kde.org/plasma-desktop/4af59a374bce56ee4be273b8397aff28953d483d
[12:30:52] <tosky> I think I already asked, but did you consider using something like meetbot or similar to host the IRC meetings?
[12:30:59] <pursuivant> plasma-desktop (Plasma/5.9) v5.8.95-14-g9eeac2fd * Eike Hein: containments/desktop/package/contents/ui/FolderView.qml
[12:30:59] <pursuivant> Merge branch 'Plasma/5.8' into Plasma/5.9
[12:31:00] <pursuivant> https://commits.kde.org/plasma-desktop/9eeac2fdfdd59f667c984ef3651ecfefc5648332
[12:31:10] <bshah> tosky: we did.. kinda
[12:31:18] <bshah> then it ping'd all people in channel
[12:31:19] <tosky> bshah: kinda? :)
[12:31:20] <bshah> :P
[12:31:29] <tosky> bshah: it does not do it by default
[12:31:41] <bshah> not sure what kind of bot that was though
[12:31:49] <tosky> or maybe it depends on which bot and the configuration, but I use it for work $elsewhere and there is no ping
[12:31:52] <tosky> I mean, I would reconsider
[12:32:00] <sebas> I haven't looked into meetbot
[12:32:05] <d_ed> [Arfrever]:fixed
[12:32:07] <d_ed> thanks
[12:32:40] --> NicoRod (~nico at 190.189.52.104) has joined #plasma
[12:33:30] <tosky> sebas, bshah: this is what I use at work: http://docs.openstack.org/infra/system-config/irc.html#meetbot
[12:33:41] <tosky> well, used for all openstack meetings
[12:34:21] <sebas> tosky: I'll have a look, thanks!