Re: KDE Multimedia related CMake question

2011-11-01 Thread Colin Guthrie
://colin.guthr.ie/git/phonon/commit/?h=4.5id=06a3d4ac02fbc9197d1c1bd4a9cef658713303b1 -- Colin Guthrie colin(at)mageia.org http://colin.guthr.ie/ Day Job: Tribalogic Limited http://www.tribalogic.net/ Open Source: Mageia Contributor http://www.mageia.org/ PulseAudio Hacker http://www.pulseaudio.org

Re: KDE Multimedia related CMake question

2011-11-01 Thread Colin Guthrie
Hiya, 'Twas brillig, and Alexander Neundorf at 19/10/11 20:25 did gyre and gimble: On Wednesday 19 October 2011, Colin Guthrie wrote: I'm not super fussed on Windows at the moment as although we ship Windows builds for PA, none of the backends for windows are PA aware and thus it's a rather

Re: PulseAudio 1.0 and kdemultimedia 4.7.1

2011-10-20 Thread Colin Guthrie
'Twas brillig, and Colin Guthrie at 01/10/11 08:13 did gyre and gimble: 'Twas brillig, and Harald Sitter at 30/09/11 16:31 did gyre and gimble: On Fri, Sep 30, 2011 at 4:02 PM, Colin Guthrie co...@guthr.ie wrote: Or is this bit of code copied around horribly? Yes it is, at least

Re: PulseAudio 1.0 and kdemultimedia 4.7.1

2011-10-03 Thread Colin Guthrie
too in places we can't fix when I chuck out a bugfix, I'll likey put in 1.1.0 (or 2.0.0 or whatever) in these functions. Col -- Colin Guthrie http://colin.guthr.ie/ ___ Kde-buildsystem mailing list Kde-buildsystem@kde.org https://mail.kde.org

Re: PulseAudio 1.0 and kdemultimedia 4.7.1

2011-10-03 Thread Colin Guthrie
'Twas brillig, and Harald Sitter at 30/09/11 16:31 did gyre and gimble: On Fri, Sep 30, 2011 at 4:02 PM, Colin Guthrie co...@guthr.ie wrote: Or is this bit of code copied around horribly? Yes it is, at least there is a version in kdelibs and one in phonon (since phonon is a dep of kdelibs we

Re: PulseAudio 1.0 and kdemultimedia 4.7.1

2011-09-30 Thread Colin Guthrie
. On a related note the version check should prefer the dedicated defines for major, minor, and micro version macros and only fall back to this hideous pa_get_headers_version macro for compatibility. I thought I'd fixed that a while ago? Or is this bit of code copied around horribly? Col -- Colin