D8267: Correcting symlinks between col- and row-resize and split_v and _hBug 384864

2018-06-14 Thread Jack Ostroff
ostroffjh added a comment.


  Fixed those two symlinks and updated the diff.  Anything else blocking this?

REPOSITORY
  R113 Oxygen Theme

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

To: ostroffjh, #plasma, apol
Cc: tfoster, apol, asturmlechner, cfeck, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart


D8267: Correcting symlinks between col- and row-resize and split_v and _hBug 384864

2018-06-14 Thread Jack Ostroff
ostroffjh updated this revision to Diff 36177.
ostroffjh added a comment.


  - Merge branch 'master' into fixlinks
  - fix cursors/src/symlinks/col-resize and .../row-resize
  
  Updating D8267: Correcting symlinks between col- and row-resize and split_v 
and _h
  
==
  
  Bug 384864

REPOSITORY
  R113 Oxygen Theme

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8267?vs=20651&id=36177

BRANCH
  fixlinks

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

AFFECTED FILES
  cursors/KDE_Classic/cursors/col-resize
  cursors/KDE_Classic/cursors/row-resize
  cursors/Oxygen_Black/cursors/col-resize
  cursors/Oxygen_Black/cursors/row-resize
  cursors/Oxygen_Blue/cursors/col-resize
  cursors/Oxygen_Blue/cursors/row-resize
  cursors/Oxygen_White/cursors/col-resize
  cursors/Oxygen_White/cursors/row-resize
  cursors/Oxygen_Yellow/cursors/col-resize
  cursors/Oxygen_Yellow/cursors/row-resize
  cursors/Oxygen_Zion/cursors/col-resize
  cursors/Oxygen_Zion/cursors/row-resize
  cursors/src/symlinks/col-resize
  cursors/src/symlinks/row-resize

To: ostroffjh, #plasma, apol
Cc: tfoster, apol, asturmlechner, cfeck, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart


D12498: Fully remove `Application Name` from Details panel

2018-06-14 Thread Stefan Brüns
bruns added a comment.


  In D12498#278422 , @ngraham wrote:
  
  > Looks good to me, though the string needs to make clear that the bug report 
should go to the 3rd-party app developer, not to us! Otherwise we'll get a 
bunch of un-actionable bugs.
  
  
  I think its sufficiently clear. Directly below is the action id, which, as in 
reverse URL format, is a good indicator whom to address (mind the target 
audience again). Also there hopefully is the "Vendor"/"Vendor URL" tag. There 
may be a few cases where a BR ends up in the KDE bugzilla, but I think we can 
deal with these (either file a bug report, or point the reporter to upstream).

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

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

To: sharvey, bruns, ngraham, davidedmundson
Cc: davidedmundson, bruns, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D12498: Fully remove `Application Name` from Details panel

2018-06-14 Thread Nathaniel Graham
ngraham added a comment.


  Looks good to me, though the string needs to make clear that the bug report 
should go to the 3rd-party app developer, not to us! Otherwise we'll get a 
bunch of un-actionable bugs.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

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

To: sharvey, bruns, ngraham, davidedmundson
Cc: davidedmundson, bruns, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D12498: Fully remove `Application Name` from Details panel

2018-06-14 Thread Stefan Brüns
bruns added a comment.


  In D12498#278380 , @ngraham wrote:
  
  > Ah yeah, that makes sense. Instead of "Missing", how about "Not available; 
please file a bug with the developer of this software"  ...Or something like 
that.
  
  
  Somewhat to verbose, and it needs 'Description' somewhere in the sentence, as 
it looks like this:
  
Action:  
 
  
  e.g.
  
Action:  'Description' not provided, please file a bug report
 org.foo.bar

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

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

To: sharvey, bruns, ngraham, davidedmundson
Cc: davidedmundson, bruns, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D12498: Fully remove `Application Name` from Details panel

2018-06-14 Thread Nathaniel Graham
ngraham added a comment.


  Ah yeah, that makes sense. Instead of "Missing", how about "Not available; 
please file a bug with the developer of this software"  ...Or something like 
that.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

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

To: sharvey, bruns, ngraham, davidedmundson
Cc: davidedmundson, bruns, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D12278: WIP: [Colors KCM] Port to new design

2018-06-14 Thread Andres Betts
abetts added a comment.


  Ping? Is this KCM doing alright?

REPOSITORY
  R119 Plasma Desktop

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

To: broulik, #plasma, #vdg
Cc: mart, abetts, ngraham, davidedmundson, plasma-devel, ragreen, Pitel, 
ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol


D12498: Fully remove `Application Name` from Details panel

2018-06-14 Thread Stefan Brüns
bruns added a comment.


  In D12498#278285 , @ngraham wrote:
  
  > This concept (and the proposed label) is suitable for a developer audience, 
not a regular user audience.
  >
  > We have the same issue in Discover when upstreams don't ship enough 
ApStream metadata. We omit anything that's missing instead of labeling the 
missing sections with some kind of admonishment to the upstreams. Developers 
like us should take care of browbeating non-compliant upstream software 
vendors; we shouldn't push that work onto users.
  
  
  The description is visible in the "Details" panel only, which is targeted at 
a Developer/Admin audience. So the "users should not be bothered" IMHO does not 
apply here. If you just omit it, nobody will report the problematic actions, 
and nobody will fill an upstream bug report.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

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

To: sharvey, bruns, ngraham, davidedmundson
Cc: davidedmundson, bruns, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D12498: Fully remove `Application Name` from Details panel

2018-06-14 Thread Nathaniel Graham
ngraham added a comment.


  In D12498#278282 , @bruns wrote:
  
  > In D12498#272379 , @ngraham 
wrote:
  >
  > > I feel like if we go with "missing", users will blame us and we'll get 
bug reports ("If it's missing, why can't you provide it? Duh!").
  >
  >
  > Its simple "We can not provide it because upstream was to lazy to read and 
follow the spec. Please raise a upstream bug report". If we just omit it, we 
encourage upstream to keep shipping faulty polkit action files.
  
  
  This concept (and the proposed label) is suitable for a developer audience, 
not a regular user audience.
  
  We have the same issue in Discover when upstreams don't ship enough ApStream 
metadata. We omit anything that's missing instead of labeling the missing 
sections with some kind of admonishment to the upstreams. Developers like us 
should take care of browbeating non-compliant upstream software vendors; we 
shouldn't push that work onto users.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

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

To: sharvey, bruns, ngraham, davidedmundson
Cc: davidedmundson, bruns, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D13529: [minimizeAll] Do not try to restore window state on model change

2018-06-14 Thread Anthony Fieroni
anthonyfieroni updated this revision to Diff 36160.

REPOSITORY
  R114 Plasma Addons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D13529?vs=36142&id=36160

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

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

To: anthonyfieroni, davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart


D12498: Fully remove `Application Name` from Details panel

2018-06-14 Thread Stefan Brüns
bruns added a comment.


  In D12498#272379 , @ngraham wrote:
  
  > I feel like if we go with "missing", users will blame us and we'll get bug 
reports ("If it's missing, why can't you provide it? Duh!").
  
  
  Its simple "We can not provide it because upstream was to lazy to read and 
follow the spec. Please raise a upstream bug report". If we just omit it, we 
encourage upstream to keep shipping faulty polkit action files.
  
  Btw, which action files are affected by this problem - I have not a single 
action file on my system which lacks a description tag.

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

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

To: sharvey, bruns, ngraham, davidedmundson
Cc: davidedmundson, bruns, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D13095: Scren brightness follow a quatratic progression

2018-06-14 Thread Nathaniel Graham
ngraham added a comment.


Lowest setting:
$ cat /sys/class/backlight/intel_backlight/actual_brightness 
0
Next setting:
$ cat /sys/class/backlight/intel_backlight/actual_brightness 
2
  
  Same values as `/sys/class/backlight/*/brightness`, in other words.

REPOSITORY
  R122 Powerdevil

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

To: thsurrel, #plasma, broulik, ngraham
Cc: sharvey, zzag, ngraham, romangg, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D13529: [minimizeAll] Do not try to restore window state on model change

2018-06-14 Thread David Edmundson
davidedmundson added a comment.


  Sounds better.

REPOSITORY
  R114 Plasma Addons

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

To: anthonyfieroni, davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart


D13529: [minimizeAll] Do not try to restore window state on model change

2018-06-14 Thread Anthony Fieroni
anthonyfieroni added a comment.


  On previous version onStackingOrderChanged will not restore windows state, it 
just stops the effect. So onActiveTaskChanged to replace onDataChanged?

REPOSITORY
  R114 Plasma Addons

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

To: anthonyfieroni, davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart


D13433: Introduce ActionToolbar

2018-06-14 Thread Marco Martin
This revision was automatically updated to reflect the committed changes.
Closed by commit R169:fd8fb834843c: remove the dead code (authored by mart).

CHANGED PRIOR TO COMMIT
  https://phabricator.kde.org/D13433?vs=35847&id=36143#toc

REPOSITORY
  R169 Kirigami

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D13433?vs=35847&id=36143

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

AFFECTED FILES
  src/controls/ActionToolBar.qml
  src/controls/InlineMessage.qml

To: mart, #kirigami, hein
Cc: apol, davidedmundson, plasma-devel, mart, hein


D13529: [minimizeAll] Do not try to restore window state on model change

2018-06-14 Thread David Edmundson
davidedmundson requested changes to this revision.
davidedmundson added a comment.
This revision now requires changes to proceed.


  I'm happy to admit there's a bug there, but just removing the code isn't 
fixing it.
  
  Now you're restoring windows as soon as any property, such as a download 
progressing in any window changes.

REPOSITORY
  R114 Plasma Addons

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

To: anthonyfieroni, davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart


D13529: [minimizeAll] Do not try to restore window state on model change

2018-06-14 Thread Anthony Fieroni
anthonyfieroni created this revision.
anthonyfieroni added reviewers: davidedmundson, Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
anthonyfieroni requested review of this revision.

REVISION SUMMARY
  onDataChanged: some cases deactivate can not be called or persistent model 
index is invalidated (this should not happen in this case or it's a bug) which 
result in effect to stay active

TEST PLAN
  1. Click on applet
  2. Open new windows that rearrange task manager
  3. Try to unminimize window will result in unminimizing other

REPOSITORY
  R114 Plasma Addons

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

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

To: anthonyfieroni, davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart


D13433: Introduce ActionToolbar

2018-06-14 Thread David Edmundson
davidedmundson added inline comments.

INLINE COMMENTS

> ActionToolBar.qml:31
> + *
> + * @inherits RowLayout
> + * @since 2.5

No it doesn't?

REPOSITORY
  R169 Kirigami

BRANCH
  mart/actiontoolbar

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

To: mart, #kirigami, hein
Cc: apol, davidedmundson, plasma-devel, mart, hein


KDE CI: Plasma plasma-desktop kf5-qt5 SUSEQt5.10 - Build # 92 - Still unstable!

2018-06-14 Thread CI System
BUILD UNSTABLE
 Build URL
https://build.kde.org/job/Plasma%20plasma-desktop%20kf5-qt5%20SUSEQt5.10/92/
 Project:
Plasma plasma-desktop kf5-qt5 SUSEQt5.10
 Date of build:
Thu, 14 Jun 2018 08:02:24 +
 Build duration:
16 min and counting
   JUnit Tests
  Name: (root) Failed: 1 test(s), Passed: 11 test(s), Skipped: 0 test(s), Total: 12 test(s)Failed: TestSuite.test_kio_fonts
   Cobertura Report
  
   Project Coverage Summary
  
   Name
  PackagesFilesClassesLinesConditionalsCobertura Coverage Report13%
(10/79)11%
(51/446)11%
(51/446)10%
(3734/36523)9%
(2312/26039)Coverage Breakdown by Package
Name
   FilesClassesLinesConditionalsapplets.kicker.plugin0%
(0/39)0%
(0/39)0%
(0/3326)0%
(0/2746)applets.kimpanel.backend.ibus0%
(0/1)0%
(0/1)0%
(0/31)0%
(0/10)applets.kimpanel.backend.ibus.ibus150%
(0/10)0%
(0/10)0%
(0/1092)0%
(0/661)applets.kimpanel.backend.scim0%
(0/1)0%
(0/1)0%
(0/641)0%
(0/397)applets.kimpanel.plugin0%
(0/3)0%
(0/3)0%
(0/44)0%
(0/26)applets.pager.plugin0%
(0/4)0%
(0/4)0%
(0/330)0%
(0/224)applets.taskmanager.plugin0%
(0/4)0%
(0/4)0%
(0/313)0%
(0/294)applets.taskmanager.plugin.smartlaunchers0%
(0/4)0%
(0/4)0%
(0/291)0%
(0/212)applets.trash.plugin0%
(0/6)0%
(0/6)0%
(0/112)0%
(0/62)attica-kde.kdeplugin0%
(0/1)0%
(0/1)0%
(0/124)0%
(0/112)containments.desktop.plugins.desktop0%
(0/3)0%
(0/3)0%
(0/49)0%
(0/18)containments.desktop.plugins.folder25%
(6/24)25%
(6/24)38%
(849/2218)27%
(418/1561)containments.desktop.plugins.folder.autotests100%
(8/8)100%
(8/8)100%
(562/562)57%
(310/544)dataengines.kimpanel0%
(0/12)0%
(0/12)0%
(0/336)0%
(0/135)imports.activitymanager0%
(0/4)0%
(0/4)0%
(0/388)0%
(0/252)kaccess0%
(0/3)0%
(0/3)0%
(0/583)0%
(0/310)kcms.access0%
(0/1)0%
(0/1)0%
(0/306)0%
(0/80)kcms.activities0%
(0/7)0%
  

Re: KDE CI: Dependency Build Extragear kf5-qt5 SUSEQt5.9 - Build # 106 - Failure!

2018-06-14 Thread Ben Cooksley
On Tue, Jun 12, 2018 at 11:12 PM, Ben Cooksley  wrote:

> Can someone please look into and correct this breakage please?
>
> We've recently had a Binary Incompatible system update rollout (thanks Qt
> private interfaces) so getting this fixed quickly is rather important.
>
> If Plasma no longer supports Qt 5.9 then someone will need to inform the
> Mycroft and active-window-control applet developers that their software is
> no longer supported by Plasma as that is the Qt version they currently
> request (yes, due to the dependency tree in Plasma, these two applets
> depend on KWin, along with the whole Plasma Desktop)
>

Due to a lack of response to my request I have now disabled the offending
test.

This is the second time in recent history where the Plasma team has failed
to respond to mails from me regarding the buildability of your software,
which is quite disappointing. I'd like an explanation for that before the
impacted test is restored.

Regards,
Ben


>
> Cheers,
> Ben
>
> -- Forwarded message --
> From: CI System 
> Date: Tue, Jun 12, 2018 at 11:06 PM
> Subject: KDE CI: Dependency Build Extragear kf5-qt5 SUSEQt5.9 - Build #
> 106 - Failure!
> To: bcooks...@kde.org
>
>
> *BUILD FAILURE*
> Build URL https://build.kde.org/job/Dependency%20Build%20Extragear%
> 20kf5-qt5%20SUSEQt5.9/106/
> Project: Dependency Build Extragear kf5-qt5 SUSEQt5.9
> Date of build: Tue, 12 Jun 2018 08:30:03 +
> Build duration: 2 hr 36 min and counting
> * CONSOLE OUTPUT *
> [...truncated 67.12 MB...]
> Scanning dependencies of target testColorCorrectNightColor
> [ 95%] Building CXX object autotests/integration/CMakeFil
> es/testColorCorrectNightColor.dir/colorcorrect_nightcolor_test.cpp.o
> [ 95%] Building CXX object autotests/integration/CMakeFil
> es/testColorCorrectNightColor.dir/testColorCorrectNightColor
> _autogen/mocs_compilation.cpp.o
> [ 95%] Linking CXX executable ../../bin/testColorCorrectNightColor
> [ 95%] Built target testColorCorrectNightColor
> Scanning dependencies of target testIdleInhibition
> [ 95%] Building CXX object autotests/integration/CMakeFil
> es/testIdleInhibition.dir/idle_inhibition_test.cpp.o
> [ 95%] Building CXX object autotests/integration/CMakeFil
> es/testIdleInhibition.dir/testIdleInhibition_autogen/
> mocs_compilation.cpp.o
> [ 95%] Linking CXX executable ../../bin/testIdleInhibition
> [ 95%] Built target testIdleInhibition
> Scanning dependencies of target testIdleInhibition_waylandonly
> [ 95%] Building CXX object autotests/integration/CMakeFil
> es/testIdleInhibition_waylandonly.dir/idle_inhibition_test.cpp.o
> [ 95%] Building CXX object autotests/integration/CMakeFil
> es/testIdleInhibition_waylandonly.dir/testIdleInhibition_way
> landonly_autogen/mocs_compilation.cpp.o
> [ 95%] Linking CXX executable ../../bin/testIdleInhibition_waylandonly
> [ 95%] Built target testIdleInhibition_waylandonly
> Scanning dependencies of target testDontCrashCursorPhysicalSizeEmpty
> [ 95%] Building CXX object autotests/integration/CMakeFil
> es/testDontCrashCursorPhysicalSizeEmpty.dir/dont_crash_
> cursor_physical_size_empty.cpp.o
> [ 96%] Building CXX object autotests/integration/CMakeFil
> es/testDontCrashCursorPhysicalSizeEmpty.dir/testDontCrashCur
> sorPhysicalSizeEmpty_autogen/mocs_compilation.cpp.o
> [ 96%] Linking CXX executable ../../bin/testDontCrashCursorP
> hysicalSizeEmpty
> [ 96%] Built target testDontCrashCursorPhysicalSizeEmpty
> Scanning dependencies of target testDontCrashCursorPhysicalSiz
> eEmpty_waylandonly
> [ 96%] Building CXX object autotests/integration/CMakeFil
> es/testDontCrashCursorPhysicalSizeEmpty_waylandonly.dir/
> dont_crash_cursor_physical_size_empty.cpp.o
> [ 96%] Building CXX object autotests/integration/CMakeFil
> es/testDontCrashCursorPhysicalSizeEmpty_waylandonly.dir/test
> DontCrashCursorPhysicalSizeEmpty_waylandonly_autogen/mocs_
> compilation.cpp.o
> [ 96%] Linking CXX executable ../../bin/testDontCrashCursorP
> hysicalSizeEmpty_waylandonly
> [ 96%] Built target testDontCrashCursorPhysicalSizeEmpty_waylandonly
> Scanning dependencies of target testShade
> [ 96%] Building CXX object autotests/integration/CMakeFil
> es/testShade.dir/shade_test.cpp.o
> [ 96%] Building CXX object autotests/integration/CMakeFil
> es/testShade.dir/testShade_autogen/mocs_compilation.cpp.o
> [ 96%] Linking CXX executable ../../bin/testShade
> [ 96%] Built target testShade
> Scanning dependencies of target testWindowSelection_waylandonly
> [ 96%] Building CXX object autotests/integration/CMakeFil
> es/testWindowSelection_waylandonly.dir/window_selection_test.cpp.o
> [ 96%] Building CXX object autotests/integration/CMakeFil
> es/testWindowSelection_waylandonly.dir/testWindowSelection_w
> aylandonly_autogen/mocs_compilation.cpp.o
> [ 96%] Linking CXX executable ../../bin/testWindowSelection_waylandonly
> Scanning dependencies of target testPlasmaWindow
> Scanning dependencies of target testDontCrashAuroraeDestroyDeco
> [ 96%] Building CXX object autotests/integration/

[kwin] autotests/integration: Disable unit test which fails to compile on the CI system.

2018-06-14 Thread Ben Cooksley
Git commit e6cdf966ff10d875ceaf65c85356ed7732b585a4 by Ben Cooksley.
Committed on 14/06/2018 at 08:00.
Pushed by bcooksley into branch 'master'.

Disable unit test which fails to compile on the CI system.
This test is blocking the ability of the CI system to return to service for 
Extragear projects on some platforms.

This commit may not be reverted without the explicit consent of Sysadmin.

CCMAIL: plasma-devel@kde.org
CCMAIL: k...@kde.org

M  +1-1autotests/integration/CMakeLists.txt

https://commits.kde.org/kwin/e6cdf966ff10d875ceaf65c85356ed7732b585a4

diff --git a/autotests/integration/CMakeLists.txt 
b/autotests/integration/CMakeLists.txt
index dde39ee8b..dca17779b 100644
--- a/autotests/integration/CMakeLists.txt
+++ b/autotests/integration/CMakeLists.txt
@@ -72,7 +72,7 @@ if (XCB_ICCCM_FOUND)
 integrationTest(NAME testQuickTiling SRCS quick_tiling_test.cpp LIBS 
XCB::ICCCM)
 integrationTest(NAME testGlobalShortcuts SRCS globalshortcuts_test.cpp 
LIBS XCB::ICCCM)
 integrationTest(NAME testSceneQPainter SRCS scene_qpainter_test.cpp LIBS 
XCB::ICCCM)
-integrationTest(NAME testSceneQPainterShadow SRCS 
scene_qpainter_shadow_test.cpp LIBS XCB::ICCCM)
+#integrationTest(NAME testSceneQPainterShadow SRCS 
scene_qpainter_shadow_test.cpp LIBS XCB::ICCCM)
 
 if (KWIN_BUILD_ACTIVITIES)
 integrationTest(NAME testActivities SRCS activities_test.cpp LIBS 
XCB::ICCCM)