D7823: Add FindGLIB2.cmake and FindPulseAudio.cmake to ECM

Christophe Giboudeaux noreply at phabricator.kde.org
Sat Sep 16 15:25:27 UTC 2017


cgiboudeaux added a comment.


  Looks good. What's your migration plan ?
  if you don't want to bump the ECM requirements for applications looking for pulseaudio, you have to set some compatibility/deprecated vars (eg: kmix uses uppercase variables)

INLINE COMMENTS

> FindPulseAudio.cmake:109
> +
> +if(PulseAudio_FOUND AND NOT TARGET PulseAudio::PulseAudio)
> +  add_library(PulseAudio::PulseAudio UNKNOWN IMPORTED)

PulseAudio_FOUND is undefined if you add this block here.

Move it below find_package_handle_standard_args(...)

REPOSITORY
  R240 Extra CMake Modules

REVISION DETAIL
  https://phabricator.kde.org/D7823

To: marten, #frameworks, #build_system
Cc: cgiboudeaux, cfeck, heikobecker
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20170916/5c1d88d1/attachment.html>


More information about the Kde-buildsystem mailing list