do not stderr that NoDisplay is set
ClosedPublic

Authored by sitter on Jul 7 2017, 3:58 PM.

Details

Summary

KF5PackageMacros turns stderr output into cmake level warnings.

setting nodisplay is perfectly reasonable and doesn't warrant a warning.
additionally these warnings are turned into test data by the CI, so
every plasmoid with nodisplay is causing unstable builds for no reason.

iff kpackagetool had a -o outputfile argument and stopped using stdout
for file output there'd be a place to print non-essential info such as
nodisplay being off. alas, no such luck.

Diff Detail

Repository
R290 KPackage
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
sitter created this revision.Jul 7 2017, 3:58 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJul 7 2017, 3:58 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
dfaure accepted this revision.Jul 8 2017, 11:19 AM
This revision is now accepted and ready to land.Jul 8 2017, 11:19 AM
This revision was automatically updated to reflect the committed changes.