Fix Formatter library CMake
ClosedPublic

Authored by ahiemstra on Mon, Mar 23, 10:57 AM.

Details

Summary

On older versions, install(TARGETS target PUBLIC_HEADER) does not seem
to work too well, so just use install(FILES) for now. In addition, the
Qml and Quick components were missing from the main CMake file.

Test Plan

Still builds and installs

Diff Detail

Repository
R111 KSysguard Library
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ahiemstra created this revision.Mon, Mar 23, 10:57 AM
Restricted Application added a project: Plasma. · View Herald TranscriptMon, Mar 23, 10:57 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ahiemstra requested review of this revision.Mon, Mar 23, 10:57 AM
romangg accepted this revision.Mon, Mar 23, 11:02 AM
romangg added a subscriber: romangg.

Compiles again on Neon.

CMakeLists.txt
30

Accidentally added QML and Quick?

This revision is now accepted and ready to land.Mon, Mar 23, 11:02 AM
romangg added inline comments.Mon, Mar 23, 11:06 AM
CMakeLists.txt
30

I removed experimentally the Qml and Quick package specifiers and it still compiled.

ahiemstra added inline comments.Mon, Mar 23, 11:23 AM
CMakeLists.txt
30

It's in response to the comment here: https://phabricator.kde.org/D27869#632172

This revision was automatically updated to reflect the committed changes.