Differential D14805

[AppStream Runner] Use categorized logging
ClosedPublic
Actions

Authored by bruns on Aug 13 2018, 7:59 PM.

Details

Reviewers
apol
ngraham
Group Reviewers
Plasma
Commits
R120:86e3748f904a: [AppStream Runner] Use categorized logging
Summary

Add category org.kde.plasma.runner.appstream, like done for the Services
runner, i.e. org.kde.plasma.runner.services

Diff Detail

Repository
R120 Plasma Workspace
Branch
appstream_runner_2
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 1793
Build 1811: arc lint + arc unit
bruns created this revision.Aug 13 2018, 7:59 PM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 13 2018, 7:59 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
bruns requested review of this revision.Aug 13 2018, 7:59 PM
apol accepted this revision.Aug 15 2018, 10:06 AM
This revision is now accepted and ready to land.Aug 15 2018, 10:06 AM
apol requested changes to this revision.Aug 15 2018, 10:07 AM

Please squash all changes

This revision now requires changes to proceed.Aug 15 2018, 10:07 AM
bruns requested review of this revision.Aug 22 2018, 1:05 PM
bruns added a reviewer: ngraham.
bruns updated this revision to Diff 40773.Aug 31 2018, 2:29 PM

rebase after D14807

bruns updated this revision to Diff 40777.Aug 31 2018, 2:40 PM

Change all other (one) occurences of qWarning(), there is no qDebug() etc.

apol accepted this revision.Aug 31 2018, 3:21 PM
This revision is now accepted and ready to land.Aug 31 2018, 3:21 PM
Closed by commit R120:86e3748f904a: [AppStream Runner] Use categorized logging (authored by bruns). · Explain WhyAug 31 2018, 3:23 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 39635

View Options

runners/appstream/CMakeLists.txt

Loading...
View Options

runners/appstream/appstreamrunner.cpp

Loading...