[AD] [ alleg-Bugs-3153783 ] Check of PulseAudio version in cmake

["SourceForge.net" <noreply@xxxxxxxxxx>]

Bugs item #3153783, was opened at 2011-01-10 07:07
Message generated for change (Settings changed) made by tjaden
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105665&aid=3153783&group_id=5665

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Building
Group: 5.1
>Status: Closed
Resolution: Fixed
Priority: 5
Private: No
Submitted By: Jehan (hysseo)
Assigned to: Nobody/Anonymous (nobody)
Summary: Check of PulseAudio version in cmake

Initial Comment:
It seems that for building Allegro 5.0.0rc4 with PulseAudio support, version 0.9.15 is required.
The main problem is that right now the "cmake" step will succeed (apparently it only checks for the presence of libpulse-simple, not its version), but it will fail during compilation with the error:

[ 25%] Building C object addons/audio/CMakeFiles/allegro_audio.dir/pulseaudio.c.o
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c: In function 'sink_info_cb':
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c:49: error: 'pa_sink_state_t' undeclared (first use in this function)
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c:49: error: (Each undeclared identifier is reported only once
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c:49: error: for each function it appears in.)
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c:49: error: 'ret' undeclared (first use in this function)
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c:51: error: 'pa_sink_info' has no member named 'state'
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c: In function 'pulseaudio_open':
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c:93: error: 'pa_sink_state_t' undeclared (first use in this function)
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c:93: error: expected ';' before 'state'
/home/jehan/.local/src/allegro-5.0.0rc4/addons/audio/pulseaudio.c:95: error: 'state' undeclared (first use in this function)
make[2]: *** [addons/audio/CMakeFiles/allegro_audio.dir/pulseaudio.c.o] Erreur 1
make[1]: *** [addons/audio/CMakeFiles/allegro_audio.dir/all] Erreur 2
make: *** [all] Erreur 2

Setting WANT_PULSEAUDIO to off make the compilation actually succeed but it would be better to detect this during the configuration step.

This issue was already in Allegro 4.9 (this is thanks to this forum that I saw that PulseAudio 0.9.15 would fix the problem, I did not try to update it: http://www.allegro.cc/forums/thread/603153/851477#target ) and not fixed in the 5.0 RC apparently.

Thanks for this library!

----------------------------------------------------------------------

Comment By: Peter Wang (tjaden)
Date: 2011-01-11 00:42

Message:
Should be fixed, thanks.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105665&aid=3153783&group_id=5665