FindPulseAudio insufficient

[Matthew Woehlke]

Matthew Woehlke wrote:
> Matthew Woehlke wrote:
>> I've had a kdemultimedia build failure for a while. The problem is that 
>> pulse-mainloop-glib is not added to the libraries to link.
>>
>> How should this be fixed? Should FindPulseAudio.cmake look for it, or 
>> should there be a (mouthful!) FindPulseAudioMainloopGlib.cmake? (FWIW, 
>> there is a separate .pc for pulse-mainloop-glib.)
> 
> Apparently PA is usable without pulse-mainloop-glib, so I don't think 
> adding that library to the generic Pulse libraries is desirable.
> 
> Currently, I have this patch,

Ahem :-).

> which fixes compilation for me. I'm 
> disinclined to use only "MAINLOOP" versus "MAINLOOP_GLIB", since that 
> does not match the library name (and on the theory that is was named 
> -glib for a reason, i.e. there might some day be a library named 
> mainloop-something_else).
> 
> Please let me know if this looks like a good idea. (An alternative could 
> be to fold FindPulseAudioMainloopGlib into FindPulseAudio, with the same 
> variable names, and keeping them separate, i.e. PULSE_FOUND could be 
> true but PULSE_MAINLOOP_GLIB_FOUND could be false.)

-- 
Matthew
Please do not quote my e-mail address unobfuscated in message bodies.
-- 
"Ah, yes. Control the media and you control the world."
   -- from a story by Raven Blackmane
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: pamainloop.patch
Url: http://mail.kde.org/pipermail/kde-buildsystem/attachments/20090114/cf5a0182/attachment.bat