[FFmpeg-devel] Patch: icl linker errors in cmdutils

[Michael Niedermayer]

On Fri, Feb 07, 2014 at 11:56:04AM +1100, Matt Oliver wrote:
> Currently there is a linker error in cmdutils.c as used by ffmpeg and the
> other programs. This error is due to missing dependencies where the
> function print_all_libs_info() uses data from libs that may not have
> actually been built for the current configuration (A common offender here
> is avresample).
> 
> This adds a fix that puts some preprocessor guards so only headers and data
> for libs that are actually built using the current configuration are used.
> 
> This was obviously previously in ffmpeg as the comment on line 28 would
> suggest:
> /* Include only the enabled headers since some compilers (namely, Sun
>    Studio) will not omit unused inline functions and create undefined
>    references to libraries that are not being built. */
> 
> At some point these were removed but this patch readds them in order to fix
> a linker error with standard release settings for icl.

has sun written multiple compilers ?
because the only sun compiler i have seems working fine:
http://fate.ffmpeg.org/report.cgi?time=20140206172840&slot=x86-opensolaris-suncc5.10

also if(0) is used alot, and removing just one as a hack to make just
one (default i assume) build working seems a bit inconsistant

also why does your icl not optimize of if(0) code?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140207/3328c7fa/attachment.asc>