I should close this task.
- Queries
- All Stories
- Search
- Advanced Search
Advanced Search
Jul 12 2019
ah, yes I did. It looks like dbus-daemon is used for a lot of plugin interfaces within KDE Connect.
no more dbus on Windows build of KNotifications
Jul 11 2019
Jul 10 2019
Jul 9 2019
updated
rebased
may I land this?
Updated with QT_ pre-procs for S_IFDIR, S_IFLNK and others.
Jul 5 2019
In D21660#481421, @bcooksley wrote:Where possible D-Bus should be avoided on Windows.
added handling for WIndows. I'll try it on linux and revert on this patch!
Jul 3 2019
should I land this?
I believe I have covered all the issues with the patch now. (:
renamed variables (start with m_ now)
In D22105#489954, @vonreth wrote:Any reason why you skip the symlinks?
I mean displaying thm should be fine, dolphin probably can also follow them.
Creating them on windows is a bit more problematic.
okay. Landing it. \o/
We still haven't landed KNotifications patch, so I'm waiting on that, since this patch uses SnoreToast call.
We could land it anyway, and instructions related to SnoreToast will just start working after landing the KNotifications patch(es)
Jul 2 2019
Jun 27 2019
Jun 26 2019
Jun 21 2019
Jun 20 2019
Jun 19 2019
removed executable key since it's deprecated ✂
Jun 18 2019
rebased
closing this as duplicate
rebased
awesome! landing this! 🎉
umm, phonon isn't used for Windows notifications with/without the upcoming notifybysnoretoast back-end. Can we not land this?
Jun 17 2019
Jun 14 2019
updated acc to workaround for MSVC2019 suggested by Hannah
added inline comment for it at L92 -- notifybysnore.cpp
rebased on upstream/master
Jun 13 2019
updated code incoming. I think I should make a new diff for further discussions, as this one is quite riddled with suggestions now. Are there any more issues with this patch or should I continue with a new one instead? I'm willing to fix it further if you have some suggestions! ✊
Jun 11 2019
Oh! Apologies, Pino. Actually I accidentally referred to you as toscanos from IRC, so I deleted that comment. I'll avoid deleting them from now on.
updated acc to new reviews by Pino and Hannah
Jun 10 2019
fixed indentation