Re: [gst-devel] flac plugin and the new flac 1.0.4beta

[Thomas V. S. <th...@ur...>]

I've been thinking about the FLAC situation.

They tend to do a lot of incompatible releases.  This is pretty annoying 
for both us and people trying to compile gstreamer with it.

Here's my current stance on it :

* FLAC released a 1.0, and my guess is a lot of people will stick with 
that for some time.  Our plug-in should be able to build cleanly against 
1.0

* FLAC also keeps releasing new versions.  Our plug-in should also be able 
to build cleanly against the latest *officially released* version.
This, IMO, doesn't include beta's.

Now, on how to do this - I think the code for the plug-in should be 
ifdef'd for htese two cases.  We had a version of the plug-in that worked 
with flac 1.0, so this should be easy to do.

Thoughts ?

Thomas

 > Hi,
> 
> there's a new incompatible version of libFLAC out. The current CVS
> version of the gstreamer plugin doesn't compile with it:
> /bin/sh ../../libtool --mode=compile i586-mandrake-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I. -I../..    -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -I/ussr/include/gstreamer-0.4.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/libxml2   -g -I../../gst-libs -Wall -O3 -fomit-frame-pointer -pipe -mcpu=pentiumpro -march=i586 -ffast-math -fno-strength-reduce -c -o libgstflac_la-gstflacenc.lo `test -f gstflacenc.c || echo './'`gstflacenc.c
> i586-mandrake-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I. -I../.. -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -I/usr/include/gstreamer-0.4.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/libxml2 -g -I../../gst-libs -Wall -O3 -fomit-frame-pointer -pipe -mcpu=pentiumpro -march=i586 -ffast-math -fno-strength-reduce -c gstflacenc.c  -fPIC -DPIC -o libgstflac_la-gstflacenc.lo
> gstflacenc.c: In function `gst_flacenc_write_callback':
> gstflacenc.c:169: `FLAC__STREAM_ENCODER_WRITE_OK' undeclared (first use in this function)
> gstflacenc.c:169: (Each undeclared identifier is reported only once
> gstflacenc.c:169: for each function it appears in.)
> make: *** [libgstflac_la-gstflacenc.lo] Error 1
> 
> Is anybody working on a port to the new API?
> 

-- 

The Dave/Dina Project : future TV today ! - http://davedina.apestaart.org/
<-*-                      -*->
If lovin' you is wrong
then I don't wanna be right
<-*- th...@ap... -*->
URGent, the best radio on the Internet - 24/7 ! - http://urgent.rug.ac.be/