Summary: | Knotify not logging notifications to file | ||
---|---|---|---|
Product: | [Frameworks and Libraries] frameworks-knotifications | Reporter: | Prime Jyothi <primejyothi> |
Component: | general | Assignee: | Martin Klapetek <mklapetek> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | gabriel.caudrelier, kdelibs-bugs |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Fedora RPMs | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/knotifications/cd83fadb9888ff344f7b89deda5b17ecbba94b1c | Version Fixed In: | 5.31.0 |
Description
Prime Jyothi
2016-05-16 14:46:13 UTC
This bug is still current. to fix: uncomment the line in https://cgit.kde.org/knotifications.git/tree/src/knotificationmanager.cpp:80 ~~~ code //FIXME: port and reenable // addPlugin(new NotifyByLogfile(this)); ~~~ I believe this was ported already ... a couple of years ago. https://cgit.kde.org/knotifications.git/log/src/notifybylogfile.cpp Just need to re-enable it. I can confirm it correctly compiles with the "NotifyByLogfile" plugin added, using Qt 5.8 and without KDELibs4Support. Needs someone able to test it, though. works fine for me. Although you probably meant someone more official. Thanks for the confirmation. https://phabricator.kde.org/D4357 Git commit cd83fadb9888ff344f7b89deda5b17ecbba94b1c by Christoph Feck. Committed on 02/02/2017 at 00:26. Pushed by cfeck into branch 'master'. Re-add logging notifications to files Thanks to Gabriel Caudrelier for the investigation. Differential Revision: https://phabricator.kde.org/D4357 FIXED-IN: 5.31.0 M +1 -1 src/CMakeLists.txt M +1 -2 src/knotificationmanager.cpp https://commits.kde.org/knotifications/cd83fadb9888ff344f7b89deda5b17ecbba94b1c |