<table><tr><td style="">marten created this revision.<br />Restricted Application added projects: Frameworks, Build System.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D7823" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>These modules are used in a number of places within Frameworks, Plasma and dependencies:</p>

<p>FindGLIB2:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">plasma-desktop/applets/kimpanel/cmake/FindGLIB2.cmake
kdelibs4support/cmake/modules/FindGLIB2.cmake
ecm/attic/modules/FindGLIB2.cmake
phonon/cmake/FindGLIB2.cmake
polkit-qt/cmake/modules/FindGLIB2.cmake</pre></div>

<p>FindPulseAudio:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">kdelibs4support/cmake/modules/FindPulseAudio.cmake
ecm/attic/modules/FindPulseAudio.cmake
phonon/cmake/FindPulseAudio.cmake</pre></div>

<p>Apart from those in Phonon, each set is all the same apart from minor formatting differences.</p>

<p>These two modules will also be required by KMix when it is finally ported away from KDElibs4Support.  Since these common files are used in multiple places, it makes sense to have them included in ECM instead of making local copies.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Built and installed ECM with these two modules added, configured KMix with KDELibs4Support removed.<br />
GLIB2 and PulseAudio are found correctly using the modules installed by ECM.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7823" rel="noreferrer">https://phabricator.kde.org/D7823</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>find-modules/FindGLIB2.cmake<br />
find-modules/FindPulseAudio.cmake</div></div></div><br /><div><strong>To: </strong>marten, Frameworks, Build System<br /></div>