D28765: KSettings::Dialog: add support for KPluginInfos without a KService

rikmills added a comment. In D28765#673191 , @wbauer wrote: > This caused a crash in openSUSE: > https://bugs.kde.org/show_bug.cgi?id=421566 The same crash can been seen in Kubuntu when trying to launch the external software-properties

D26226: [KListOpenFilesJob] Use QString::splitRef()

rikmills added a comment. Fails to build. https://build.kde.org/job/Frameworks/job/kcoreaddons/job/kf5-qt5%20SUSEQt5.12/135/console https://launchpadlibrarian.net/457575337/buildlog_ubuntu-eoan-amd64.kcoreaddons_5.65.0+p19.10+git20191227.0406-0_BUILDING.txt.gz REPOSITORY R244 KCo

D24754: Fix KNSCore::Cache fetching when initialising Engine

rikmills accepted this revision. This revision is now accepted and ready to land. REPOSITORY R304 KNewStuff BRANCH fix-knscore-cache (branched from master) REVISION DETAIL https://phabricator.kde.org/D24754 To: leinir, rikmills, #frameworks, #knewstuff Cc: kde-frameworks-devel, LeGast00n,

D24754: Fix KNSCore::Cache fetching when initialising Engine

rikmills added a comment. Patched knewstuff build with this and tested. Switching between KCMs in systemsettings now seems to pull the correct KNS results. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D24754 To: leinir, rikmills, #frameworks, #knewstuff Cc: kde-

D22143: Add proper logging using ECMQtDeclareLoggingCategory

rikmills added a comment. This is installing BOTH /etc/xdg/kactivities-stats.categories AND/usr/share/qlogging-categories5/kactivities-stats.categories which is surely wrong? REPOSITORY R159 KActivities Statistics REVISION DETAIL https://phabricator.kde.org/D22143 To: meven, ivan, #fra

D22346: Should we install bluezqt_dbustypes.h?

rikmills added a comment. > We detected this thing because we have a test in our packaging which checks if the installed headers can be built and fails if they are missing bits. I ran the test in question with the attached patch and it doesn't fail. The KDE CI has the acc test as well:

D12378: Properly search for NetworkManager

Herald added a subscriber: kde-frameworks-devel. REPOSITORY R239 KDELibs4Support REVISION DETAIL https://phabricator.kde.org/D12378 To: jgrulich, #frameworks Cc: kde-frameworks-devel, rikmills, #frameworks, michaelh, ngraham, bruns

D19888: Add runtime guard that pages are KCMs in KCMultiDialog

rikmills added a comment. Patch seemed to fix the crash. I don't use Kontact (apart from testing things like this), but on a quick test all seems functional once more. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D19888 To: davidedmundson Cc: rikmills, kde-framew

D19821: Fix breeze dialog background with Qt 5.12.2

rikmills added a comment. I guess it is very likely, but all the same for the record I will request that this results in a 5.56.2 release of plasma-framework. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D19821 To: fvogt, #plasm

D18610: Introduce KF5AuthCore

rikmills added subscribers: sitter, rikmills. rikmills added a comment. Is this not breaking ABI? even if just moving to another lib? @sitter FYI I get https://paste.ubuntu.com/p/NHsbqg7DbN/ REPOSITORY R283 KAuth REVISION DETAIL https://phabricator.kde.org/D18610 To: aacid, apol Cc:

D18178: fix environment setup of klanguagename test

rikmills accepted this revision. rikmills added a comment. This revision is now accepted and ready to land. Build and passes on Kubuntu CI now. https://launchpadlibrarian.net/405807438/buildlog_ubuntu-disco-amd64.kconfigwidgets_5.54.0+p19.04+git20190111.1036-0_BUILDING.txt.gz REPOSITORY

D10446: Add KLanguageName

rikmills added a comment. Kubuntu CI builds 2/4 Test #3: klanguagenametest ***Failed0.03 sec - Start testing of KLanguageNameTest * Config: Using QtTest library 5.11.3, Qt 5.11.3 (x86_64-little_endian-lp64 shared (dynamic) release build; by GCC 8.2.0)

D16867: Don't generate appstream data for files that lack a description

rikmills added a comment. And we have a casualty from this ;) Marble 18.12 branch: > CMake Warning at /usr/lib/x86_64-linux-gnu/cmake/KF5Package/KF5PackageMacros.cmake:68 (message): >couldn't generate metainfo for org.kde.plasma.worldmap: Package type >"Plasma/Wallpaper

D6513: Add support for Attica tags support

rikmills added a comment. In D6513#326071 , @ngraham wrote: > FYI, this included an ABI change to `SearchRequest` that broke Discover: https://bugs.kde.org/show_bug.cgi?id=398412 I see this revision has been reverted in master in 293ae244

D13488: Make XDGShellV5 co-compilable with XDGWMBase

rikmills added a comment. In libkf5waylandclient5 and libkf5waylandserver5 I see the previously emitted symbol xdg_shell_interface@Base is now reported as MISSING. Can you confirm this is private and/or not an ABI break? Build symbols output: https://paste.kde.org/pfpkps1mb#line-109

D11925: Add "SkipSwitcher" to API

rikmills added a comment. D11924 and D11926 have been pushed, so kwin master branch builds are now broken with: 14:49:26 /home/jenkins/workspace/Plasma kwin kf5-qt5 SUSEQt5.10/abstract_client.cpp: In member functi

D12064: Don't alter memory management to hide an item

rikmills added a comment. Applied this to 5.44 in test package for Ubuntu 18.04 Switched containment folderview -> desktop and back several times. No longer any crash. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D12064 To: davidedmundson,

D10671: Fix auto-mount regression

rikmills added a comment. I assume this would be safe to backport to 5.43, should it be accepted? REPOSITORY R245 Solid REVISION DETAIL https://phabricator.kde.org/D10671 To: dkhlestkov, #frameworks, broulik Cc: rikmills, #frameworks, michaelh

D10054: Fix: View jumps when Scroll past end of document is enabled

rikmills added a comment. This change appears to cause ktexteditor to fail it's vimode_completion autotests. From KDE CI: https://build.kde.org/job/Frameworks%20ktexteditor%20kf5-qt5%20SUSEQt5.10/35/testReport/junit/(root)/TestSuite/vimode_completion/ Ubuntu autotests: F5707888: tes

D7891: expose the colorset for the stylesheet

rikmills added a comment. Since this change, polkit-kde-agent-1 fails to build from source with the error: > In file included from /usr/include/KF5/KIconThemes/KIconLoader:1:0, > from /<>/polkit-kde-agent-1-5.10.95+p17.10+git20170929.1419/AuthDialog.cpp:37: > /usr/include/KF5/K

Re: Kirigami in Frameworks

On 01/07/17 12:48, David Faure wrote: > On vendredi 30 juin 2017 15:16:17 CEST Marco Martin wrote: >> Hi, >> I have moved it, should be good to go. >> one sidenote (hoping is not a problem) for historical reasons, the >> tarballs were called kirigami2-version instead of just kirigami (or >> distrib

D5889: Keep QIcon::fromTheme in main thread

rikmills added a comment. This breaks build on Neon and Kubuntu CI with Qt 5.7.1 http://build.neon.kde.org/job/xenial_unstable_plasma_breeze_bin_amd64/120/console 05:10:59 In file included from /workspace/build/build-qt4/kstyle/moc_breezeshadowhelper.cpp:9:0, 05:10:59