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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
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.