Re-enable Log-to-File for KNotifications
ClosedPublic
Actions

Authored by cfeck on Jan 31 2017, 1:11 AM.

Details

Reviewers

mck182

Summary

According to https://bugs.kde.org/show_bug.cgi?id=363138#c1 we "forgot" to re-enable the Log-to-File KNotify plugin.

Test Plan

Only compilation tested by me. Gabriel confirms it works for him.

Lint

Lint Skipped

Unit

Unit Tests Skipped

cfeck retitled this revision from to Re-enable Log-to-File for KNotifications.

cfeck updated this object.

cfeck edited the test plan for this revision. (Show Details)

cfeck set the repository for this revision to R289 KNotifications.

Restricted Application added a subscriber: Frameworks. · View Herald Transcript

cfeck removed R289 KNotifications as the repository for this revision.

cfeck edited the test plan for this revision. (Show Details)

cfeck added a reviewer: mck182.

cfeck set the repository for this revision to R289 KNotifications.

Looks good, should be fine.

This revision is now accepted and ready to land.Feb 1 2017, 3:52 PM

Submitted, but was not closed automatically.

			Path	Packages
M			src/CMakeLists.txt (2 lines)
M			src/knotificationmanager.cpp (3 lines)