[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Minutes Monday Plasma Meeting
From:       Sebastian =?ISO-8859-1?Q?K=FCgler?= <sebas () kde ! org>
Date:       2017-01-16 12:13:33
Message-ID: 1522440.ppk8gGPZN6 () miro
[Download RAW message or body]

The minutes of today's meeting.
-- 
sebas

http://www.kde.org | http://vizZzion.org
["minutes-week-03.txt" (minutes-week-03.txt)]

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@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@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@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@frugalware/developer/crazy) has quit (Quit: \
Leaving) [12:24:01] <sebas> bshah's turn
[12:24:07] --> crazy (~crazy@x4e31437f.dyn.telefonica.de) has joined #plasma
[12:24:07] <-- crazy (~crazy@x4e31437f.dyn.telefonica.de) has quit (Changing host)
[12:24:07] --> crazy (~crazy@frugalware/developer/crazy) has joined #plasma
[12:24:08] --> kbroulik (~quassel@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@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!



[prev in list] [next in list] [prev in thread] [next in thread]