Bug#394153: Fwd: Bug#394153: twinkle: FTBFS: undefined reference to gsm_decode

[Steve Langasek]

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.
vorlon at debian.org                                   http://www.debian.org/