Adapt FindKF5 to stricter checks in newer find_package_handle_standard_args
ClosedPublic

Authored by kossebau on Apr 22 2020, 1:43 PM.

Details

Test Plan

KF modules configure build as before, same some apps.

Diff Detail

Repository
R240 Extra CMake Modules
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau created this revision.Apr 22 2020, 1:43 PM
Restricted Application added projects: Frameworks, Build System. · View Herald TranscriptApr 22 2020, 1:43 PM
Restricted Application added subscribers: kde-buildsystem, kde-frameworks-devel. · View Herald Transcript
kossebau requested review of this revision.Apr 22 2020, 1:43 PM

Warning: I have not really investigated in full detail if the current code of FindKF5 is completely sane besides a quick check, more or less assume that it has worked all the time, so the new warning here can be silenced.

krop accepted this revision.Apr 22 2020, 1:55 PM
This revision is now accepted and ready to land.Apr 22 2020, 1:55 PM
apol added a subscriber: apol.Apr 22 2020, 3:05 PM

+1 makes sense

This revision was automatically updated to reflect the committed changes.