marten updated this revision to Diff 19570. marten added a comment.
Modules updated with standard header, documentation and copyright; endif() used throughout. Yes, Phonon does not use ECM but the modules there are very different to those in ECM anyway. Within Plasma+applications these modules are needed by plasma-desktop (applets/kimpanel and kcms/phonon) as well as KMix. kcms/phonon does not have a local copy, so this will break or a local copy will be needed if it is ported away from KDELibs4Support. REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7823?vs=19531&id=19570 REVISION DETAIL https://phabricator.kde.org/D7823 AFFECTED FILES find-modules/FindGLIB2.cmake find-modules/FindPulseAudio.cmake To: marten, #frameworks, #build_system Cc: cgiboudeaux, cfeck, heikobecker