Make Qt5X11Extras optional
ClosedPublic

Authored by asturmlechner on Jul 11 2019, 12:11 AM.

Details

Summary

It is only required with X11 and XCB-XKB.

Diff Detail

Repository
R336 KTouch
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 13906
Build 13924: arc lint + arc unit
asturmlechner created this revision.Jul 11 2019, 12:11 AM
Restricted Application added a project: KDE Edu. · View Herald TranscriptJul 11 2019, 12:11 AM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
asturmlechner requested review of this revision.Jul 11 2019, 12:11 AM
asturmlechner edited the summary of this revision. (Show Details)Jul 11 2019, 12:12 AM
gottfried requested changes to this revision.Jul 11 2019, 6:28 AM

Thanks for the patch. If you change the dependencies in the CMake files, you also need to update the README so they stay in sync.

CMakeLists.txt
40 ↗(On Diff #61557)

Is it possible to reformulate this line like line 41 (without the QUIET, though)? I would prefer to request Qt features in a consistent way.

This revision now requires changes to proceed.Jul 11 2019, 6:28 AM
asturmlechner added inline comments.Jul 11 2019, 6:35 AM
CMakeLists.txt
40 ↗(On Diff #61557)

For optional dependencies, it is better to do it this way. Otherwise, if the dependency is disabled or not found, Qt5_FOUND is set to 0, which does not necessarily lead to failure later on (it can, though) but at a minimum leads to conflicting FeatureSummary output (Qt5 not found or sth. along those lines).

gottfried added inline comments.Jul 11 2019, 7:02 AM
CMakeLists.txt
40 ↗(On Diff #61557)

Okay.

update README.md

gottfried accepted this revision.Jul 14 2019, 8:22 AM

Good to go.

This revision is now accepted and ready to land.Jul 14 2019, 8:22 AM
This revision was automatically updated to reflect the committed changes.