Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 110667 - gentoo specific patch in faad2 breaks build of gstreamer's faad plugin (attempting to build gstreamer 0.9 cvs)
Summary: gentoo specific patch in faad2 breaks build of gstreamer's faad plugin (attem...
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-10-27 23:19 UTC by Raphael Jacquot
Modified: 2005-12-19 01:56 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Raphael Jacquot 2005-10-27 23:19:10 UTC
faad contains originally the following prototypes

long FAADAPI faacDecInit(faacDecHandle hDecoder,
                        unsigned char *buffer,
                        unsigned long buffer_size,
                        unsigned long *samplerate,
                        unsigned char *channels);

/* Init the library using a DecoderSpecificInfo */
char FAADAPI faacDecInit2(faacDecHandle hDecoder, unsigned char *pBuffer,
                         unsigned long SizeOfDecoderSpecificInfo,
                         unsigned long *samplerate, unsigned char *channels);

that a gentoo specific patch changes to 

long FAADAPI faacDecInit(faacDecHandle hDecoder,
                         uint8_t *buffer,
                         uint32_t buffer_size,
                         uint32_t *samplerate,
                         uint8_t *channels);
--
int8_t FAADAPI faacDecInit2(faacDecHandle hDecoder, uint8_t *pBuffer,
                            uint32_t SizeOfDecoderSpecificInfo,
                            uint32_t *samplerate, uint8_t *channels);

/* Init the library for DRM */
int8_t FAADAPI faacDecInitDRM(faacDecHandle hDecoder, uint32_t samplerate,
                              uint8_t channels);

void FAADAPI faacDecPostSeekReset(faacDecHandle hDecoder, long frame);

this seems to prevent the gstreamer faad2 plugin to compile correctly

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Actual Results:  
compilation of the gst-gaad plugin fails

Expected Results:  
plugin doesn't fail...
Comment 1 Raphael Jacquot 2005-10-28 06:47:34 UTC
it seems all this comes from the amd64 patch. well, turns out this patch is
fairly useless...
the only areas where unsigned long are used are things legths, in which case,
the patch wouldn't change anything
Comment 2 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-12-19 01:53:51 UTC
Taking over faad2 for sound herd.
Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-12-19 01:56:32 UTC
It's not gentoo-specific as other distributions uses the same patch, too, and it's needed to fix 64-bit platforms (amd64, ia64, alpha).

The change to gstreamer code you'd need it's trivial, but I think now you can just use gstreamer from portage and go away with it.
Note that some upstream maintainer is already aware of the needed patch to faad2, such as avidemux's, and provides workaround for modified APIs.