On Wed, Nov 15, 2006 at 11:11:31AM +0000, Mark Purcell wrote:
> On Wednesday 15 November 2006 03:35, Julien Cristau wrote:
> > clone 394153 -1
> > reassign -1 libgsm
> > retitle -1 please add extern "C" declaration for use in c++ programs
> Julien, Kurt,
> libgsm1 does have an extern "C" declaration:
> /usr/include/gsm.h
> [...]
> #ifdef __cplusplus
> extern "C" {
> #endif
> I'm not sure why this is occurring...
Sure enough, this isn't a bug in libgsm.
#ifdef HAVE_GSM
#include <gsm/gsm.h>
#else
#include "gsm/inc/gsm.h"
#endif
this is the code in audio_encoder.h, and the HAVE_GSM check is failing --
which means the build is using a local copy of gsm in the src/audio/gsm
directory (bad) and the build is failing as a result (good :).
I don't know yet why this define isn't propagating; it seems to be part of
debian/patches/libgsm.patch though, and there's probably a race condition in
the autotools handling that's preventing it from getting where it needs to
be. In a rebuild test on amd64, config.log looks right to me, yet it's not
*consistently* ending up in the generated src/twinkle_config.h, because
there's a race condition with whether src/twinkle_config.h gets regenerated
after autoheader is run.
--
Steve Langasek Give me a lever long enough and a Free OS
Debian Developer to set it on, and I can move the world.
[EMAIL PROTECTED] http://www.debian.org/
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
