Respin request for KConfigWidgets

Hello, I would like to make a respin for kconfigwidgets because of the porting aid introduced in https://invent.kde.org/frameworks/kconfigwidgets/-/merge_requests/191. The change about making KF5::CoreAddons a public was discussed at the KF6 weekly. Because we already require KCoreAddons

KF6 weekly changes

Hello, regarding the low attendance at the KF6 weeklies, we decided that it is best to have this meeting every two weeks. With the recent Plasma sprint and previous meetings we were able to unblock lots of tasks, but they still need doing . To better structure the meetings, we will be

Aw: Re: kcm_bluetooth changed ?

re I can not rely on compatibility even > between minor versions, that is really not fun. > Perhaps the breakage is due to these commits [1-3]. Need to ask Alexander Lohnau if there is a proper solution. I added him to the Cc list. Sorry for the inconvenience

Aw: KF6 meeting notes 2021-10-26

Hello,   regarding the kcmutils core lib discussion I have a quick addition to the topic: KPluginSelector should get a KPluginMetaData based replacement. This will have a core model and a QML/QWidgets version. Meaning that it makes sense to introduce a core lib even in KF5 for this new class.

D19565: kconfig_compiler: new kcfgc args HeaderExtension & SourceExtension

alex added a comment. However the option seems pretty unused, even from a github search: https://github.com/search?q=HeaderExtension%3D+extension%3Akcfgc=Code=advsearch== > KF targets also consumers outside of KDE spheres Also I think there are currently quite a lot of kcfgc

D19565: kconfig_compiler: new kcfgc args HeaderExtension & SourceExtension

alex added a comment. I am wondering if there is really a need for it. SourceExtension seems completely unused and HeaderExtension is only used in okteta. Though in KDE code, Qt code (and most other that I know of) it is the ".h" extension is the most common one for headers. Are

Aw: Re: Deprecation wrappers for pre-KF6 time (was: Re: KF6 meeting notes 2021-06-05)

rs for pre-KF6 time (was: Re: KF6 meeting notes 2021-06-05) Am Samstag, 5. Juni 2021, 22:39:15 CEST schrieb Alexander Lohnau: > Or could we use the BUILD_DEPRECATED_SINCE variant in this case? Like we > already have to do with virtual methods. > > This way one would still be able to

KF6 meeting notes 2021-07-26

Hi all,   here are the notes from the KF6 weekly.   Regards Alexander   https://phabricator.kde.org/T12176: If a usecase comes up one could think if implementing a cross-platform autostart API. In its current state & the current usages it should be deprecated. The needed logic should be

Re: KService as a platform abstraction framework?

On Saturday, 3 July 2021 19:39:17 CEST Nicolas Fella wrote: > On 03.07.21 10:25, Volker Krause wrote: > > Hi, > > > > while looking at implementing a pretty straightforward KApplicationTrader/ > > KIO::ApplicationLauncherJob use ([1]) for Android, I found myself > > wondering > > whether we should

Re: Deprecation wrappers for pre-KF6 time (was: Re: KF6 meeting notes 2021-06-05)

Or could we use the BUILD_DEPRECATED_SINCE variant in this case? Like we already have to do with virtual methods. This way one would still be able to test it when compiling the framework without deprecations. Existing API users will still be informed about the deprecations by the compiler

Re: Re KIO workers

On Saturday, 5 June 2021 17:51:18 CEST David Faure wrote: > On samedi 5 juin 2021 16:29:10 CEST Volker Krause wrote: > > Do KIO slaves still need the klauncher/kinit loading mechanism? > > No. My request for developers to test KIO_FORK_SLAVES=1 > for daily use is so that apps fork kio worker

T12173: KService: provide solution to migrate away from KServiceTypeTrader/KMimeTypeTrader for loading plugins and parts

alex added a comment. > KRunner uses KServiceTypeTrader to load plugins. This is non-trivial to port because of the DBus runners It is ported and the old methods are deprecated TASK DETAIL https://phabricator.kde.org/T12173 To: alex Cc: alex, #frameworks, nicolasfella, dfaure, mart,

Proposing reroll of KNewStuff for Frameworks 5.76

Hello,   I would like to request a reroll of KNewStuff to include c15fbc05dbc275ef1bd85f1578c078f6a873909c, which fixes a severe issue for day to day use cases. Without it entries could end being marked as uninstalled when a KCM is reopened, while they are actually installed. Also entries

D29281: Deprecate defunct functions

alex added a comment. Ufff, never thought sth. simple like deprecating a few functions could cause such an issue! @davidre Thanks a lot for fixing this! REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29281 To: alex, #plasma, broulik, davidedmundson, vkrause,

D29050: KRunner fix prepare/teardown signals

alex closed this revision. REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29050 To: alex, meven, ngraham, broulik Cc: davidedmundson, cfeck, kde-frameworks-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, michaelh, ZrenBot,

D29281: Deprecate defunct functions

alex closed this revision. REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29281 To: alex, #plasma, broulik, davidedmundson, vkrause, meven Cc: kossebau, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29281: Deprecate defunct functions

alex marked an inline comment as done. REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29281 To: alex, #plasma, broulik, davidedmundson, vkrause, meven Cc: kossebau, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29281: Deprecate defunct functions

alex updated this revision to Diff 83119. alex added a comment. Fix typo in version number REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29281?vs=83115=83119 BRANCH deprecations (branched from master) REVISION DETAIL

D29281: Deprecate defunct functions

alex added inline comments. INLINE COMMENTS > meven wrote in querymatch.cpp:338 > Did you really need to remove this code ? The question this method should answer is: Does this match have a configuration interface? And the answer is no, because the functionality does not exist anymore. And if

D29281: Deprecate defunct functions

alex added inline comments. INLINE COMMENTS > kossebau wrote in abstractrunner.h:154 > Should be "5.0", not "5,0" here in the API dox text and elsewhere, no? (dot > instead of comma) Yes. I accidentally did it this way, because in German you use a comma :D > kossebau wrote in

D29281: Deprecate defunct functions

alex marked 9 inline comments as done. REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29281 To: alex, #plasma, broulik, davidedmundson, vkrause, meven Cc: kossebau, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29281: Deprecate defunct functions

alex updated this revision to Diff 83115. alex added a comment. Fix comma, KRUNNER_DEPRECATED_VERSION_BELATED macro REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29281?vs=83110=83115 BRANCH deprecations (branched from master) REVISION DETAIL

D29281: Deprecate defunct functions

alex added a reviewer: meven. REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29281 To: alex, #plasma, broulik, davidedmundson, vkrause, meven Cc: kossebau, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29050: KRunner fix prepare/teardown signals

alex added a comment. Now I understand where it is called and how it works exactly. May I ship this? REPOSITORY R308 KRunner BRANCH krunner_signal_bugfix (branched from master) REVISION DETAIL https://phabricator.kde.org/D29050 To: alex, meven, ngraham, broulik Cc: davidedmundson,

D29050: KRunner fix prepare/teardown signals

alex edited the summary of this revision. REPOSITORY R308 KRunner BRANCH krunner_signal_bugfix (branched from master) REVISION DETAIL https://phabricator.kde.org/D29050 To: alex, meven, ngraham, broulik Cc: davidedmundson, cfeck, kde-frameworks-devel, Orage, LeGast00n, The-Feren-OS-Dev,

D29281: Deprecate defunct functions

alex updated this revision to Diff 83110. alex added a comment. Change deprecation version from 5.70 to 5.71 REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29281?vs=81546=83110 BRANCH deprecations (branched from master) REVISION DETAIL

D29050: KRunner fix prepare/teardown signals

alex added a comment. > Repo is called milou Thanks! REPOSITORY R308 KRunner BRANCH krunner_signal_bugfix (branched from master) REVISION DETAIL https://phabricator.kde.org/D29050 To: alex, meven, ngraham, broulik Cc: davidedmundson, cfeck, kde-frameworks-devel, Orage,

D29050: KRunner fix prepare/teardown signals

alex added a comment. But can you please explain, where in the QML code the actual KRunner backend is used? Maybe then I can understand it better . REPOSITORY R308 KRunner BRANCH krunner_signal_bugfix (branched from master) REVISION DETAIL https://phabricator.kde.org/D29050 To:

D29050: KRunner fix prepare/teardown signals

alex added a comment. Thanks and I get why it is hard to review. Could @broulik please help/have a look at this. A issue I have with understanding some of the KRunner stuff is that I don't know where exactly the runner "backend" is used (from the QML side of things). Thanks 

D29050: KRunner fix prepare/teardown signals

alex edited the summary of this revision. REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29050 To: alex, meven, ngraham, broulik Cc: cfeck, kde-frameworks-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, michaelh, ZrenBot,

D29050: KRunner fix prepare/teardown signals

alex retitled this revision from "WIP KRunner fix prepare/teardown signals" to "KRunner fix prepare/teardown signals". REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29050 To: alex, meven, ngraham, broulik Cc: cfeck, kde-frameworks-devel, Orage, LeGast00n,

D29201: KCMUtils: Add option to append service file to list of arguments

alex abandoned this revision. alex added a comment. Abandoning this in favor of https://invent.kde.org/frameworks/kconfigwidgets/-/merge_requests/1 and a follow up PR. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D29201 To: alex, #plasma, ngraham, meven,

D29201: KCMUtils: Add option to append service file to list of arguments

alex added inline comments. INLINE COMMENTS > meven wrote in kpluginselector.cpp:855 > Seems like a property would make sense, after all it is about it, or a ref to > the KCModuleInfo I have created a PR on Gitlab for this REPOSITORY R295 KCMUtils REVISION DETAIL

D29201: KCMUtils: Add option to append service file to list of arguments

alex added inline comments. INLINE COMMENTS > meven wrote in kpluginselector.cpp:855 > moduleInfo is part of the ctor here, so the fileName is already available > indirectly. > A good thing would be to avoid having an option for such a niche usage > although legitimate. > moduleInfo is part

D29201: KCMUtils: Add option to append service file to list of arguments

alex added inline comments. INLINE COMMENTS > meven wrote in kpluginselector.cpp:855 > Adding a fileName field to KCModuleProxy would make more sense to me, and do > it by default. > Plus KCModuleProxy has already access to the fileName since it receives > moduleInfo. > But I am not a

D29201: KCMUtils: Add option to append service file to list of arguments

alex added a comment. Friendly Ping  REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D29201 To: alex, #plasma, ngraham, meven, broulik, mart Cc: mart, apol, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29123: Do not mark entry as uninstalled if uninstallation script failed

This revision was automatically updated to reflect the committed changes. Closed by commit R304:a2ecdd333c49: Do not mark entry as uninstalled if uninstallation script failed (authored by alex). REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex marked 4 inline comments as done. REPOSITORY R304 KNewStuff BRANCH arcpatch-D29123_1 REVISION DETAIL https://phabricator.kde.org/D29123 To: alex, #knewstuff, meven, ngraham, leinir Cc: leinir, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex updated this revision to Diff 82228. alex added a comment. Update docs I adjusted your suggestion a bit, the signalEntryChanged gets also emitted, when the user cancels the uninstallation :-). REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE

D29455: KNS: Deprecate isRemote method and handle parse error properly

This revision was automatically updated to reflect the committed changes. Closed by commit R304:9575f2defb27: KNS: Deprecate isRemote method and handle parse error properly (authored by alex). REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE

D29451: KNS: Do not mark entry as installed if install script failed

This revision was automatically updated to reflect the committed changes. Closed by commit R304:2838e55acd17: KNS: Do not mark entry as installed if install script failed (authored by alex). REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE

D29451: KNS: Do not mark entry as installed if install script failed

alex updated this revision to Diff 82180. alex added a comment. Use qOverload No problem :-) REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29451?vs=82177=82180 BRANCH arcpatch-D29451 REVISION DETAIL https://phabricator.kde.org/D29451

D29451: KNS: Do not mark entry as installed if install script failed

alex updated this revision to Diff 82177. alex added a comment. Fix connect Thanks, I wasn't aware of that until now :-) REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29451?vs=82100=82177 BRANCH arcpatch-D29451 REVISION DETAIL

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex updated this revision to Diff 82114. alex added a comment. Ensure bic REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29123?vs=81918=82114 BRANCH arcpatch-D29123_1 REVISION DETAIL https://phabricator.kde.org/D29123 AFFECTED FILES

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex added inline comments. INLINE COMMENTS > leinir wrote in installation.h:124 > Hmm... i find myself wondering what effect this has on BIC... something tells > me it might not be so great... Specifically, >

D29451: KNS: Do not mark entry as installed if install script failed

alex updated this revision to Diff 82100. alex added a comment. Make lambda inline, capture variables REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29451?vs=82004=82100 BRANCH handle_install_script_error (branched from master) REVISION DETAIL

D29455: KNS: Deprecate isRemote method and handle parse error properly

alex marked 2 inline comments as done. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D29455 To: alex, #knewstuff, ngraham, leinir Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29455: KNS: Deprecate isRemote method and handle parse error properly

alex updated this revision to Diff 82098. alex marked an inline comment as done. alex added a comment. Typo and error message If the error message is on two lines they aren't very readable, but that is an issue for another day :-). REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE

D29455: KNS: Deprecate isRemote method and handle parse error properly

alex updated this revision to Diff 82024. alex retitled this revision from "KNS: Remove isRemote method and handle parse error properly" to "KNS: Deprecate isRemote method and handle parse error properly". alex added a comment. Make isRemote deprecated REPOSITORY R304 KNewStuff CHANGES

D29447: Fix showing updates when the option is selected

alex added a comment. No problem, always happy to help  REPOSITORY R304 KNewStuff BRANCH fix-show-only-updates (branched from master) REVISION DETAIL https://phabricator.kde.org/D29447 To: leinir, #knewstuff, #plasma, bugseforuns, ngraham, #frameworks Cc: alex, kde-frameworks-devel,

D29455: KNS: Remove isRemote method and handle parse error properly

alex created this revision. alex added reviewers: KNewStuff, ngraham, leinir. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. alex requested review of this revision. REVISION SUMMARY The isRemote check and the FIXMEs are removed, instead the result from the

D29447: Fix showing updates when the option is selected

alex added inline comments. INLINE COMMENTS > itemsmodel.cpp:71 > +bool duplicate{false}; > +for (const EntryInternal : qAsConst(m_entries)) { > +if (existingEntry == entry) { On second thought why not just use: `bool duplicate = m_entries.contains(entry);` REPOSITORY R304

D29451: KNS: Do not mark entry as installed if install script failed

alex edited the summary of this revision. alex edited the test plan for this revision. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D29451 To: alex, #knewstuff, ngraham, leinir Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29451: KNS: Do not mark entry as installed if install script failed

alex created this revision. alex added reviewers: KNewStuff, ngraham, leinir. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. alex requested review of this revision. REVISION SUMMARY If the install script failed/was aborted the entry gets marked as

D29050: WIP KRunner fix prepare/teardown signals

alex added a comment. How should I proceed with this patch? REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29050 To: alex, meven, ngraham, broulik Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29447: Fix showing updates when the option is selected

alex added inline comments. INLINE COMMENTS > itemsmodel.cpp:71 > +bool duplicate{false}; > +for (const EntryInternal& existingEntry : m_entries) { > +if (existingEntry == entry) { Use qAsConst(m_entries) and space before & not after :-) REPOSITORY R304 KNewStuff REVISION

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex updated this revision to Diff 81918. alex added a comment. Merge branch master REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29123?vs=81476=81918 BRANCH arcpatch-D29123_1 REVISION DETAIL https://phabricator.kde.org/D29123 AFFECTED FILES

D29201: KCMUtils: Add option to append service file to list of arguments

alex updated this revision to Diff 81873. alex added a comment. Increase @since to 5.71 REPOSITORY R295 KCMUtils CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29201?vs=81856=81873 BRANCH service_path_append (branched from master) REVISION DETAIL

D29201: KCMUtils: Add option to append service file to list of arguments

alex added a reviewer: mart. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D29201 To: alex, #plasma, ngraham, meven, broulik, mart Cc: mart, apol, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29201: KCMUtils: Add option to append service file to list of arguments

alex updated this revision to Diff 81856. alex added a comment. Linebreak REPOSITORY R295 KCMUtils CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29201?vs=81855=81856 BRANCH service_path_append (branched from master) REVISION DETAIL https://phabricator.kde.org/D29201

D29201: KCMUtils: Add option to append service file to list of arguments

alex updated this revision to Diff 81855. alex added a comment. Improve documentation Is that what you had in mind :-) ? And should I maybe add a comment to make this the default in KF6? REPOSITORY R295 KCMUtils CHANGES SINCE LAST UPDATE

D29201: KCMUtils: Add option to append service file to list of arguments

alex added a comment. > Would it make sense to always pass it? Would it regress in any way? It would make sense, but it would change the existing behavior. If you use the method setConfigurationArguments then you expect your argument list to be the list of arguments you specified + an

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex marked 2 inline comments as done. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D29123 To: alex, #knewstuff, meven, ngraham, leinir Cc: leinir, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29201: KCMUtils: Add option to append service file to list of arguments

alex added a dependent revision: D29384: KCM Runners: Use setAppendServiceFile method for plugin selector. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D29201 To: alex, #plasma, ngraham, meven, broulik Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham,

D29201: KCMUtils: Add option to append service file to list of arguments

alex updated this revision to Diff 81799. alex added a comment. Little typo :-) REPOSITORY R295 KCMUtils CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29201?vs=81798=81799 BRANCH service_path_append (branched from master) REVISION DETAIL https://phabricator.kde.org/D29201

D29201: KCMUtils: Add option to append service file to list of arguments

alex updated this revision to Diff 81798. alex added a comment. Rename method REPOSITORY R295 KCMUtils CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29201?vs=81239=81798 BRANCH service_path_append (branched from master) REVISION DETAIL https://phabricator.kde.org/D29201

D29201: KCMUtils: Add option to append service file to list of arguments

alex edited the summary of this revision. alex added reviewers: meven, broulik. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D29201 To: alex, #plasma, ngraham, meven, broulik Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29101: KNewStuff: Fix file path and process call

This revision was automatically updated to reflect the committed changes. Closed by commit R304:4c6b3afcd80a: KNewStuff: Fix file path and process call (authored by alex). REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29101?vs=81467=81753 REVISION DETAIL

D29101: KNewStuff: Fix file path and process call

alex added a comment. Ups, that makes sense ^^ Thanks! REPOSITORY R304 KNewStuff BRANCH arcpatch-D29101 REVISION DETAIL https://phabricator.kde.org/D29101 To: alex, #knewstuff, ngraham, nicolasfella, elvisangelaccio, meven, mlaurent, leinir, anthonyfieroni Cc: anthonyfieroni,

D29101: KNewStuff: Fix file path and process call

alex added a comment. Friendly ping regarding the question :-) REPOSITORY R304 KNewStuff BRANCH arcpatch-D29101 REVISION DETAIL https://phabricator.kde.org/D29101 To: alex, #knewstuff, ngraham, nicolasfella, elvisangelaccio, meven, mlaurent, leinir Cc: anthonyfieroni,

D29175: DBus Runner: Add service property to request actions once

alex added inline comments. INLINE COMMENTS > meven wrote in dbusrunnertest.cpp:206 > You should have added a QSignalSpy to check prepare was called, and another > for requestAction to check it has been called only once. The concept behind this test is, that the runner is just loaded. If the

D29175: DBus Runner: Add service property to request actions once

This revision was automatically updated to reflect the committed changes. Closed by commit R308:7b6222d8a10b: DBus Runner: Add service property to request actions once (authored by alex). REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29175?vs=81623=81687

D29175: DBus Runner: Add service property to request actions once

alex added a dependent revision: D29320: Baloo Runner: Use X-Plasma-Request-Actions-Once property in service. REPOSITORY R308 KRunner BRANCH request_actions_once (branched from master) REVISION DETAIL https://phabricator.kde.org/D29175 To: alex, #plasma, meven, ngraham, broulik Cc:

D29050: WIP KRunner fix prepare/teardown signals

alex added a comment. > Adding a test would be the best way to demonstrate the issue and the fix. I debugged this and the debugger said that the `matchSessionComplete` method got called somewhere from QML, but I don't know from where. I also know only basic QML and the way the QML

D29175: DBus Runner: Add service property to request actions once

alex updated this revision to Diff 81623. alex added a comment. Create test REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29175?vs=81165=81623 BRANCH request_actions_once (branched from master) REVISION DETAIL https://phabricator.kde.org/D29175

D29050: WIP KRunner fix prepare/teardown signals

alex edited the test plan for this revision. REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29050 To: alex, meven, ngraham, broulik Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29281: Deprecate defunct functions

alex updated this revision to Diff 81546. alex added a comment. Wrap macro about single method, change deprecation text And thanks for the feedback and explanations :-). REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29281?vs=81543=81546 BRANCH

D29281: Deprecate defunct functions

alex updated this revision to Diff 81543. alex added a comment. Wrap implementations REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29281?vs=81539=81543 BRANCH deprecations (branched from master) REVISION DETAIL https://phabricator.kde.org/D29281

D29281: Deprecate defunct functions

alex added inline comments. INLINE COMMENTS > kossebau wrote in abstractrunner.h:154 > The text in the API dox can stay "Since 5.0". > > The only crititical thing where the number of an older, already released > version should not be used is the KRUNNER_ENABLE_DEPRECATED_SINCE macro. >

D29281: Deprecate defunct functions

alex updated this revision to Diff 81539. alex added a comment. Use mordern macros Thanks :-) REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29281?vs=81528=81539 BRANCH deprecations (branched from master) REVISION DETAIL

D29281: Deprecate defunct functions

alex updated this revision to Diff 81528. alex added a comment. Add missing KRUNNER_DEPRECATED REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29281?vs=81527=81528 BRANCH deprecations (branched from master) REVISION DETAIL

D29281: Deprecate defunct functions

alex created this revision. alex added reviewers: Plasma, broulik, davidedmundson, vkrause. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. alex requested review of this revision. REVISION SUMMARY As in T12163 explained

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex updated this revision to Diff 81476. alex added a comment. Display more technical information in error message REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29123?vs=81076=81476 BRANCH arcpatch-D29123 REVISION DETAIL

D29101: KNewStuff: Fix file path and process call

alex updated this revision to Diff 81467. alex added a comment. Get program in exclusive line And should this go to release/20.04? REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29101?vs=81062=81467 BRANCH arcpatch-D29101 REVISION DETAIL

D29201: KCMUtils: Add option to append service file to list of arguments

alex created this revision. alex added reviewers: Plasma, ngraham. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. alex requested review of this revision. REVISION SUMMARY By setting the new variable to true the service path gets appended to the list of

D29175: DBus Runner: Add service property to request actions once

alex added a comment. No, they are independent. I just mentioned this to avoid confusion when debugging. REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29175 To: alex, #plasma, meven, ngraham, broulik Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh,

D29050: WIP KRunner fix prepare/teardown signals

alex retitled this revision from "WIP KRunner: Fix Bug 420311" to "WIP KRunner fix prepare/teardown signals". REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D29050 To: alex, meven, ngraham, broulik Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29175: DBus Runner: Add service property to request actions once

alex created this revision. alex added reviewers: Plasma, meven, ngraham, broulik. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. alex requested review of this revision. REVISION SUMMARY With the `X-Plasma-Request-Actions-Once` property the plugin can

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex added inline comments. INLINE COMMENTS > leinir wrote in installation.cpp:632 > This will want to be more... question like... "The thing failed" isn't really > a question, not sure how the user's supposed to make an informed choice based > on that... Perhaps something like "The

D29101: KNewStuff: Fix file path and process call

alex added a dependent revision: D29123: Do not mark entry as uninstalled if uninstallation script failed. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D29101 To: alex, #knewstuff, ngraham, nicolasfella, elvisangelaccio, meven, mlaurent Cc: kde-frameworks-devel,

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex edited the summary of this revision. alex added a dependency: D29101: KNewStuff: Fix file path and process call. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D29123 To: alex, #knewstuff, meven, ngraham, leinir Cc: leinir, kde-frameworks-devel, LeGast00n,

D29123: Do not mark entry as uninstalled if uninstallation script failed

alex updated this revision to Diff 81076. alex retitled this revision from "WIP BUG: 420312. Do not mark entry as uninstalled if uninstallation script failed" to "Do not mark entry as uninstalled if uninstallation script failed". alex edited the summary of this revision. alex added a comment.

D29123: WIP BUG: 420312. Do not mark entry as uninstalled if uninstallation script failed

alex updated this revision to Diff 81075. alex added a comment. Merge branch 'bugfix_uninstall' REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29123?vs=80976=81075 BRANCH bugfix_install_uninstall_messages (branched from master) REVISION DETAIL

D29101: KNewStuff: Fix file path and process call

alex retitled this revision from "WIP Fix file path and process call" to "KNewStuff: Fix file path and process call". REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D29101 To: alex, #knewstuff, ngraham, nicolasfella, elvisangelaccio, meven, mlaurent Cc:

D29101: WIP BUG: 420312 Fix file path and process call

alex edited the summary of this revision. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D29101 To: alex, #knewstuff, ngraham, nicolasfella, elvisangelaccio, meven, mlaurent Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29101: WIP BUG: 420312 Fix file path and process call

alex updated this revision to Diff 81062. alex added a comment. Use KShell to split args REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29101?vs=80907=81062 BRANCH bugfix_uninstall (branched from master) REVISION DETAIL

D28582: KRunner: Show error message for Actions in dbus runner

alex added a comment. I am also going to try a friendly ping here :-). REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D28582 To: alex, #plasma, davidedmundson, broulik, ngraham Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29123: WIP BUG: 420312. Do not mark entry as uninstalled if uninstallation script failed

alex marked 3 inline comments as done. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D29123 To: alex, #knewstuff, meven, ngraham, leinir Cc: leinir, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29123: WIP BUG: 420312. Do not mark entry as uninstalled if uninstallation script failed

alex updated this revision to Diff 80976. alex added a comment. Use internal question system PS: I am not sure on which branch this should land, thats why I haven't edited the translations. REPOSITORY R304 KNewStuff CHANGES SINCE LAST UPDATE

D29123: WIP BUG: 420312. Do not mark entry as uninstalled if uninstallation script failed

alex added a comment. No problem :-). And good to know that the concept of this patch makes sense ^^. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D29123 To: alex, #knewstuff, meven, ngraham, leinir Cc: leinir, kde-frameworks-devel, LeGast00n, cblack, michaelh,

  1   2   >