D27470: Set good default value for cursor theme and size in KRDB

[Benjamin Port]

This revision was automatically updated to reflect the committed changes.
Closed by commit R119:ca9f0eeb9b53: Set good default value for cursor theme and 
size in KRDB (authored by bport).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27470?vs=75892=75962

REVISION DETAIL
  https://phabricator.kde.org/D27470

AFFECTED FILES
  kcms/krdb/krdb.cpp

To: bport, #plasma, broulik, ervin, meven, crossi, davidedmundson
Cc: ngraham, davidre, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27329: Make Taskmanager's tooltip a fixed ratio rectangle

[Tranter Madi]

trmdi added a comment.


  In D27329#611907 , @mvourlakos 
wrote:
  
  > In D27329#611906 , @trmdi wrote:
  >
  > > How about this?
  > >  F8104781: image.png 
  >
  >
  > You can also try the GoldNumber/Ratio that is found in all real nature 
patterns and also at Ancient Greeks in their architectural buildings. That is 
1.618, there is a big chance that it feels right without being able to explain 
why.
  
  
  But others don't like a wide thumbnail. :(

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27329

To: trmdi, #vdg, #plasma, ngraham
Cc: mvourlakos, filipf, ngraham, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, ahiemstra, mart

D27329: Make Taskmanager's tooltip a fixed ratio rectangle

[Tranter Madi]

trmdi updated this revision to Diff 75959.
trmdi added a comment.


  - Increase the height a bit more, the last effort.

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27329?vs=75486=75959

BRANCH
  adjust-size-of-tooltip (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27329

AFFECTED FILES
  applets/taskmanager/package/contents/ui/ToolTipInstance.qml

To: trmdi, #vdg, #plasma, ngraham
Cc: mvourlakos, filipf, ngraham, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, ahiemstra, mart

KDE CI: Plasma » plasma-desktop » stable-kf5-qt5 SUSEQt5.12 - Build # 266 - Fixed!

[CI System]

BUILD SUCCESS
 Build URL
https://build.kde.org/job/Plasma/job/plasma-desktop/job/stable-kf5-qt5%20SUSEQt5.12/266/
 Project:
stable-kf5-qt5 SUSEQt5.12
 Date of build:
Tue, 18 Feb 2020 23:37:10 +
 Build duration:
7 min 0 sec and counting
   JUnit Tests
  Name: (root) Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)Name: projectroot.containments.desktop.plugins.folder Failed: 0 test(s), Passed: 4 test(s), Skipped: 0 test(s), Total: 4 test(s)Name: projectroot.kcms.keyboard Failed: 0 test(s), Passed: 4 test(s), Skipped: 0 test(s), Total: 4 test(s)Name: projectroot.kcms.kfontinst.kio Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)Name: projectroot.kcms.lookandfeel Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)
   Cobertura Report
  
   Project Coverage Summary
  
   Name
  PackagesFilesClassesLinesConditionalsCobertura Coverage Report15%
(12/79)17%
(62/364)17%
(62/364)13%
(4136/31492)10%
(1929/19027)Coverage Breakdown by Package
Name
   FilesClassesLinesConditionalsapplets.kimpanel.backend.ibus0%
(0/1)0%
(0/1)0%
(0/30)0%
(0/10)applets.kimpanel.backend.ibus.emojier0%
(0/1)0%
(0/1)0%
(0/187)0%
(0/82)applets.kimpanel.backend.ibus.ibus150%
(0/10)0%
(0/10)0%
(0/1098)0%
(0/620)applets.kimpanel.backend.scim0%
(0/1)0%
(0/1)0%
(0/663)0%
(0/395)applets.kimpanel.plugin0%
(0/2)0%
(0/2)0%
(0/43)0%
(0/26)applets.pager.plugin0%
(0/3)0%
(0/3)0%
(0/356)0%
(0/234)applets.showdesktop.plugin0%
(0/3)0%
(0/3)0%
(0/14)0%
(0/4)applets.taskmanager.plugin0%
(0/3)0%
(0/3)0%
(0/320)0%
(0/244)applets.taskmanager.plugin.smartlaunchers0%
(0/4)0%
(0/4)0%
(0/235)0%
(0/208)applets.trash.plugin0%
(0/5)0%
(0/5)0%
(0/116)0%
(0/52)attica-kde.kdeplugin0%
(0/1)0%
(0/1)0%
(0/125)0%
(0/118)containments.desktop.plugins.desktop0%
(0/2)0%
(0/2)0%
(0/48)0%
(0/18)containments.desktop.plugins.folder30%
(6/20)30%
(6/20)39%
(914/2350)28%
(419/1501)containments.desktop.plugins.folder.autotests100%
(4/4)100%
(4/4)100%
(558/558)60%
(221/368)dataengines.kimpanel0%
(0/7)0%
(0/7)0%
(0/343)0%
(0/131)imports.activitymanager0%
(0/3)0%

KDE CI: Plasma » kde-cli-tools » stable-kf5-qt5 SUSEQt5.12 - Build # 59 - Still unstable!

[CI System]

BUILD UNSTABLE
 Build URL
https://build.kde.org/job/Plasma/job/kde-cli-tools/job/stable-kf5-qt5%20SUSEQt5.12/59/
 Project:
stable-kf5-qt5 SUSEQt5.12
 Date of build:
Tue, 18 Feb 2020 23:37:14 +
 Build duration:
1 min 50 sec and counting
   JUnit Tests
  Name: (root) Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)Name: projectroot.keditfiletype Failed: 1 test(s), Passed: 0 test(s), Skipped: 0 test(s), Total: 1 test(s)Failed: projectroot.keditfiletype.tests.filetypestest
   Cobertura Report
  
   Project Coverage Summary
  
   Name
  PackagesFilesClassesLinesConditionalsCobertura Coverage Report20%
(2/10)15%
(4/26)15%
(4/26)19%
(493/2587)14%
(212/1505)Coverage Breakdown by Package
Name
   FilesClassesLinesConditionalskbroadcastnotification0%
(0/1)0%
(0/1)0%
(0/49)0%
(0/22)kcmshell0%
(0/2)0%
(0/2)0%
(0/165)0%
(0/86)kdesu0%
(0/2)0%
(0/2)0%
(0/276)0%
(0/192)keditfiletype23%
(3/13)23%
(3/13)23%
(285/1249)20%
(136/664)keditfiletype.tests100%
(1/1)100%
(1/1)81%
(208/257)41%
(76/186)kioclient0%
(0/2)0%
(0/2)0%
(0/259)0%
(0/138)kmimetypefinder0%
(0/1)0%
(0/1)0%
(0/33)0%
(0/10)kstart0%
(0/2)0%
(0/2)0%
(0/220)0%
(0/168)ksvgtopng0%
(0/1)0%
(0/1)0%
(0/16)0%
(0/4)ktraderclient0%
(0/1)0%
(0/1)0%
(0/63)0%
(0/35)

KDE CI: Plasma » kde-cli-tools » stable-kf5-qt5 FreeBSDQt5.13 - Build # 36 - Still unstable!

[CI System]

BUILD UNSTABLE
 Build URL
https://build.kde.org/job/Plasma/job/kde-cli-tools/job/stable-kf5-qt5%20FreeBSDQt5.13/36/
 Project:
stable-kf5-qt5 FreeBSDQt5.13
 Date of build:
Tue, 18 Feb 2020 23:37:15 +
 Build duration:
1 min 52 sec and counting
   JUnit Tests
  Name: projectroot.keditfiletype Failed: 1 test(s), Passed: 0 test(s), Skipped: 0 test(s), Total: 1 test(s)Failed: projectroot.keditfiletype.tests.filetypestest

KDE CI: Plasma » systemsettings » stable-kf5-qt5 SUSEQt5.12 - Build # 100 - Fixed!

[CI System]

BUILD SUCCESS
 Build URL
https://build.kde.org/job/Plasma/job/systemsettings/job/stable-kf5-qt5%20SUSEQt5.12/100/
 Project:
stable-kf5-qt5 SUSEQt5.12
 Date of build:
Tue, 18 Feb 2020 23:37:08 +
 Build duration:
1 min 40 sec and counting
   JUnit Tests
  Name: (root) Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)
   Cobertura Report
  
   Project Coverage Summary
  
   Name
  PackagesFilesClassesLinesConditionalsCobertura Coverage Report0%
(0/6)0%
(0/18)0%
(0/18)0%
(0/1495)0%
(0/651)Coverage Breakdown by Package
Name
   FilesClassesLinesConditionalsapp0%
(0/3)0%
(0/3)0%
(0/274)0%
(0/128)app.ToolTips0%
(0/1)0%
(0/1)0%
(0/100)0%
(0/27)core0%
(0/9)0%
(0/9)0%
(0/524)0%
(0/252)icons0%
(0/3)0%
(0/3)0%
(0/122)0%
(0/32)sidebar0%
(0/1)0%
(0/1)0%
(0/367)0%
(0/179)sidebar.ToolTips0%
(0/1)0%
(0/1)0%
(0/108)0%
(0/33)

KDE CI: Plasma » powerdevil » stable-kf5-qt5 SUSEQt5.12 - Build # 53 - Fixed!

[CI System]

BUILD SUCCESS
 Build URL
https://build.kde.org/job/Plasma/job/powerdevil/job/stable-kf5-qt5%20SUSEQt5.12/53/
 Project:
stable-kf5-qt5 SUSEQt5.12
 Date of build:
Tue, 18 Feb 2020 23:37:09 +
 Build duration:
1 min 42 sec and counting
   JUnit Tests
  Name: (root) Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)
   Cobertura Report
  
   Project Coverage Summary
  
   Name
  PackagesFilesClassesLinesConditionalsCobertura Coverage Report0%
(0/8)0%
(0/65)0%
(0/65)0%
(0/4082)0%
(0/2646)Coverage Breakdown by Package
Name
   FilesClassesLinesConditionalsdaemon0%
(0/17)0%
(0/17)0%
(0/1444)0%
(0/1063)daemon.actions.bundled0%
(0/18)0%
(0/18)0%
(0/823)0%
(0/478)daemon.actions.dpms0%
(0/9)0%
(0/9)0%
(0/237)0%
(0/122)daemon.backends.upower0%
(0/13)0%
(0/13)0%
(0/1026)0%
(0/750)kcmodule.activities0%
(0/2)0%
(0/2)0%
(0/198)0%
(0/93)kcmodule.common0%
(0/3)0%
(0/3)0%
(0/156)0%
(0/72)kcmodule.global0%
(0/1)0%
(0/1)0%
(0/80)0%
(0/22)kcmodule.profiles0%
(0/2)0%
(0/2)0%
(0/118)0%
(0/46)

D27462: Include comment for translators for Power Down

[Aleix Pol Gonzalez]

This revision was automatically updated to reflect the committed changes.
Closed by commit R122:6d063a82ca2a: Include comment for translators for Power 
Down (authored by apol).

REPOSITORY
  R122 Powerdevil

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27462?vs=75836=75947

REVISION DETAIL
  https://phabricator.kde.org/D27462

AFFECTED FILES
  daemon/actions/bundled/handlebuttonevents.cpp

To: apol, broulik, aacid
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27422: [KCM]Update Device item layout based on applet

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> DeviceListItem.qml:32
>  id: delegate
> +spacing: units.smallSpacing * 2
>  width: parent.width

Kirigami.Units.smallSpacing please

(for all units in KCM code)

again, for complex reasons, sorry.

> DeviceListItem.qml:106
> +spacing: units.smallSpacing
> +Layout.preferredWidth: parent.width
> +

Do you need this?

It should be the same as Layout.fillWidth which will be on by default for a 
layout in a layout

REPOSITORY
  R115 Plasma Audio Volume Applet

BRANCH
  kcmports2

REVISION DETAIL
  https://phabricator.kde.org/D27422

To: gvgeo, #plasma, drosca, ngraham, #vdg
Cc: davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27422: [KCM]Update Device item layout based on applet

[Nathaniel Graham]

ngraham added a comment.


  You might be hitting https://bugs.kde.org/show_bug.cgi?id=356446

REPOSITORY
  R115 Plasma Audio Volume Applet

BRANCH
  kcmports2

REVISION DETAIL
  https://phabricator.kde.org/D27422

To: gvgeo, #plasma, drosca, ngraham, #vdg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Plasma 5.18.1

[Jonathan Riddell]

Plasma 5.18.1 is now released
https://www.kde.org/announcements/plasma-5.18.1.php

Re: 2 kirigami fixes for a point release

[Nate Graham]

On 2/18/20 1:37 PM, Albert Astals Cid wrote:

I still don't see why this is a problem, as said Plasma depends on a myriad of 
libraries that are building each with their own release model, most probably 
with no bugfix releases at all either.


The "we don't control the whole stack" argument does not apply to parts 
of the stack that we do control. Improvement is possible even when 
perfection is not.




Incidentally what happens is that those libraries are not buggy, and it seems 
the Plasma-facing parts of KF5 are, well, let's make them not be.


Agreed. Everyone wants less buggy releases.

However "less buggy releases" does not fully solve the problem for LTS 
distros that freeze their KF version. Without point releases of the 
version they freeze on, we are unable to ship fixes for regressions that 
do sneak in, and we are unable to ship fixes for old or longstanding 
issues that we find a way to fix later. We can do both of these things 
with Plasma. We cannot do either with Frameworks. That's the problem.


Ultimately I think we need to decide whether we want to fully support 
the Plasma LTS or can it. Right now we're in this awkward position where 
we can hand packagers tarballs with bugfix point releases of Plasma, but 
not Frameworks. Ultimately this means that there's a class of bug that 
just doesn't get fixed in the distros with LTS Plasma, which in the end 
makes us look bad.


Nate

Re: 2 kirigami fixes for a point release

[Nate Graham]

On 2/18/20 2:13 PM, Luca Beltrame wrote:

In data martedì 18 febbraio 2020 19:26:21 CET, Nate Graham ha scritto:


Neon is already an OS, whether or not you want to admit it. It's
installed from an ISO. A hardware vendor (Slimbook) is shipping it on


Erm, where did I say that in my reply? ;)  I merely say that going "Neon or
else unsupported" is a very downstream-hostile attitude,


...And where did I say that in my reply? ;) I think we should absolutely 
continue to support all downstreams, not just Neon; that would be crazy! 
:) This is in fact why I'm advocating for an LTS Frameworks release to 
accompany the Plasma LTS release: to better support our non-Neon 
downstreams who want to freeze on the Plasma LTS releases. Right now 
we're pushing the job of backporting bugfixes to Frameworks onto them, 
rather than making it easy by just giving them tarballs. We can argue 
about what packagers should do, but the best way to get them to do that 
is to make it easy for them. :)




(Neon suffers from the same "LTS problem" for everything that's not KDE-made
software, FTR, but that's not the issue I want to raise here)


Indeed, and that's the reason why I'm happy with Tumbleweed. I'm quite 
on board with the seemingly prevailing opinion that discrete LTS 
releases amount to a broken, defective model. It's just that for the 
time being we have a product explicitly catering to distros using that 
broken, defective model. It just seems like an awkward situation that 
should be addressed one way or another, not left in tension indefinitely.


Nate

Re: 2 kirigami fixes for a point release

[Luca Beltrame]

In data martedì 18 febbraio 2020 19:26:21 CET, Nate Graham ha scritto:

> Neon is already an OS, whether or not you want to admit it. It's
> installed from an ISO. A hardware vendor (Slimbook) is shipping it on

Erm, where did I say that in my reply? ;)  I merely say that going "Neon or 
else unsupported" is a very downstream-hostile attitude, already seen with 
other software ("my way or the highway" in these cases) such as Audacity or 
Anki (the latter being one of the most downstream-hostile projects I've ever 
known). 

(Neon suffers from the same "LTS problem" for everything that's not KDE-made 
software, FTR, but that's not the issue I want to raise here)


-- 
Luca Beltrame - KDE Forums team
GPG key ID: A29D259B

signature.asc
Description: This is a digitally signed message part.

D27462: Include comment for translators for Power Down

[Albert Astals Cid]

aacid added a comment.


  i think it's better than nothing yes

REPOSITORY
  R122 Powerdevil

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D27462

To: apol, broulik, aacid
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Re: 2 kirigami fixes for a point release

[Albert Astals Cid]

El dimarts, 18 de febrer de 2020, a les 4:03:05 CET, Nate Graham va escriure:
> On 2020-02-16 14:43, Albert Astals Cid wrote:
> > Maybe i explain myself wrongly, i'm not blaming distros at all.
> > 
> > They made a decision, we/I may agree with them or not, that's *my/our* 
> > problem, what I was disagreeing is to us having to do extra work because 
> > someone elses (the distros) decision.
> 
> We already do this: it's called the Plasma LTS product. :-) It's been 
> specifically created to cater to various distros' desires for an 
> extended-support product they can ship to users of their own LTS releases.
> 
> But yes, I can also agree with more tests, better stability, moving to 
> GitLab so tests are run before merging, etc. I think we can all agree 
> with that.
> 
> However all the autotests in the world will not resolve the fundamental 
> incompatibility between the Plasma LTS product, which is built around 
> the release model of extended, ongoing bugfix releases, and Frameworks, 
> which is built around a rolling release model with no bugfix releases at 
> all.

I still don't see why this is a problem, as said Plasma depends on a myriad of 
libraries that are building each with their own release model, most probably 
with no bugfix releases at all either.

Incidentally what happens is that those libraries are not buggy, and it seems 
the Plasma-facing parts of KF5 are, well, let's make them not be.

Cheers,
  Albert

D27422: [KCM]Update Device item layout based on applet

[George Vogiatzis]

gvgeo added a comment.


  Although this is a general problem, just to be clear. Without Plasma 
Components I don't get any scaling.
  F8111566: Screenshot_20200218_214017.png 


REPOSITORY
  R115 Plasma Audio Volume Applet

BRANCH
  kcmports2

REVISION DETAIL
  https://phabricator.kde.org/D27422

To: gvgeo, #plasma, drosca, ngraham, #vdg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[George Vogiatzis]

gvgeo added a comment.


  It was more of a general question...
  
  Anyway I tried, nothing happens. I Believe it uses the qmlc files.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg, apol, ngraham, davidedmundson
Cc: davidedmundson, apol, ngraham, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[David Edmundson]

davidedmundson added a comment.


  I don't know
  
  You can find
  
  ./qml/QtGraphicalEffects/private/DropShadowBase.qml
  
  somewhere in /usr on your local system,
  
  add in some
  console.log lines and see what's happening.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg, apol, ngraham, davidedmundson
Cc: davidedmundson, apol, ngraham, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[George Vogiatzis]

gvgeo added a comment.


  I use arch's Qt 5.14.1
  Any chance I only have this problem? My build is not exactly stable.
  
  Btw found some places that use radius spallSpacing or gridUnit, like 
knewstuff. I wonder how that look, with that commit.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg, apol, ngraham, davidedmundson
Cc: davidedmundson, apol, ngraham, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[David Edmundson]

davidedmundson added a comment.


  That commit has been there since 5.12.5
  
  So I imagine you have it on your system. 
  It's possible it doesn't work correctly, but it needs some investigation.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg, apol, ngraham, davidedmundson
Cc: davidedmundson, apol, ngraham, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[David Edmundson]

davidedmundson added a comment.


  See 
https://code.qt.io/cgit/qt/qtgraphicaleffects.git/commit/src/effects/private/DropShadowBase.qml?id=f9664053132affe48a069235eb6561a32d38b2a7
  
  You're about double it twice

REPOSITORY
  R119 Plasma Desktop

BRANCH
  shadow2 (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg, apol, ngraham
Cc: davidedmundson, apol, ngraham, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, ahiemstra, mart

Re: 2 kirigami fixes for a point release

[David Edmundson]

> > IMHO distributions using Plasma LTS, Plasma team & other stakeholders should
> > team up here and maintain a matching LTS branch of Frameworks together at 
> > the
> > central KDE repos together. Well, and a version also satisfying other 
> > clients
> > of KF, like non-workspace applications from KDE.
> >

Just to  clarify something. "the Plasma team" is not unified in the
position that there should be any changes to the current framework's
release cycle.

> So perhaps we could say that the KF version which
happens to be the dependency for a Plasma LTS release could have bugfix
releases?

Note that next Plasma LTS is a different case anyway as KF6 will be
well into development and KF5 branches will be somewhat frozen. The
exact details of which is another discussion.

David

D27362: Linux/cpuinfo.c: grow buffer size as needed for 12+ core CPUs

[Nathaniel Graham]

ngraham added a comment.


  One would expect that, yes. But one would be expecting too much of 
Phabricator. :)

REPOSITORY
  R106 KSysguard

REVISION DETAIL
  https://phabricator.kde.org/D27362

To: jakobkummerow, davidedmundson, ahiemstra
Cc: ngraham, ahiemstra, cfeck, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart

D27316: [Folder view]Make shadow scale aware

[Nathaniel Graham]

ngraham accepted this revision.
ngraham added a comment.


  So I guess we do need to do this in all places where we use drop shadows, 
urgh.
  
  Any chance you'd be interested in tackling that, @gvgeo?
  
  As for this patch, seems like a candidate for the stable branch.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  shadow2 (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg, apol, ngraham
Cc: apol, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D27408: Fix build with Qt 5.15.0

[Nathaniel Graham]

ngraham added a comment.


  Yes I at least look at all new patches for Plasma, Frameworks, and many KDE 
apps. If I'm capable of reviewing them, I do. If not, I try to add people to 
the reviewers list who I think can, if they're not already marked as reviewers.
  
  We can add you as a reviewer for porting/Qt-related patches, sure.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

REVISION DETAIL
  https://phabricator.kde.org/D27408

To: apol, #plasma
Cc: ngraham, mlaurent, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27408: Fix build with Qt 5.15.0

[Laurent Montel]

mlaurent added a comment.


  So you review all patch from all repo ? :) awesome ! I am not able to do it! 
And I don't want to do it otherwise no possible to work on my projects.
  But I am happy that you do it ! :)
  
  So for sure I will not review all patchs from framework/plasma, but I am not 
against for reviewing qt5.15 patch.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

REVISION DETAIL
  https://phabricator.kde.org/D27408

To: apol, #plasma
Cc: ngraham, mlaurent, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

Re: 2 kirigami fixes for a point release

[Nate Graham]

On 2/16/20 2:55 PM, Friedrich W. H. Kossebau wrote:

Yes, this has been questioned a few times. Also seeing Plasma LTS used
together with a non-LTS Qt is a bit strange.
But somehow it seems there has not been enough pain for those using the Plasma
LTS to change something. Possibly because distributions simply backport
important bug fixes for KF themselves, kind of maintaining their own KF LTS
version of the KF version they pinpointed to when they froze the ingredients
to their OS. Because they are used to do this for other projects as well, and
so miss this could be done in cooperation with upstream.


There has been pain. This thread mentions a number of examples, and 
There were quite a few for the last 5.12 LTS too. But more generally, 
the pain is baked into Frameworks due to the lack of any bugfix 
releases. For example Kubuntu 18.04 shipped with the Plasma 5.12 LTS and 
Frameworks 5.44. That Plasma version has continued to receive bugfix 
point releases since then. But the Frameworks product has not, and so 
users have now missed out on two years worth of bugfixes. I don't know 
about openSUSE, but I know that Kubuntu does not have the resources to 
backport individual KF bugfixes--I repeatedly requested this as I 
identified them and none ever got backported. But they do ship point 
releases for Plasma, so they could ship point releases for an LTS 
Frameworks version.




IMHO distributions using Plasma LTS, Plasma team & other stakeholders should
team up here and maintain a matching LTS branch of Frameworks together at the
central KDE repos together. Well, and a version also satisfying other clients
of KF, like non-workspace applications from KDE.

It's not a reason to change normal KF release cycle.


I like that idea. So perhaps we could say that the KF version which 
happens to be the dependency for a Plasma LTS release could have bugfix 
releases? Would that be reasonable?



Nate

Re: How to close PMC?

[Nate Graham]

What is PMC? Plasma Media Center? Something else?

Nate


On 2/15/20 4:47 AM, Peter Lorenz wrote:

Hi all,


I' wondering: How do I close the PMC? I can't find anything to do this.

I am running PMC on a laptop with arch linux.
And have always the newest builds installed.

It seems, all I can do is restart the computer.

Can somebody help with this?

Thanks in advance!

Peter Lorenz

Re: 2 kirigami fixes for a point release

[Nate Graham]

On 2/16/20 2:55 PM, Friedrich W. H. Kossebau wrote:

Yes, this has been questioned a few times. Also seeing Plasma LTS used
together with a non-LTS Qt is a bit strange.
But somehow it seems there has not been enough pain for those using the Plasma
LTS to change something. Possibly because distributions simply backport
important bug fixes for KF themselves, kind of maintaining their own KF LTS
version of the KF version they pinpointed to when they froze the ingredients
to their OS. Because they are used to do this for other projects as well, and
so miss this could be done in cooperation with upstream.


There has been pain. This thread mentions a number of examples, and 
There were quite a few for the last 5.12 LTS too. But more generally, 
the pain is baked into Frameworks due to the lack of any bugfix 
releases. For example Kubuntu 18.04 shipped with the Plasma 5.12 LTS and 
Frameworks 5.44. That Plasma version has continued to receive bugfix 
point releases since then. But the Frameworks product has not, and so 
users have now missed out on two years worth of bugfixes. I don't know 
about openSUSE, but I know that Kubuntu does not have the resources to 
backport individual KF bugfixes--I repeatedly requested this as I 
identified them and none ever got backported. But they do ship point 
releases for Plasma, so they could ship point releases for an LTS 
Frameworks version.




IMHO distributions using Plasma LTS, Plasma team & other stakeholders should
team up here and maintain a matching LTS branch of Frameworks together at the
central KDE repos together. Well, and a version also satisfying other clients
of KF, like non-workspace applications from KDE.

It's not a reason to change normal KF release cycle.


I like that idea. So perhaps we could say that the KF version which 
happens to be the dependency for a Plasma LTS release could have bugfix 
releases? Would that be reasonable?



Nate

Re: 2 kirigami fixes for a point release

[Nate Graham]

On 2/17/20 11:08 PM, Luca Beltrame wrote:

In data martedì 18 febbraio 2020 04:03:05 CET, Nate Graham ha scritto:


think KDE software should be presented to users. Basically, we
acknowledge that Neon is already an actual OS--the "KDE OS"--and we


Please don't suggest such downstream-hostile solutions, in particular because
this failing is entirely upstream. We have already plenty in FOSS, I don't
want KDE to be yet another community that "adopts" them.

"We messed up so let's make things our way" is not an acceptable approach.


Neon is already an OS, whether or not you want to admit it. It's 
installed from an ISO. A hardware vendor (Slimbook) is shipping it on 
laptops that people can and do buy. My wife has it installed on her 
computer. It's an OS as much as any other Ubuntu-derived distro can be 
considered an OS.


I actually happen to use openSUSE Tumbleweed myself instead of Neon for 
a variety of reasons, and I'm happy with it. I'm not saying "Neon is an 
OS!" because I think everyone should immediately switch to it and stop 
using other distros. There's room in the universe of KDE distros for one 
more that happens to be a first-party product--as evidenced by the fact 
that Neon has existed for four years and the whole world hasn't come 
tumbling down. I mean, Microsoft got into the PC hardware business in 
competition with Dell, HP, Toshiba, et al, and it didn't destroy their 
business. Far from it: the new entries from Microsoft spurred everyone 
else to improve their own offerings, broadly lifting the quality of PC 
hardware for everyone.


Nate

D27422: [KCM]Update Device item layout based on applet

[George Vogiatzis]

gvgeo updated this revision to Diff 75935.
gvgeo added a comment.


  More style fixes. (indentation restore)

REPOSITORY
  R115 Plasma Audio Volume Applet

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27422?vs=75820=75935

BRANCH
  kcmports2

REVISION DETAIL
  https://phabricator.kde.org/D27422

AFFECTED FILES
  src/kcm/package/contents/ui/DeviceListItem.qml
  src/kcm/package/contents/ui/Devices.qml
  src/kcm/package/contents/ui/MuteButton.qml
  src/kcm/package/contents/ui/StreamListItem.qml

To: gvgeo, #plasma, drosca, ngraham, #vdg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[George Vogiatzis]

gvgeo added inline comments.

INLINE COMMENTS

> FolderItemDelegate.qml:435
> +radius: Math.round(5 * units.devicePixelRatio)
> +samples: radius * 2 + 1
>  spread: 0.05

And I ate the "one"
`Ideally, this value should be twice as large as the highest required radius 
value plus one`

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg
Cc: apol, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[George Vogiatzis]

gvgeo added inline comments.

INLINE COMMENTS

> FolderItemDelegate.qml:435
> +radius: Math.round(5 * units.devicePixelRatio)
> +samples: radius * 2 + 1
>  spread: 0.05

https://doc.qt.io/qt-5/qml-qtgraphicaleffects-dropshadow.html#samples-prop
`Ideally, this value should be twice as large as the highest required radius 
value plus`

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg
Cc: apol, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[Aleix Pol Gonzalez]

apol added a comment.


  +1 patch makes sense to me

INLINE COMMENTS

> FolderItemDelegate.qml:435
> +radius: Math.round(5 * units.devicePixelRatio)
> +samples: radius * 2 + 1
>  spread: 0.05

Why's the + 1?

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg
Cc: apol, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D27482: Update kdeglobals config file for Breeze widgetStyle

[Nathaniel Graham]

ngraham added a comment.


  In D27482#613714 , @crossi wrote:
  
  > In D27482#613699 , @ngraham 
wrote:
  >
  > > How do I trigger this to get run? Restarting plasmashell or opening the 
Style KCM didn't seem to do it.
  >
  >
  > To run the update script, just apply the patch and rebuild plasma-desktop.
  
  
  Did that.
  
  > To confirm it has run in .config/kconf_updaterc at the end you will see
  > 
  >   [style_widgetstyle_default_breeze.upd]
  >   done=StyleWidgetStyleDefaultBreeze,StyleRemoveGeneralWidgetStyle
  
  I see the following:
  
[style_widgetstyle_default_breeze.upd]
ctime=1582046255
done=StyleWidgetStyleDefaultBreeze
mtime=1582046231
  
  
  
  > And in .config/kdeglobals should have updated `widgetStyle` entry in group 
`KDE`
  
  I still see `widgetStyle=breeze` (not capitalized) in the file. :(

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27482

To: crossi, ngraham, ervin, bport, meven, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27482: Update kdeglobals config file for Breeze widgetStyle

[Cyril Rossi]

crossi added a comment.


  This fix the issue for Breeze theme, but according to bug description 
(https://bugs.kde.org/show_bug.cgi?id=417515) some configuration file may also 
have `oxygen` instead of `Oxygen`

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27482

To: crossi, ngraham, ervin, bport, meven, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27482: Update kdeglobals config file for Breeze widgetStyle

[Cyril Rossi]

crossi added a comment.


  In D27482#613699 , @ngraham wrote:
  
  > How do I trigger this to get run? Restarting plasmashell or opening the 
Style KCM didn't seem to do it.
  
  
  To run the update script, just apply the patch and rebuild plasma-desktop.
  
  To confirm it has run in .config/kconf_updaterc at the end you will see
  
[style_widgetstyle_default_breeze.upd]
done=StyleWidgetStyleDefaultBreeze,StyleRemoveGeneralWidgetStyle
  
  And in .config/kdeglobals should have updated `widgetStyle` entry in group 
`KDE`

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27482

To: crossi, ngraham, ervin, bport, meven, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27098: Bugfix: Konsole does not launch, optimize and simplify runner

[Nathaniel Graham]

ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  Looks like a sane clean-up to me. #konsole 
 and.or #plasma 
 folks?

REPOSITORY
  R114 Plasma Addons

BRANCH
  konsole_bugfix (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27098

To: alex, davidedmundson, ngraham, #plasma, #konsole, tcanabrava, hindenburg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27088: [applets/SystemTray] Implement sorting in the model

[David Edmundson]

davidedmundson requested changes to this revision.
davidedmundson added a comment.
This revision now requires changes to proceed.


  The role names part is nice.
  
  I have one major-ish comment, and 2 pendantic comments that I don't really 
care about.

INLINE COMMENTS

> sortedsystemtraymodel.cpp:47
> +if (categoriesComparison == 0) {
> +return compareDisplayAlphabetically(left, right);
> +} else {

I think calling QSortFilterProxyModel::lessThan(left, right); would do the same

then you don't need compareDisplayAlphabetically

your implementation looks fine though, so do whichever

> sortedsystemtraymodel.h:35
> +protected:
> +virtual bool lessThan(const QModelIndex _left, const QModelIndex 
> _right) const override;
> +

We tend not to write virtual at the start now we have the clearer override 
keyword

> systemtraymodel.cpp:115
> +dataItem->setData(applet->title(), Qt::DisplayRole);
> +connect(applet, ::Applet::titleChanged, this, [dataItem] (const 
> QString ) {
> +dataItem->setData(title, static_cast(Qt::DisplayRole));

The applet is still alive after removeApplet is called. "this" is still alive

dataItem is not.

If an applet is added, removed   and then (potentially during applet teardown) 
it changes its title, this would crash.

I don't know if that's a realistic scenario or not,  but I would still maybe 
disconnect all signals from applet -> this on removeApplet?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D27088

To: kmaterka, #plasma_workspaces, #plasma, davidedmundson, ngraham, broulik
Cc: mart, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra

D27483: KCM look and feel : set proper default values for kwin kdecoration when applying global style

[Nathaniel Graham]

ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  Stable branch please. :)
  
  Oh and the syntax is `BUG: 397595` (note the colon)

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27483

To: crossi, ervin, bport, meven, davidedmundson, #plasma, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27482: Update kdeglobals config file for Breeze widgetStyle

[Nathaniel Graham]

ngraham added a comment.


  How do I trigger this to get run? Restarting plasmashell or opening the Style 
KCM didn't seem to do it.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27482

To: crossi, ngraham, ervin, bport, meven, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27483: KCM look and feel : set proper default values for kwin kdecoration when applying global style

[Cyril Rossi]

crossi updated this revision to Diff 75933.
crossi added a comment.


  restore blank line

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27483?vs=75932=75933

REVISION DETAIL
  https://phabricator.kde.org/D27483

AFFECTED FILES
  kcms/lookandfeel/kcm.cpp

To: crossi, ervin, bport, meven, davidedmundson, #plasma, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27481: [WIP]Panel spacer that can center things

[Nathaniel Graham]

ngraham added a comment.


  Super cool feature! This was one of the things I was missing for D24223 
.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D27481

To: mart, #plasma, #vdg
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27408: Fix build with Qt 5.15.0

[Nathaniel Graham]

ngraham added a comment.


  In D27408#613637 , @mlaurent wrote:
  
  > no each reviews but when you create a patch for qt5.15 as I was the only 
dev which worked on qt5.15 it was logical to add me.
  >  I will not subscribe for each commit in plasma/framework, for sure after 
20-30 commits I will not read them. I have ~ 200 mails by days, so for sure I 
will not be able to read them.
  
  
  We all get that many emails per day, or more.
  
  Making sure that commits get reviewed is an important part of keeping up the 
quality. Even obvious-seeming bugfixes can regress things. If your workflow 
makes it impossible for you to seek review for your commits, or to review other 
people's commits, then maybe there's something wrong with your workflow.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

REVISION DETAIL
  https://phabricator.kde.org/D27408

To: apol, #plasma
Cc: ngraham, mlaurent, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27483: KCM look and feel : set proper default values for kwin kdecoration when applying global style

[Cyril Rossi]

crossi created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
crossi requested review of this revision.

REVISION SUMMARY
  BUG 397595

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27483

AFFECTED FILES
  kcms/lookandfeel/kcm.cpp

To: crossi
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27482: Update kdeglobals config file for Breeze widgetStyle

[Cyril Rossi]

crossi created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
crossi requested review of this revision.

REVISION SUMMARY
  Following D27407  provide kconf update 
script to fix user configuration for widgetStyle when Breeze global theme is 
applied

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27482

AFFECTED FILES
  kcms/style/CMakeLists.txt
  kcms/style/style_widgetstyle_default_breeze.pl
  kcms/style/style_widgetstyle_default_breeze.upd

To: crossi
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27422: [KCM]Update Device item layout based on applet

[Nathaniel Graham]

ngraham added a comment.


  You're welcome!

REPOSITORY
  R115 Plasma Audio Volume Applet

BRANCH
  kcmports2

REVISION DETAIL
  https://phabricator.kde.org/D27422

To: gvgeo, #plasma, drosca, ngraham, #vdg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D26806: [Applets/Power Manager] Update layout based on T10470

[Nathaniel Graham]

ngraham added a comment.


  We're waiting for @davidedmundson to re-review it.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D26806

To: gvgeo, #plasma, #vdg, davidedmundson, manueljlin
Cc: broulik, ngraham, manueljlin, davidedmundson, plasma-devel, Orage, 
LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, 
ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[Nathaniel Graham]

ngraham added a comment.


  I don't know enough to say whether it makes sense to fix this here, or in Qt. 
I'd like a #plasma  or maybe #kwin 
 person to weigh in.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27316: [Folder view]Make shadow scale aware

[George Vogiatzis]

gvgeo added a comment.


  Can I close this patch?

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27316

To: gvgeo, #plasma, #vdg
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D26806: [Applets/Power Manager] Update layout based on T10470

[George Vogiatzis]

gvgeo added a comment.


  Is there any problem with this patch?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D26806

To: gvgeo, #plasma, #vdg, davidedmundson, manueljlin
Cc: broulik, ngraham, manueljlin, davidedmundson, plasma-devel, Orage, 
LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, 
ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, ahiemstra, mart

D27470: Set good default value for cursor theme and size in KRDB

[Nathaniel Graham]

ngraham added a comment.


  Can you run `arc amend` on your local branch to pick up the changes I just 
made to the description, and then land this on the stable branch?

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27470

To: bport, #plasma, broulik, ervin, meven, crossi, davidedmundson
Cc: ngraham, davidre, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27422: [KCM]Update Device item layout based on applet

[George Vogiatzis]

gvgeo added a comment.


  Thanks Nate, both for explaining and taking care of the patches yesterday.

REPOSITORY
  R115 Plasma Audio Volume Applet

BRANCH
  kcmports2

REVISION DETAIL
  https://phabricator.kde.org/D27422

To: gvgeo, #plasma, drosca, ngraham, #vdg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27098: Bugfix: Konsole does not launch, optimize and simplify runner

[Alexander Lohnau]

alex updated this revision to Diff 75930.
alex added a comment.


  Rebase patch as requested

REPOSITORY
  R114 Plasma Addons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27098?vs=75724=75930

BRANCH
  konsole_bugfix (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27098

AFFECTED FILES
  runners/konsoleprofiles/CMakeLists.txt
  runners/konsoleprofiles/konsoleprofiles.cpp
  runners/konsoleprofiles/konsoleprofiles.h

To: alex, davidedmundson, ngraham, #plasma, #konsole, tcanabrava, hindenburg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27408: Fix build with Qt 5.15.0

[Laurent Montel]

mlaurent added a comment.


  no each reviews but when you create a patch for qt5.15 as I was the only dev 
which worked on qt5.15 it was logical to add me.
  I will not subscribe for each commit in plasma/framework, for sure after 
20-30 commits I will not read them. I have ~ 200 mails by days, so for sure I 
will not be able to read them.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

REVISION DETAIL
  https://phabricator.kde.org/D27408

To: apol, #plasma
Cc: ngraham, mlaurent, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27098: Bugfix: Konsole does not launch, optimize and simplify runner

[Nathaniel Graham]

ngraham added a comment.


  Can you rebase this patch now?

REPOSITORY
  R114 Plasma Addons

REVISION DETAIL
  https://phabricator.kde.org/D27098

To: alex, davidedmundson, ngraham, #plasma, #konsole, tcanabrava, hindenburg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27365: fix condition in KonsoleProfiles::run

[Nathaniel Graham]

This revision was automatically updated to reflect the committed changes.
Closed by commit R114:f0431c95a895: fix condition in KonsoleProfiles::run 
(authored by McPain, committed by ngraham).

REPOSITORY
  R114 Plasma Addons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27365?vs=75596=75929

REVISION DETAIL
  https://phabricator.kde.org/D27365

AFFECTED FILES
  runners/konsoleprofiles/konsoleprofiles.cpp

To: McPain, heikobecker, ngraham
Cc: ngraham, alex, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27365: fix condition in KonsoleProfiles::run

[Nathaniel Graham]

ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  Thanks for your patch @alex. However since this is a targeted bugfix, I would 
like to land it now on the stable branch. Since your patch includes 
refactoring, it needs to go into the next release. I'll land this now, and then 
you can remove this bugfix from your patch. Thanks!

REPOSITORY
  R114 Plasma Addons

REVISION DETAIL
  https://phabricator.kde.org/D27365

To: McPain, heikobecker, ngraham
Cc: ngraham, alex, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27367: fix activity runner

[Nathaniel Graham]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:2a979a4bbebe: fix activity runner (authored by McPain, 
committed by ngraham).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27367?vs=75900=75928

REVISION DETAIL
  https://phabricator.kde.org/D27367

AFFECTED FILES
  runners/activities/activityrunner.cpp

To: McPain, broulik, ngraham, davidedmundson
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27408: Fix build with Qt 5.15.0

[Aleix Pol Gonzalez]

apol added a comment.


  In D27408#613598 , @mlaurent wrote:
  
  > Just a question: How I can see it if nobody adds me as reviewer (if I 
remember I was the only one guys who works against qt5.15 from several months 
so why don't add me as reviewer ? )? (For sure I have some scripts for automate 
theses changes so I commit quickly it)
  
  
  Do you want us to add you on every review? I'd say you can just subscribe to 
Plasma and Frameworks if you are planning to work on Plasma and Frameworks.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

REVISION DETAIL
  https://phabricator.kde.org/D27408

To: apol, #plasma
Cc: ngraham, mlaurent, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27395: KCM/ComponentChooser Treat cases when there is no app for a usage

[Méven Car]

meven updated this revision to Diff 75926.
meven added a comment.


  Make validLastCurrentIndex const, newline before {

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27395?vs=75920=75926

BRANCH
  arcpatch-D27395

REVISION DETAIL
  https://phabricator.kde.org/D27395

AFFECTED FILES
  kcms/componentchooser/componentchooser.h
  kcms/componentchooser/componentchooserbrowser.cpp
  kcms/componentchooser/componentchooseremail.cpp
  kcms/componentchooser/componentchooserfilemanager.cpp
  kcms/componentchooser/componentchooserterminal.cpp

To: meven, bport, ervin, crossi, davidedmundson, #plasma
Cc: anthonyfieroni, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27481: [WIP]Panel spacer that can center things

[Marco Martin]

mart added a comment.


  Implementation-wise could make sense for the implementation be in the panel 
itself, tough it may be difficult to control from the external panel 
configuration tool
  
  Usability-wise i think would be fine if the first spacer is added at the end 
of the panel, and the second at the beginning, trigering the centering for the 
whole panel contents, tough again is a bit difficult to trigger from the 
external panel configuration toolbar

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D27481

To: mart, #plasma, #vdg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27481: [WIP]Panel spacer that can center things

[Marco Martin]

mart created this revision.
mart added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
mart requested review of this revision.

REVISION SUMMARY
  - Make the panel spacer take a lot more space when is in expanding mode, so 
much
  
  to "win" against the taskbar and collapse it to its minimum size (seems the
  most expected behavior after some discussions in vdg channel)
  
  - make it paint a background when in edit mode
  - when two spacers are present in the panel, they try to center all the 
content
  
  that is in between them (if possible), so that is always at the center of the 
panel,
  even if there is more content on one side rather than the other

REPOSITORY
  R120 Plasma Workspace

BRANCH
  mart/panelSpacer

REVISION DETAIL
  https://phabricator.kde.org/D27481

AFFECTED FILES
  applets/CMakeLists.txt
  applets/panelspacer/CMakeLists.txt
  applets/panelspacer/Messages.sh
  applets/panelspacer/contents/config/main.xml
  applets/panelspacer/contents/ui/main.qml
  applets/panelspacer/metadata.desktop
  applets/panelspacer/package/contents/config/main.xml
  applets/panelspacer/package/contents/ui/main.qml
  applets/panelspacer/package/metadata.desktop
  applets/panelspacer/plugin/CMakeLists.txt
  applets/panelspacer/plugin/panelspacer.cpp
  applets/panelspacer/plugin/panelspacer.h

To: mart, #plasma, #vdg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27395: KCM/ComponentChooser Treat cases when there is no app for a usage

[Anthony Fieroni]

anthonyfieroni added inline comments.

INLINE COMMENTS

> componentchooser.h:51
>  
> +int validLastCurrentIndex() {
> +// m_currentIndex == -1 means there are no previously saved value

{ in new line, add const

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27395

To: meven, bport, ervin, crossi, davidedmundson, #plasma
Cc: anthonyfieroni, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27408: Fix build with Qt 5.15.0

[Laurent Montel]

mlaurent added a comment.


  In D27408#613565 , @ngraham wrote:
  
  > @mlaurent ^^
  
  
  Just a question: How I can see it if nobody adds me as reviewer (if I 
remember I was the only one guys who works against qt5.15 from several months 
so why don't add me as reviewer ? )? (For sure I have some scripts for automate 
theses changes so I commit quickly it)
  
  but this patch was wrong as QOverload::of(::currentIndexChanged), was created in qt5.15 not in 
"<= qt5.15" so in qt5.15.1 it will failed when  we compile without deprecated 
methods.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

REVISION DETAIL
  https://phabricator.kde.org/D27408

To: apol, #plasma
Cc: ngraham, mlaurent, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27127: Solid-device-automounter/kcm: Use KConfigXT in ui

[Méven Car]

meven marked 2 inline comments as done.
meven added inline comments.

INLINE COMMENTS

> meven wrote in DeviceAutomounterKCM.cpp:57
> Just waiting for this review then, I am preparing the next patch

D27480 

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27127

To: meven, ervin, ngraham, #plasma, bport, crossi
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27480: Solid-device-automounter/kcm: Get rid of singleton for AutomounterSettings

[Méven Car]

meven created this revision.
meven added reviewers: ervin, crossi, bport, Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
meven requested review of this revision.

REVISION SUMMARY
  Depends on D27127 

REPOSITORY
  R119 Plasma Desktop

BRANCH
  arcpatch-D27127

REVISION DETAIL
  https://phabricator.kde.org/D27480

AFFECTED FILES
  solid-device-automounter/kcm/DeviceAutomounterKCM.cpp
  solid-device-automounter/kcm/DeviceAutomounterKCM.h
  solid-device-automounter/kcm/DeviceModel.cpp
  solid-device-automounter/kcm/DeviceModel.h
  solid-device-automounter/kded/DeviceAutomounter.cpp
  solid-device-automounter/kded/DeviceAutomounter.h
  solid-device-automounter/lib/AutomounterSettings.cpp
  solid-device-automounter/lib/AutomounterSettings.h
  solid-device-automounter/lib/AutomounterSettingsBase.kcfgc

To: meven, ervin, crossi, bport, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27470: Set good default value for cursor theme and size in KRDB

[Nathaniel Graham]

ngraham added a comment.


  This probably fixes at least one of these:
  
  - https://bugs.kde.org/show_bug.cgi?id=385920
  - https://bugs.kde.org/show_bug.cgi?id=413783
  - https://bugs.kde.org/show_bug.cgi?id=382604
  - https://bugs.kde.org/show_bug.cgi?id=414109

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27470

To: bport, #plasma, broulik, ervin, meven, crossi, davidedmundson
Cc: ngraham, davidre, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27408: Fix build with Qt 5.15.0

[Nathaniel Graham]

ngraham added subscribers: mlaurent, ngraham.
ngraham added a comment.


  @mlaurent ^^

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

REVISION DETAIL
  https://phabricator.kde.org/D27408

To: apol, #plasma
Cc: ngraham, mlaurent, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27457: Move kcminit_startup and kded to plasma-session

[Aleix Pol Gonzalez]

apol added a comment.


  Both the isServiceRegistered and the env changes make sense to me.

INLINE COMMENTS

> startup.cpp:232
>  connect(loginSound, ::finished, loginSound, 
> ::deleteLater);
>  loginSound->start();});
>  

Unrelated change.

> startup.cpp:421
> +m_process->setArguments(args);
> +m_process->setProcessEnvironment(env);
> +

Maybe it would make sense to do the merging of envs here? This way we only set 
when it's necessary.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D27457

To: apol, #plasma, davidedmundson, fvogt
Cc: fvogt, lbeltrame, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27127: Solid-device-automounter/kcm: Use KConfigXT in ui

[Méven Car]

meven marked 4 inline comments as done.
meven added inline comments.

INLINE COMMENTS

> ervin wrote in DeviceAutomounterKCM.cpp:57
> Sure, moving away from a singleton is always intrusive (just like moving away 
> from a global variable which it is really). Let's aim for it in a different 
> specific patch.

Just waiting for this review then, I am preparing the next patch

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27127

To: meven, ervin, ngraham, #plasma, bport, crossi
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27411: Port away from deprecated API

[Aleix Pol Gonzalez]

apol abandoned this revision.
apol added a subscriber: mlaurent.
apol added a comment.


  @mlaurent did the same but he doesn't put his patches through review.

REPOSITORY
  R838 Flatpak Support: KDE Portal for XDG Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27411

To: apol, jgrulich
Cc: mlaurent, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27465: Make the icon hitboxes for the System Tray Plasmoid larger when Kirigami Tablet Mode is enabled

[Nathaniel Graham]

ngraham accepted this revision.
ngraham added a reviewer: Plasma.
ngraham added a comment.
This revision is now accepted and ready to land.


  LGTM. #plasma  folks, is this all 
technically correct?

REPOSITORY
  R120 Plasma Workspace

BRANCH
  D27465 (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27465

To: The-Feren-OS-Dev, ngraham, #vdg, #plasma
Cc: davidre, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27358: Sync ("Wayland") appending to match SDDM

[Fabian Vogt]

fvogt added a comment.


  Note that this behaviour of sddm is not in any release yet - only in git.

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

REVISION DETAIL
  https://phabricator.kde.org/D27358

To: davidedmundson, #plasma, ngraham
Cc: fvogt, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27395: KCM/ComponentChooser Treat cases when there is no app for a usage

[Méven Car]

meven updated this revision to Diff 75920.
meven marked 2 inline comments as done.
meven added a comment.


  Add a validLastCurrentIndex function, check emailClientService is not null 
before using it

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27395?vs=75695=75920

BRANCH
  arcpatch-D27395

REVISION DETAIL
  https://phabricator.kde.org/D27395

AFFECTED FILES
  kcms/componentchooser/componentchooser.h
  kcms/componentchooser/componentchooserbrowser.cpp
  kcms/componentchooser/componentchooseremail.cpp
  kcms/componentchooser/componentchooserfilemanager.cpp
  kcms/componentchooser/componentchooserterminal.cpp

To: meven, bport, ervin, crossi, davidedmundson, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27424: Add missing cursors for Chromium-based apps

[Guilherme Silva]

guilhermesilva added a comment.


  Absolutely! Thank you very much.

REPOSITORY
  R31 Breeze

REVISION DETAIL
  https://phabricator.kde.org/D27424

To: guilhermesilva, #breeze, broulik, ngraham
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27408: Fix build with Qt 5.15.0

[Aleix Pol Gonzalez]

apol abandoned this revision.
apol added a comment.


  Abandoned because @lmontel just commits changes instead of reviewing other 
people's patches. :(

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

REVISION DETAIL
  https://phabricator.kde.org/D27408

To: apol, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27466: Increase the size of system tray icon hitboxes on the System Tray Plasmoid

[Dominic Hayes]

The-Feren-OS-Dev updated this revision to Diff 75913.
The-Feren-OS-Dev added a comment.


  Updated diff

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27466?vs=75912=75913

BRANCH
  D27466 (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27466

AFFECTED FILES
  applets/systemtray/package/contents/ui/main.qml

To: The-Feren-OS-Dev, #vdg
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27466: Increase the size of system tray icon hitboxes on the System Tray Plasmoid

[Dominic Hayes]

The-Feren-OS-Dev updated this revision to Diff 75912.
The-Feren-OS-Dev added a comment.


  Updated diff

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27466?vs=75873=75912

BRANCH
  D27466 (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27466

AFFECTED FILES
  applets/systemtray/package/contents/ui/main.qml

To: The-Feren-OS-Dev, #vdg
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27465: Make the icon hitboxes for the System Tray Plasmoid larger when Kirigami Tablet Mode is enabled

[Dominic Hayes]

The-Feren-OS-Dev updated this revision to Diff 75911.
The-Feren-OS-Dev marked an inline comment as done.
The-Feren-OS-Dev added a comment.


  Moved baseSize declaration and changed it to a standard variable

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27465?vs=75869=75911

BRANCH
  D27465 (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27465

AFFECTED FILES
  applets/systemtray/package/contents/ui/main.qml

To: The-Feren-OS-Dev, ngraham, #vdg
Cc: davidre, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27407: Breeze widgetStyle value is Breeze

[Cyril Rossi]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:c7c190488e46: Breeze widgetStyle value is Breeze 
(authored by crossi).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27407?vs=75699=75909

REVISION DETAIL
  https://phabricator.kde.org/D27407

AFFECTED FILES
  lookandfeel/contents/defaults

To: crossi, ervin, bport, meven, mart, davidedmundson, ngraham
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

KDE CI: Plasma » plasma-desktop » stable-kf5-qt5 SUSEQt5.12 - Build # 265 - Still Failing!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/plasma-desktop/job/stable-kf5-qt5%20SUSEQt5.12/265/
 Project:
stable-kf5-qt5 SUSEQt5.12
 Date of build:
Tue, 18 Feb 2020 12:50:28 +
 Build duration:
11 min and counting
   CONSOLE OUTPUT
  [...truncated 276 lines...][2020-02-18T13:01:35.455Z] WINDOWMANAGER = 'xterm'[2020-02-18T13:01:35.455Z] PAGER = 'less'[2020-02-18T13:01:35.455Z] EXECUTOR_NUMBER   = '0'[2020-02-18T13:01:35.455Z] CVS_RSH   = 'ssh'[2020-02-18T13:01:35.455Z] LIBGL_DEBUG   = 'quiet'[2020-02-18T13:01:35.455Z] STAGE_NAME= 'Configuring Build'[2020-02-18T13:01:35.455Z] NODE_LABELS   = 'Docker Swarm-1c2f368d47fe SUSEQt5.12'[2020-02-18T13:01:35.455Z] XDG_DATA_DIRS = '/home/jenkins/install-prefix/share:/usr/share:/usr/local/share:/usr/share'[2020-02-18T13:01:35.455Z] CONFIG_SITE   = '/usr/share/site/x86_64-unknown-linux-gnu'[2020-02-18T13:01:35.455Z] PATH  = '/home/jenkins/install-prefix/bin:/home/jenkins//bin:/usr/local/bin:/usr/bin:/bin:/usr/lib/mit/bin:/usr/lib/mit/sbin'[2020-02-18T13:01:35.455Z] PROFILEREAD   = 'true'[2020-02-18T13:01:35.455Z] MAIL  = '/var/spool/mail/jenkins'[2020-02-18T13:01:35.455Z] HOSTTYPE  = 'x86_64'[2020-02-18T13:01:35.455Z] BUILD_ID  = '265'[2020-02-18T13:01:35.455Z] LESSKEY   = '/usr/etc/lesskey.bin'[2020-02-18T13:01:35.455Z] OLDPWD= '/home/jenkins'[2020-02-18T13:01:35.455Z] LC_CTYPE  = 'C.UTF-8'[2020-02-18T13:01:35.455Z] CMAKE_PREFIX_PATH = '/home/jenkins/install-prefix:/usr/:/usr/local/'[2020-02-18T13:01:35.455Z] LD_LIBRARY_PATH   = '/home/jenkins/install-prefix/lib64'[2020-02-18T13:01:35.455Z] PKG_CONFIG_PATH   = '/home/jenkins/install-prefix/lib64/pkgconfig:/usr/lib/pkgconfig:/usr/lib64/pkgconfig'[2020-02-18T13:01:35.455Z] QT_PLUGIN_PATH= '/home/jenkins/install-prefix/lib64/plugins'[2020-02-18T13:01:35.455Z] QML2_IMPORT_PATH  = '/home/jenkins/install-prefix/lib64/qml'[2020-02-18T13:01:35.455Z] QT_SELECT = 'qt5'[2020-02-18T13:01:35.455Z] LANG  = 'en_US.UTF-8'[2020-02-18T13:01:35.455Z] XDG_CURRENT_DESKTOP   = 'KDE'[2020-02-18T13:01:35.455Z] ASAN_OPTIONS  = 'detect_leaks=0:new_delete_type_mismatch=0:detect_odr_violation=0:stack-use-after-scope=0:alloc_dealloc_mismatch=0'[2020-02-18T13:01:35.455Z] [2020-02-18T13:01:35.455Z] [2020-02-18T13:01:35.455Z] == Commencing Configuration:[2020-02-18T13:01:35.455Z] cmake -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=ON -DCMAKE_INSTALL_PREFIX="/home/jenkins//install-prefix/" -DBUILD_COVERAGE=ON -DECM_ENABLE_SANITIZERS='address'  "/home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 SUSEQt5.12"[2020-02-18T13:01:35.727Z] -- The C compiler identification is GNU 9.2.1[2020-02-18T13:01:36.130Z] -- The CXX compiler identification is GNU 9.2.1[2020-02-18T13:01:36.130Z] -- Check for working C compiler: /usr/bin/cc[2020-02-18T13:01:36.130Z] -- Check for working C compiler: /usr/bin/cc -- works[2020-02-18T13:01:36.130Z] -- Detecting C compiler ABI info[2020-02-18T13:01:36.130Z] -- Detecting C compiler ABI info - done[2020-02-18T13:01:36.130Z] -- Detecting C compile features[2020-02-18T13:01:36.130Z] -- Detecting C compile features - done[2020-02-18T13:01:36.130Z] -- Check for working CXX compiler: /usr/bin/c++[2020-02-18T13:01:36.402Z] -- Check for working CXX compiler: /usr/bin/c++ -- works[2020-02-18T13:01:36.402Z] -- Detecting CXX compiler ABI info[2020-02-18T13:01:36.667Z] -- Detecting CXX compiler ABI info - done[2020-02-18T13:01:36.667Z] -- Detecting CXX compile features[2020-02-18T13:01:36.667Z] -- Detecting CXX compile features - done[2020-02-18T13:01:36.667Z] Installing in /home/jenkins/install-prefix. Run /home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 SUSEQt5.12/build/prefix.sh to set the environment for plasma-desktop.[2020-02-18T13:01:36.667Z] -- Looking for __GLIBC__[2020-02-18T13:01:37.077Z] -- Looking for __GLIBC__ - found[2020-02-18T13:01:37.077Z] -- Performing Test _OFFT_IS_64BIT[2020-02-18T13:01:37.362Z] -- Performing Test _OFFT_IS_64BIT - Success[2020-02-18T13:01:37.362Z] -- Performing Test HAVE_DATE_TIME[2020-02-18T13:01:37.623Z] -- Performing Test HAVE_DATE_TIME - Success[2020-02-18T13:01:38.633Z] -- Found KF5Auth: /home/jenkins/install-prefix/lib64/cmake/KF5Auth/KF5AuthConfig.cmake (found version "5.68.0") [2020-02-18T13:01:38.633Z] Installing in /home/jenkins/install-prefix. Run /home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 SUSEQt5.12/build/prefix.sh to set the environment for plasma-desktop.[2020-02-18T13:01:38.633Z] Installing in /home/jenkins/install-prefix. Run /home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 SUSEQt5.12/build/prefix.sh to set the environment for 

KDE CI: Plasma » systemsettings » stable-kf5-qt5 SUSEQt5.12 - Build # 99 - Failure!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/systemsettings/job/stable-kf5-qt5%20SUSEQt5.12/99/
 Project:
stable-kf5-qt5 SUSEQt5.12
 Date of build:
Tue, 18 Feb 2020 12:43:47 +
 Build duration:
15 min and counting
   CONSOLE OUTPUT
  [...truncated 267 lines...][2020-02-18T12:58:45.426Z] LESSOPEN  = 'lessopen.sh %s'[2020-02-18T12:58:45.426Z] USER  = 'jenkins'[2020-02-18T12:58:45.426Z] MORE  = '-sl'[2020-02-18T12:58:45.426Z] CSHEDIT   = 'emacs'[2020-02-18T12:58:45.426Z] SHLVL = '2'[2020-02-18T12:58:45.426Z] BUILD_TAG = 'jenkins-Plasma-systemsettings-stable-kf5-qt5 SUSEQt5.12-99'[2020-02-18T12:58:45.426Z] WINDOWMANAGER = 'xterm'[2020-02-18T12:58:45.426Z] PAGER = 'less'[2020-02-18T12:58:45.426Z] EXECUTOR_NUMBER   = '0'[2020-02-18T12:58:45.426Z] CVS_RSH   = 'ssh'[2020-02-18T12:58:45.426Z] LIBGL_DEBUG   = 'quiet'[2020-02-18T12:58:45.426Z] STAGE_NAME= 'Configuring Build'[2020-02-18T12:58:45.426Z] NODE_LABELS   = 'Docker Swarm-b03e5c8e6b9f SUSEQt5.12'[2020-02-18T12:58:45.426Z] XDG_DATA_DIRS = '/home/jenkins/install-prefix/share:/usr/share:/usr/local/share:/usr/share'[2020-02-18T12:58:45.426Z] CONFIG_SITE   = '/usr/share/site/x86_64-unknown-linux-gnu'[2020-02-18T12:58:45.426Z] PATH  = '/home/jenkins/install-prefix/bin:/home/jenkins//bin:/usr/local/bin:/usr/bin:/bin:/usr/lib/mit/bin:/usr/lib/mit/sbin'[2020-02-18T12:58:45.426Z] PROFILEREAD   = 'true'[2020-02-18T12:58:45.426Z] MAIL  = '/var/spool/mail/jenkins'[2020-02-18T12:58:45.426Z] HOSTTYPE  = 'x86_64'[2020-02-18T12:58:45.426Z] BUILD_ID  = '99'[2020-02-18T12:58:45.426Z] LESSKEY   = '/usr/etc/lesskey.bin'[2020-02-18T12:58:45.426Z] OLDPWD= '/home/jenkins'[2020-02-18T12:58:45.426Z] LC_CTYPE  = 'C.UTF-8'[2020-02-18T12:58:45.426Z] CMAKE_PREFIX_PATH = '/home/jenkins/install-prefix:/usr/:/usr/local/'[2020-02-18T12:58:45.426Z] LD_LIBRARY_PATH   = '/home/jenkins/install-prefix/lib64'[2020-02-18T12:58:45.426Z] PKG_CONFIG_PATH   = '/home/jenkins/install-prefix/lib64/pkgconfig:/usr/lib/pkgconfig:/usr/lib64/pkgconfig'[2020-02-18T12:58:45.426Z] QT_PLUGIN_PATH= '/home/jenkins/install-prefix/lib64/plugins'[2020-02-18T12:58:45.426Z] QML2_IMPORT_PATH  = '/home/jenkins/install-prefix/lib64/qml'[2020-02-18T12:58:45.426Z] QT_SELECT = 'qt5'[2020-02-18T12:58:45.426Z] LANG  = 'en_US.UTF-8'[2020-02-18T12:58:45.426Z] XDG_CURRENT_DESKTOP   = 'KDE'[2020-02-18T12:58:45.426Z] ASAN_OPTIONS  = 'detect_leaks=0:new_delete_type_mismatch=0:detect_odr_violation=0:stack-use-after-scope=0:alloc_dealloc_mismatch=0'[2020-02-18T12:58:45.426Z] [2020-02-18T12:58:45.426Z] [2020-02-18T12:58:45.426Z] == Commencing Configuration:[2020-02-18T12:58:45.426Z] cmake -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=ON -DCMAKE_INSTALL_PREFIX="/home/jenkins//install-prefix/" -DBUILD_COVERAGE=ON -DECM_ENABLE_SANITIZERS='address'  "/home/jenkins/workspace/Plasma/systemsettings/stable-kf5-qt5 SUSEQt5.12"[2020-02-18T12:58:45.426Z] -- The C compiler identification is GNU 9.2.1[2020-02-18T12:58:45.682Z] -- The CXX compiler identification is GNU 9.2.1[2020-02-18T12:58:45.682Z] -- Check for working C compiler: /usr/bin/cc[2020-02-18T12:58:45.682Z] -- Check for working C compiler: /usr/bin/cc -- works[2020-02-18T12:58:45.682Z] -- Detecting C compiler ABI info[2020-02-18T12:58:45.682Z] -- Detecting C compiler ABI info - done[2020-02-18T12:58:45.682Z] -- Detecting C compile features[2020-02-18T12:58:45.682Z] -- Detecting C compile features - done[2020-02-18T12:58:45.682Z] -- Check for working CXX compiler: /usr/bin/c++[2020-02-18T12:58:45.682Z] -- Check for working CXX compiler: /usr/bin/c++ -- works[2020-02-18T12:58:45.682Z] -- Detecting CXX compiler ABI info[2020-02-18T12:58:45.936Z] -- Detecting CXX compiler ABI info - done[2020-02-18T12:58:45.936Z] -- Detecting CXX compile features[2020-02-18T12:58:45.936Z] -- Detecting CXX compile features - done[2020-02-18T12:58:45.936Z] Installing in /home/jenkins/install-prefix. Run /home/jenkins/workspace/Plasma/systemsettings/stable-kf5-qt5 SUSEQt5.12/build/prefix.sh to set the environment for systemsettings.[2020-02-18T12:58:45.936Z] -- Looking for __GLIBC__[2020-02-18T12:58:45.936Z] -- Looking for __GLIBC__ - found[2020-02-18T12:58:45.936Z] -- Performing Test _OFFT_IS_64BIT[2020-02-18T12:58:46.191Z] -- Performing Test _OFFT_IS_64BIT - Success[2020-02-18T12:58:46.191Z] -- Performing Test HAVE_DATE_TIME[2020-02-18T12:58:46.191Z] -- Performing Test HAVE_DATE_TIME - Success[2020-02-18T12:58:46.191Z] -- Found KF5Crash: 

KDE CI: Plasma » powerdevil » stable-kf5-qt5 SUSEQt5.12 - Build # 52 - Failure!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/powerdevil/job/stable-kf5-qt5%20SUSEQt5.12/52/
 Project:
stable-kf5-qt5 SUSEQt5.12
 Date of build:
Tue, 18 Feb 2020 12:43:33 +
 Build duration:
14 min and counting
   CONSOLE OUTPUT
  [...truncated 259 lines...][2020-02-18T12:57:50.709Z] RUN_DISPLAY_URL   = 'https://build.kde.org/job/Plasma/job/powerdevil/job/stable-kf5-qt5%20SUSEQt5.12/52/display/redirect'[2020-02-18T12:57:50.709Z] JOB_DISPLAY_URL   = 'https://build.kde.org/job/Plasma/job/powerdevil/job/stable-kf5-qt5%20SUSEQt5.12/display/redirect'[2020-02-18T12:57:50.709Z] WORKSPACE = '/home/jenkins/workspace/Plasma/powerdevil/stable-kf5-qt5 SUSEQt5.12'[2020-02-18T12:57:50.709Z] HUDSON_URL= 'https://build.kde.org/'[2020-02-18T12:57:50.709Z] LESSCLOSE = 'lessclose.sh %s %s'[2020-02-18T12:57:50.709Z] NODE_NAME = 'Docker Swarm-eeb95c299469'[2020-02-18T12:57:50.709Z] HOST  = 'eeb95c299469'[2020-02-18T12:57:50.709Z] LESSOPEN  = 'lessopen.sh %s'[2020-02-18T12:57:50.709Z] USER  = 'jenkins'[2020-02-18T12:57:50.709Z] MORE  = '-sl'[2020-02-18T12:57:50.709Z] CSHEDIT   = 'emacs'[2020-02-18T12:57:50.709Z] SHLVL = '2'[2020-02-18T12:57:50.709Z] BUILD_TAG = 'jenkins-Plasma-powerdevil-stable-kf5-qt5 SUSEQt5.12-52'[2020-02-18T12:57:50.709Z] WINDOWMANAGER = 'xterm'[2020-02-18T12:57:50.709Z] PAGER = 'less'[2020-02-18T12:57:50.709Z] EXECUTOR_NUMBER   = '0'[2020-02-18T12:57:50.709Z] CVS_RSH   = 'ssh'[2020-02-18T12:57:50.709Z] LIBGL_DEBUG   = 'quiet'[2020-02-18T12:57:50.709Z] STAGE_NAME= 'Configuring Build'[2020-02-18T12:57:50.709Z] NODE_LABELS   = 'Docker Swarm-eeb95c299469 SUSEQt5.12'[2020-02-18T12:57:50.709Z] XDG_DATA_DIRS = '/home/jenkins/install-prefix/share:/usr/share:/usr/local/share:/usr/share'[2020-02-18T12:57:50.709Z] CONFIG_SITE   = '/usr/share/site/x86_64-unknown-linux-gnu'[2020-02-18T12:57:50.709Z] PATH  = '/home/jenkins/install-prefix/bin:/home/jenkins//bin:/usr/local/bin:/usr/bin:/bin:/usr/lib/mit/bin:/usr/lib/mit/sbin'[2020-02-18T12:57:50.709Z] PROFILEREAD   = 'true'[2020-02-18T12:57:50.709Z] MAIL  = '/var/spool/mail/jenkins'[2020-02-18T12:57:50.709Z] HOSTTYPE  = 'x86_64'[2020-02-18T12:57:50.709Z] BUILD_ID  = '52'[2020-02-18T12:57:50.709Z] LESSKEY   = '/usr/etc/lesskey.bin'[2020-02-18T12:57:50.709Z] OLDPWD= '/home/jenkins'[2020-02-18T12:57:50.709Z] LC_CTYPE  = 'C.UTF-8'[2020-02-18T12:57:50.709Z] CMAKE_PREFIX_PATH = '/home/jenkins/install-prefix:/usr/:/usr/local/'[2020-02-18T12:57:50.709Z] LD_LIBRARY_PATH   = '/home/jenkins/install-prefix/lib64'[2020-02-18T12:57:50.709Z] PKG_CONFIG_PATH   = '/home/jenkins/install-prefix/lib64/pkgconfig:/usr/lib/pkgconfig:/usr/lib64/pkgconfig'[2020-02-18T12:57:50.709Z] QT_PLUGIN_PATH= '/home/jenkins/install-prefix/lib64/plugins'[2020-02-18T12:57:50.709Z] QML2_IMPORT_PATH  = '/home/jenkins/install-prefix/lib64/qml'[2020-02-18T12:57:50.709Z] QT_SELECT = 'qt5'[2020-02-18T12:57:50.709Z] LANG  = 'en_US.UTF-8'[2020-02-18T12:57:50.709Z] XDG_CURRENT_DESKTOP   = 'KDE'[2020-02-18T12:57:50.709Z] ASAN_OPTIONS  = 'detect_leaks=0:new_delete_type_mismatch=0:detect_odr_violation=0:stack-use-after-scope=0:alloc_dealloc_mismatch=0'[2020-02-18T12:57:50.709Z] [2020-02-18T12:57:50.709Z] [2020-02-18T12:57:50.709Z] == Commencing Configuration:[2020-02-18T12:57:50.709Z] cmake -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=ON -DCMAKE_INSTALL_PREFIX="/home/jenkins//install-prefix/" -DBUILD_COVERAGE=ON -DECM_ENABLE_SANITIZERS='address'  "/home/jenkins/workspace/Plasma/powerdevil/stable-kf5-qt5 SUSEQt5.12"[2020-02-18T12:57:51.728Z] -- The C compiler identification is GNU 9.2.1[2020-02-18T12:57:51.728Z] -- The CXX compiler identification is GNU 9.2.1[2020-02-18T12:57:51.728Z] -- Check for working C compiler: /usr/bin/cc[2020-02-18T12:57:51.986Z] -- Check for working C compiler: /usr/bin/cc -- works[2020-02-18T12:57:51.986Z] -- Detecting C compiler ABI info[2020-02-18T12:57:51.986Z] -- Detecting C compiler ABI info - done[2020-02-18T12:57:51.986Z] -- Detecting C compile features[2020-02-18T12:57:51.986Z] -- Detecting C compile features - done[2020-02-18T12:57:51.986Z] -- Check for working CXX compiler: /usr/bin/c++[2020-02-18T12:57:51.986Z] -- Check for working CXX compiler: /usr/bin/c++ -- works[2020-02-18T12:57:51.986Z] -- Detecting CXX compiler ABI info[2020-02-18T12:57:51.986Z] -- Detecting CXX compiler ABI info - done[2020-02-18T12:57:51.986Z] -- Detecting CXX compile features[2020-02-18T12:57:51.986Z] -- 

D27457: Move kcminit_startup and kded to plasma-session

[David Edmundson]

davidedmundson updated this revision to Diff 75908.
davidedmundson added a comment.


  rebase and squash changes

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27457?vs=75901=75908

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D27457

AFFECTED FILES
  startkde/plasma-session/startup.cpp
  startkde/plasma-session/startup.h

To: apol, #plasma, davidedmundson, fvogt
Cc: fvogt, lbeltrame, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27472: [plasma-session] Cleanup StartServiceJob

[David Edmundson]

davidedmundson abandoned this revision.
davidedmundson added a comment.


  Turns out this is fixing up something that changed in the non-merged patch.
  
  I may as well squash it there.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D27472

To: davidedmundson, #plasma, apol
Cc: apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

KDE CI: Plasma » plasma-desktop » stable-kf5-qt5 SUSEQt5.12 - Build # 264 - Failure!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/plasma-desktop/job/stable-kf5-qt5%20SUSEQt5.12/264/
 Project:
stable-kf5-qt5 SUSEQt5.12
 Date of build:
Tue, 18 Feb 2020 12:41:21 +
 Build duration:
8 min 57 sec and counting
   CONSOLE OUTPUT
  [...truncated 276 lines...][2020-02-18T12:50:06.055Z] WINDOWMANAGER = 'xterm'[2020-02-18T12:50:06.055Z] PAGER = 'less'[2020-02-18T12:50:06.055Z] EXECUTOR_NUMBER   = '0'[2020-02-18T12:50:06.055Z] CVS_RSH   = 'ssh'[2020-02-18T12:50:06.055Z] LIBGL_DEBUG   = 'quiet'[2020-02-18T12:50:06.055Z] STAGE_NAME= 'Configuring Build'[2020-02-18T12:50:06.055Z] NODE_LABELS   = 'Docker Swarm-d1e497ceac68 SUSEQt5.12'[2020-02-18T12:50:06.055Z] XDG_DATA_DIRS = '/home/jenkins/install-prefix/share:/usr/share:/usr/local/share:/usr/share'[2020-02-18T12:50:06.055Z] CONFIG_SITE   = '/usr/share/site/x86_64-unknown-linux-gnu'[2020-02-18T12:50:06.055Z] PATH  = '/home/jenkins/install-prefix/bin:/home/jenkins//bin:/usr/local/bin:/usr/bin:/bin:/usr/lib/mit/bin:/usr/lib/mit/sbin'[2020-02-18T12:50:06.055Z] PROFILEREAD   = 'true'[2020-02-18T12:50:06.055Z] MAIL  = '/var/spool/mail/jenkins'[2020-02-18T12:50:06.055Z] HOSTTYPE  = 'x86_64'[2020-02-18T12:50:06.055Z] BUILD_ID  = '264'[2020-02-18T12:50:06.055Z] LESSKEY   = '/usr/etc/lesskey.bin'[2020-02-18T12:50:06.055Z] OLDPWD= '/home/jenkins'[2020-02-18T12:50:06.055Z] LC_CTYPE  = 'C.UTF-8'[2020-02-18T12:50:06.055Z] CMAKE_PREFIX_PATH = '/home/jenkins/install-prefix:/usr/:/usr/local/'[2020-02-18T12:50:06.055Z] LD_LIBRARY_PATH   = '/home/jenkins/install-prefix/lib64'[2020-02-18T12:50:06.055Z] PKG_CONFIG_PATH   = '/home/jenkins/install-prefix/lib64/pkgconfig:/usr/lib/pkgconfig:/usr/lib64/pkgconfig'[2020-02-18T12:50:06.055Z] QT_PLUGIN_PATH= '/home/jenkins/install-prefix/lib64/plugins'[2020-02-18T12:50:06.055Z] QML2_IMPORT_PATH  = '/home/jenkins/install-prefix/lib64/qml'[2020-02-18T12:50:06.055Z] QT_SELECT = 'qt5'[2020-02-18T12:50:06.055Z] LANG  = 'en_US.UTF-8'[2020-02-18T12:50:06.055Z] XDG_CURRENT_DESKTOP   = 'KDE'[2020-02-18T12:50:06.055Z] ASAN_OPTIONS  = 'detect_leaks=0:new_delete_type_mismatch=0:detect_odr_violation=0:stack-use-after-scope=0:alloc_dealloc_mismatch=0'[2020-02-18T12:50:06.055Z] [2020-02-18T12:50:06.055Z] [2020-02-18T12:50:06.055Z] == Commencing Configuration:[2020-02-18T12:50:06.055Z] cmake -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=ON -DCMAKE_INSTALL_PREFIX="/home/jenkins//install-prefix/" -DBUILD_COVERAGE=ON -DECM_ENABLE_SANITIZERS='address'  "/home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 SUSEQt5.12"[2020-02-18T12:50:06.313Z] -- The C compiler identification is GNU 9.2.1[2020-02-18T12:50:06.313Z] -- The CXX compiler identification is GNU 9.2.1[2020-02-18T12:50:06.313Z] -- Check for working C compiler: /usr/bin/cc[2020-02-18T12:50:06.574Z] -- Check for working C compiler: /usr/bin/cc -- works[2020-02-18T12:50:06.574Z] -- Detecting C compiler ABI info[2020-02-18T12:50:06.574Z] -- Detecting C compiler ABI info - done[2020-02-18T12:50:06.574Z] -- Detecting C compile features[2020-02-18T12:50:06.574Z] -- Detecting C compile features - done[2020-02-18T12:50:06.574Z] -- Check for working CXX compiler: /usr/bin/c++[2020-02-18T12:50:06.832Z] -- Check for working CXX compiler: /usr/bin/c++ -- works[2020-02-18T12:50:06.832Z] -- Detecting CXX compiler ABI info[2020-02-18T12:50:06.832Z] -- Detecting CXX compiler ABI info - done[2020-02-18T12:50:06.832Z] -- Detecting CXX compile features[2020-02-18T12:50:06.832Z] -- Detecting CXX compile features - done[2020-02-18T12:50:06.832Z] Installing in /home/jenkins/install-prefix. Run /home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 SUSEQt5.12/build/prefix.sh to set the environment for plasma-desktop.[2020-02-18T12:50:06.832Z] -- Looking for __GLIBC__[2020-02-18T12:50:07.086Z] -- Looking for __GLIBC__ - found[2020-02-18T12:50:07.086Z] -- Performing Test _OFFT_IS_64BIT[2020-02-18T12:50:07.086Z] -- Performing Test _OFFT_IS_64BIT - Success[2020-02-18T12:50:07.086Z] -- Performing Test HAVE_DATE_TIME[2020-02-18T12:50:07.342Z] -- Performing Test HAVE_DATE_TIME - Success[2020-02-18T12:50:07.342Z] -- Found KF5Auth: /home/jenkins/install-prefix/lib64/cmake/KF5Auth/KF5AuthConfig.cmake (found version "5.68.0") [2020-02-18T12:50:07.342Z] Installing in /home/jenkins/install-prefix. Run /home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 SUSEQt5.12/build/prefix.sh to set the environment for plasma-desktop.[2020-02-18T12:50:07.342Z] Installing in /home/jenkins/install-prefix. Run /home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 SUSEQt5.12/build/prefix.sh to set the environment for 

D27471: [plasma-session] Avoid potential race condition starting a service

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> broulik wrote in startup.cpp:429
> Could probably use an `m_service.isEmpty()` check since it will still 
> needlessly do a `NameHasOwner` call

All usages of this class (which is just the 1 case to start ksmserver!) have a 
name passed.

Without a name it wouldn't ever finish.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D27471

To: davidedmundson, #plasma, apol
Cc: broulik, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D27471: [plasma-session] Avoid potential race condition starting a service

[Kai Uwe Broulik]

broulik added inline comments.

INLINE COMMENTS

> startup.cpp:429
>  {
> +if 
> (QDBusConnection::sessionBus().interface()->isServiceRegistered(m_serviceId)) 
> {
> +qCDebug(PLASMA_SESSION) << m_process << "already running";

Could probably use an `m_service.isEmpty()` check since it will still 
needlessly do a `NameHasOwner` call

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D27471

To: davidedmundson, #plasma, apol
Cc: broulik, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

KDE CI: Plasma » kde-cli-tools » stable-kf5-qt5 FreeBSDQt5.13 - Build # 35 - Failure!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/kde-cli-tools/job/stable-kf5-qt5%20FreeBSDQt5.13/35/
 Project:
stable-kf5-qt5 FreeBSDQt5.13
 Date of build:
Tue, 18 Feb 2020 12:38:29 +
 Build duration:
4 min 9 sec and counting
   CONSOLE OUTPUT
  [...truncated 198 lines...][2020-02-18T12:42:23.414Z] PATH  = '/usr/home/jenkins/install-prefix/bin:/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/sbin:/usr/local/bin:/home/jenkins/bin'[2020-02-18T12:42:23.414Z] JOB_BASE_NAME = 'stable-kf5-qt5 FreeBSDQt5.13'[2020-02-18T12:42:23.414Z] EDITOR= 'vi'[2020-02-18T12:42:23.414Z] ENV   = '/home/jenkins/.shrc'[2020-02-18T12:42:23.414Z] HUDSON_HOME   = '/home/production/data'[2020-02-18T12:42:23.414Z] BUILD_ID  = '35'[2020-02-18T12:42:23.414Z] OLDPWD= '/usr/home/jenkins'[2020-02-18T12:42:23.414Z] PWD   = '/usr/home/jenkins/workspace/Plasma/kde-cli-tools/stable-kf5-qt5 FreeBSDQt5.13'[2020-02-18T12:42:23.414Z] _ = '/usr/local/bin/java'[2020-02-18T12:42:23.414Z] TERM  = 'su'[2020-02-18T12:42:23.414Z] NODE_LABELS   = 'FreeBSD Builder Iota FreeBSDQt5.13'[2020-02-18T12:42:23.414Z] BUILD_TAG = 'jenkins-Plasma-kde-cli-tools-stable-kf5-qt5 FreeBSDQt5.13-35'[2020-02-18T12:42:23.414Z] BUILD_DISPLAY_NAME= '#35'[2020-02-18T12:42:23.415Z] RUN_DISPLAY_URL   = 'https://build.kde.org/job/Plasma/job/kde-cli-tools/job/stable-kf5-qt5%20FreeBSDQt5.13/35/display/redirect'[2020-02-18T12:42:23.415Z] USER  = 'jenkins'[2020-02-18T12:42:23.415Z] HOME  = '/home/jenkins'[2020-02-18T12:42:23.415Z] BUILD_URL = 'https://build.kde.org/job/Plasma/job/kde-cli-tools/job/stable-kf5-qt5%20FreeBSDQt5.13/35/'[2020-02-18T12:42:23.415Z] JOB_URL   = 'https://build.kde.org/job/Plasma/job/kde-cli-tools/job/stable-kf5-qt5%20FreeBSDQt5.13/'[2020-02-18T12:42:23.415Z] HUDSON_URL= 'https://build.kde.org/'[2020-02-18T12:42:23.415Z] SHELL = '/bin/sh'[2020-02-18T12:42:23.415Z] WORKSPACE = '/usr/home/jenkins/workspace/Plasma/kde-cli-tools/stable-kf5-qt5 FreeBSDQt5.13'[2020-02-18T12:42:23.415Z] STAGE_NAME= 'Configuring Build'[2020-02-18T12:42:23.415Z] NODE_NAME = 'FreeBSD Builder Iota'[2020-02-18T12:42:23.415Z] BLOCKSIZE = 'K'[2020-02-18T12:42:23.415Z] EXECUTOR_NUMBER   = '0'[2020-02-18T12:42:23.415Z] SHLVL = '0'[2020-02-18T12:42:23.415Z] CMAKE_PREFIX_PATH = '/usr/home/jenkins/install-prefix:/usr/:/usr/local/'[2020-02-18T12:42:23.415Z] LD_LIBRARY_PATH   = '/usr/home/jenkins/install-prefix/lib:/usr/home/jenkins/install-prefix/libdata'[2020-02-18T12:42:23.415Z] PKG_CONFIG_PATH   = '/usr/home/jenkins/install-prefix/lib/pkgconfig:/usr/home/jenkins/install-prefix/libdata/pkgconfig:/usr/libdata/pkgconfig:/usr/local/libdata/pkgconfig'[2020-02-18T12:42:23.415Z] QT_PLUGIN_PATH= '/usr/home/jenkins/install-prefix/lib/plugins:/usr/local/lib/qca-qt5'[2020-02-18T12:42:23.415Z] QML2_IMPORT_PATH  = '/usr/home/jenkins/install-prefix/lib/qml'[2020-02-18T12:42:23.415Z] XDG_DATA_DIRS = '/usr/home/jenkins/install-prefix/share:/usr/share:/usr/local/share'[2020-02-18T12:42:23.415Z] XDG_CONFIG_DIRS   = '/usr/home/jenkins/install-prefix/etc/xdg:/usr/local/etc/xdg'[2020-02-18T12:42:23.415Z] QT_SELECT = 'qt5'[2020-02-18T12:42:23.415Z] LANG  = 'en_US.UTF-8'[2020-02-18T12:42:23.415Z] XDG_CURRENT_DESKTOP   = 'KDE'[2020-02-18T12:42:23.415Z] ASAN_OPTIONS  = 'detect_leaks=0:new_delete_type_mismatch=0:detect_odr_violation=0:stack-use-after-scope=0:alloc_dealloc_mismatch=0:detect_container_overflow=0:symbolize=1'[2020-02-18T12:42:23.415Z] ASAN_SYMBOLIZER_PATH  = '/usr/local/bin/llvm-symbolizer'[2020-02-18T12:42:23.415Z] [2020-02-18T12:42:23.415Z] [2020-02-18T12:42:23.415Z] == Commencing Configuration:[2020-02-18T12:42:23.415Z] cmake -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=ON -DCMAKE_INSTALL_PREFIX="/home/jenkins/install-prefix/"  "/usr/home/jenkins/workspace/Plasma/kde-cli-tools/stable-kf5-qt5 FreeBSDQt5.13"[2020-02-18T12:42:23.674Z] -- The C compiler identification is Clang 8.0.1[2020-02-18T12:42:23.931Z] -- The CXX compiler identification is Clang 8.0.1[2020-02-18T12:42:23.931Z] -- Check for working C compiler: /usr/bin/cc[2020-02-18T12:42:24.192Z] -- Check for working C compiler: /usr/bin/cc -- works[2020-02-18T12:42:24.192Z] -- Detecting C compiler ABI info[2020-02-18T12:42:24.526Z] -- Detecting C compiler ABI info - done[2020-02-18T12:42:24.526Z] -- Detecting C compile features[2020-02-18T12:42:24.526Z] -- Detecting C compile features - done[2020-02-18T12:42:24.526Z] -- Check for working CXX compiler: 

KDE CI: Plasma » drkonqi » stable-kf5-qt5 WindowsMSVCQt5.14 - Build # 4 - Still Failing!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/drkonqi/job/stable-kf5-qt5%20WindowsMSVCQt5.14/4/
 Project:
stable-kf5-qt5 WindowsMSVCQt5.14
 Date of build:
Tue, 18 Feb 2020 12:38:22 +
 Build duration:
3 min 11 sec and counting
   CONSOLE OUTPUT
  [...truncated 399 lines...][2020-02-18T12:41:15.125Z] [16/149] Building CXX object src\tests\crashtest\CMakeFiles\crashtest.dir\crashtest.cpp.obj[2020-02-18T12:41:16.525Z] [17/149] Building CXX object src\kdbgwin\CMakeFiles\kdbgwin.dir\kdbgwin_process.cpp.obj[2020-02-18T12:41:17.402Z] [18/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\attachmentclient.cpp.obj[2020-02-18T12:41:17.402Z] [19/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\commands\bugsearch.cpp.obj[2020-02-18T12:41:17.402Z] [20/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\connection.cpp.obj[2020-02-18T12:41:17.402Z] [21/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\bugclient.cpp.obj[2020-02-18T12:41:17.402Z] [22/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\commentclient.cpp.obj[2020-02-18T12:41:17.402Z] [23/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\commands\bugupdate.cpp.obj[2020-02-18T12:41:17.677Z] [24/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\exceptions.cpp.obj[2020-02-18T12:41:17.677Z] [25/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\clientbase.cpp.obj[2020-02-18T12:41:17.677Z] [26/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\bugfieldclient.cpp.obj[2020-02-18T12:41:17.677Z] [27/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\commands\jsoncommand.cpp.obj[2020-02-18T12:41:17.950Z] [28/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\productclient.cpp.obj[2020-02-18T12:41:18.892Z] [29/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\commands\newattachment.cpp.obj[2020-02-18T12:41:19.299Z] [30/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\commands\newbug.cpp.obj[2020-02-18T12:41:19.557Z] [31/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\models\bugfield.cpp.obj[2020-02-18T12:41:19.557Z] [32/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\models\comment.cpp.obj[2020-02-18T12:41:19.557Z] [33/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\clients\commands\querycommand.cpp.obj[2020-02-18T12:41:19.557Z] [34/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\models\logindetails.cpp.obj[2020-02-18T12:41:19.820Z] [35/149] Linking CXX static library lib\drkonqi_backtrace_parser.lib[2020-02-18T12:41:20.146Z] [36/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\models\bug.cpp.obj[2020-02-18T12:41:20.406Z] [37/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\bugzilla_debug.cpp.obj[2020-02-18T12:41:20.674Z] [38/149] Automatic MOC for target gdbbacktracelinetest[2020-02-18T12:41:20.956Z] [39/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\models\product.cpp.obj[2020-02-18T12:41:20.956Z] [40/149] Building CXX object src\bugzillaintegration\libbugzilla\CMakeFiles\qbugzilla.dir\qbugzilla_autogen\mocs_compilation.cpp.obj[2020-02-18T12:41:20.956Z] [41/149] Building CXX object src\tests\CMakeFiles\gdbbacktracelinetest.dir\gdbbacktracelinetest_autogen\mocs_compilation.cpp.obj[2020-02-18T12:41:21.228Z] [42/149] Linking CXX static library lib\qbugzilla.lib[2020-02-18T12:41:21.497Z] [43/149] Automatic MOC for target backtraceparsertest[2020-02-18T12:41:21.497Z] [44/149] Linking CXX executable bin\crashtest.exe[2020-02-18T12:41:21.768Z] [45/149] Automatic MOC for target backtraceparsertest_manual[2020-02-18T12:41:21.768Z] [46/149] Building CXX object src\kdbgwin\CMakeFiles\kdbgwin.dir\callbacks.cpp.obj[2020-02-18T12:41:22.383Z] [47/149] Building CXX object src\kdbgwin\CMakeFiles\kdbgwin.dir\main.cpp.obj[2020-02-18T12:41:22.383Z] [48/149] Building CXX object src\kdbgwin\CMakeFiles\kdbgwin.dir\__\drkonqi_debug.cpp.obj[2020-02-18T12:41:22.646Z] [49/149] Building CXX object src\kdbgwin\CMakeFiles\kdbgwin.dir\abstract_generator.cpp.obj[2020-02-18T12:41:23.661Z] [50/149] Building CXX object src\kdbgwin\CMakeFiles\kdbgwin.dir\msvc_generator.cpp.obj[2020-02-18T12:41:23.925Z] [51/149] Building CXX object src\kdbgwin\CMakeFiles\kdbgwin.dir\kdbgwin_autogen\mocs_compilation.cpp.obj[2020-02-18T12:41:23.925Z] [52/149] 

D27471: [plasma-session] Avoid potential race condition starting a service

[David Edmundson]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:854b64fc130b: [plasma-session] Avoid hypothetical race 
condition starting a service (authored by davidedmundson).

CHANGED PRIOR TO COMMIT
  https://phabricator.kde.org/D27471?vs=75902=75907#toc

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27471?vs=75902=75907

REVISION DETAIL
  https://phabricator.kde.org/D27471

AFFECTED FILES
  startkde/plasma-session/startup.cpp
  startkde/plasma-session/startup.h

To: davidedmundson, #plasma, apol
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

KDE CI: Plasma » kde-cli-tools » stable-kf5-qt5 SUSEQt5.12 - Build # 58 - Failure!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/kde-cli-tools/job/stable-kf5-qt5%20SUSEQt5.12/58/
 Project:
stable-kf5-qt5 SUSEQt5.12
 Date of build:
Tue, 18 Feb 2020 12:38:32 +
 Build duration:
1 min 29 sec and counting
   CONSOLE OUTPUT
  [...truncated 257 lines...][2020-02-18T12:39:59.631Z] JOB_NAME  = 'Plasma/kde-cli-tools/stable-kf5-qt5 SUSEQt5.12'[2020-02-18T12:39:59.631Z] LESS_ADVANCED_PREPROCESSOR = 'no'[2020-02-18T12:39:59.631Z] RUN_DISPLAY_URL   = 'https://build.kde.org/job/Plasma/job/kde-cli-tools/job/stable-kf5-qt5%20SUSEQt5.12/58/display/redirect'[2020-02-18T12:39:59.631Z] JOB_DISPLAY_URL   = 'https://build.kde.org/job/Plasma/job/kde-cli-tools/job/stable-kf5-qt5%20SUSEQt5.12/display/redirect'[2020-02-18T12:39:59.631Z] WORKSPACE = '/home/jenkins/workspace/Plasma/kde-cli-tools/stable-kf5-qt5 SUSEQt5.12'[2020-02-18T12:39:59.631Z] HUDSON_URL= 'https://build.kde.org/'[2020-02-18T12:39:59.631Z] LESSCLOSE = 'lessclose.sh %s %s'[2020-02-18T12:39:59.631Z] NODE_NAME = 'Docker Swarm-56bcb8013cc5'[2020-02-18T12:39:59.631Z] HOST  = '56bcb8013cc5'[2020-02-18T12:39:59.631Z] LESSOPEN  = 'lessopen.sh %s'[2020-02-18T12:39:59.631Z] USER  = 'jenkins'[2020-02-18T12:39:59.631Z] MORE  = '-sl'[2020-02-18T12:39:59.631Z] CSHEDIT   = 'emacs'[2020-02-18T12:39:59.631Z] SHLVL = '2'[2020-02-18T12:39:59.631Z] BUILD_TAG = 'jenkins-Plasma-kde-cli-tools-stable-kf5-qt5 SUSEQt5.12-58'[2020-02-18T12:39:59.631Z] WINDOWMANAGER = 'xterm'[2020-02-18T12:39:59.631Z] PAGER = 'less'[2020-02-18T12:39:59.631Z] EXECUTOR_NUMBER   = '0'[2020-02-18T12:39:59.631Z] CVS_RSH   = 'ssh'[2020-02-18T12:39:59.631Z] LIBGL_DEBUG   = 'quiet'[2020-02-18T12:39:59.631Z] STAGE_NAME= 'Configuring Build'[2020-02-18T12:39:59.631Z] NODE_LABELS   = 'Docker Swarm-56bcb8013cc5 SUSEQt5.12'[2020-02-18T12:39:59.631Z] XDG_DATA_DIRS = '/home/jenkins/install-prefix/share:/usr/share:/usr/local/share:/usr/share'[2020-02-18T12:39:59.631Z] CONFIG_SITE   = '/usr/share/site/x86_64-unknown-linux-gnu'[2020-02-18T12:39:59.631Z] PATH  = '/home/jenkins/install-prefix/bin:/home/jenkins//bin:/usr/local/bin:/usr/bin:/bin:/usr/lib/mit/bin:/usr/lib/mit/sbin'[2020-02-18T12:39:59.631Z] PROFILEREAD   = 'true'[2020-02-18T12:39:59.631Z] MAIL  = '/var/spool/mail/jenkins'[2020-02-18T12:39:59.631Z] HOSTTYPE  = 'x86_64'[2020-02-18T12:39:59.631Z] BUILD_ID  = '58'[2020-02-18T12:39:59.631Z] LESSKEY   = '/usr/etc/lesskey.bin'[2020-02-18T12:39:59.631Z] OLDPWD= '/home/jenkins'[2020-02-18T12:39:59.631Z] LC_CTYPE  = 'C.UTF-8'[2020-02-18T12:39:59.631Z] CMAKE_PREFIX_PATH = '/home/jenkins/install-prefix:/usr/:/usr/local/'[2020-02-18T12:39:59.631Z] LD_LIBRARY_PATH   = '/home/jenkins/install-prefix/lib64'[2020-02-18T12:39:59.631Z] PKG_CONFIG_PATH   = '/home/jenkins/install-prefix/lib64/pkgconfig:/usr/lib/pkgconfig:/usr/lib64/pkgconfig'[2020-02-18T12:39:59.631Z] QT_PLUGIN_PATH= '/home/jenkins/install-prefix/lib64/plugins'[2020-02-18T12:39:59.631Z] QML2_IMPORT_PATH  = '/home/jenkins/install-prefix/lib64/qml'[2020-02-18T12:39:59.631Z] QT_SELECT = 'qt5'[2020-02-18T12:39:59.631Z] LANG  = 'en_US.UTF-8'[2020-02-18T12:39:59.631Z] XDG_CURRENT_DESKTOP   = 'KDE'[2020-02-18T12:39:59.631Z] ASAN_OPTIONS  = 'detect_leaks=0:new_delete_type_mismatch=0:detect_odr_violation=0:stack-use-after-scope=0:alloc_dealloc_mismatch=0'[2020-02-18T12:39:59.631Z] [2020-02-18T12:39:59.631Z] [2020-02-18T12:39:59.631Z] == Commencing Configuration:[2020-02-18T12:39:59.631Z] cmake -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=ON -DCMAKE_INSTALL_PREFIX="/home/jenkins//install-prefix/" -DBUILD_COVERAGE=ON -DECM_ENABLE_SANITIZERS='address'  "/home/jenkins/workspace/Plasma/kde-cli-tools/stable-kf5-qt5 SUSEQt5.12"[2020-02-18T12:39:59.631Z] -- The C compiler identification is GNU 9.2.1[2020-02-18T12:39:59.887Z] -- The CXX compiler identification is GNU 9.2.1[2020-02-18T12:39:59.887Z] -- Check for working C compiler: /usr/bin/cc[2020-02-18T12:39:59.887Z] -- Check for working C compiler: /usr/bin/cc -- works[2020-02-18T12:39:59.887Z] -- Detecting C compiler ABI info[2020-02-18T12:39:59.887Z] -- Detecting C compiler ABI info - done[2020-02-18T12:39:59.887Z] -- Detecting C compile features[2020-02-18T12:39:59.887Z] -- Detecting C compile features - done[2020-02-18T12:39:59.887Z] -- Check for working CXX compiler: /usr/bin/c++[2020-02-18T12:39:59.887Z] -- Check for working CXX compiler: /usr/bin/c++ -- works[2020-02-18T12:39:59.887Z] -- 

D27472: [plasma-session] Cleanup StartServiceJob

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> apol wrote in startup.cpp:438
> Are you sure showing an error message on error is not useful anymore?

we have one on the  startDetached retun value.

once we double fork and detach I didn't think we would get any other warnings.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D27472

To: davidedmundson, #plasma, apol
Cc: apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D27472: [plasma-session] Cleanup StartServiceJob

[Aleix Pol Gonzalez]

apol accepted this revision.
apol added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> startup.cpp:438
> -
> -connect(p, ::errorOccurred, this, [this] 
> (QProcess::ProcessError error) {
> -qCWarning(PLASMA_SESSION) << "error on process" << error << 
> m_process << m_args;

Are you sure showing an error message on error is not useful anymore?

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D27472

To: davidedmundson, #plasma, apol
Cc: apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D27199: [Applet]Update layout based on T10470

[George Vogiatzis]

gvgeo added a comment.


  The worst problem I could found, by ignoring the margins.
  Theme "Underworld".
  F8110467: Screenshot_20200218_134914.png 


REPOSITORY
  R116 Plasma Network Management Applet

REVISION DETAIL
  https://phabricator.kde.org/D27199

To: gvgeo, #plasma, #vdg, jgrulich, manueljlin
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27199: [Applet]Update layout based on T10470

[George Vogiatzis]

gvgeo added a comment.


  1 picture, 1 problem, 2 issues.
  F8110442: Screenshot_20200218_124006.png 

  I understand why margins exist. But this, is wrong.

REPOSITORY
  R116 Plasma Network Management Applet

REVISION DETAIL
  https://phabricator.kde.org/D27199

To: gvgeo, #plasma, #vdg, jgrulich, manueljlin
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27199: [Applet]Update layout based on T10470

[George Vogiatzis]

gvgeo updated this revision to Diff 75906.
gvgeo edited the test plan for this revision.
gvgeo added a dependency: D27392: [Applet]Add info messages..
gvgeo added a comment.


  TODO from the last update.
  Improved comments.
  Removed fill anchor from listview.

REPOSITORY
  R116 Plasma Network Management Applet

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27199?vs=75831=75906

BRANCH
  arcpatch-D27199_2

REVISION DETAIL
  https://phabricator.kde.org/D27199

AFFECTED FILES
  applet/contents/ui/ConnectionItem.qml
  applet/contents/ui/DetailsText.qml
  applet/contents/ui/Header.qml
  applet/contents/ui/ListItem.qml
  applet/contents/ui/PasswordField.qml
  applet/contents/ui/PopupDialog.qml
  applet/contents/ui/SwitchButton.qml
  applet/contents/ui/Toolbar.qml
  applet/contents/ui/main.qml
  libs/models/networkmodel.h
  libs/models/networkmodelitem.cpp

To: gvgeo, #plasma, #vdg, jgrulich, manueljlin
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart