<table><tr><td style="">marten updated this revision to Diff 19611.<br />marten added a comment.
</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><div><p>Diff updated to move library target definition after find_package_handle_standard_args.</p>

<p>If these modules are accepted into ECM then KMix needs a further commit anyway, in order to remove the final requirement for KDELibs4Support.  So the correct variable names can be set at this point (I have already tested this configuration).  Other packages requiring PA and/or GLib may need to do the same when transitioning to use the ECM versions.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D7823?vs=19576&id=19611" rel="noreferrer">https://phabricator.kde.org/D7823?vs=19576&id=19611</a></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 /><strong>Cc: </strong>cgiboudeaux, cfeck, heikobecker<br /></div>