[FFmpeg-devel] [RFC] Installing libavutil's crc.h?
Wed Dec 12 01:48:55 CET 2007
Diego Biurrun wrote:
> On Tue, Dec 11, 2007 at 11:23:26PM +0100, Michael Niedermayer wrote:
> > On Tue, Dec 11, 2007 at 07:35:20PM +0100, Diego Biurrun wrote:
> > > On Tue, Dec 11, 2007 at 04:30:25PM +0100, Diego 'Flameeyes'
> > > Petten? wrote:
> > > >
> > > > So okay for bswap.h there's the configh dependency that
> > > > disallows from installing it (but I'm still thinking how to get
> > > > over the dep, maybe I'll provide a patch to use unifdef).
> > > >
> > > > For crc.h is there something else stopping it from being
> > > > installed?
> > >
> > > This sounds backwards to me. Headers should not be installed
> > > without a good reason.
> > is it not enogh that its needed to use the crc code in libavuitl?
> > its a mere oversight that it isnt installed
> It is a good enough reason.
> I'm thinking that we could solve the LIBAVUTIL_VERSION_INT issue by
> switching the condition around, i.e.:
> #if LIBAVUTIL_VERSION_INT < (50<<16)
> extern AVCRC *av_crcEDB88320;
> extern AVCRC *av_crc04C11DB7;
> extern AVCRC *av_crc8005 ;
> extern AVCRC *av_crc07 ;
> extern AVCRC av_crcEDB88320;
> extern AVCRC av_crc04C11DB7;
> extern AVCRC av_crc8005 ;
> extern AVCRC av_crc07 ;
If crc.h wasn't part of public API up to now, I wonder why do we try to
keep stable API/ABI ?
Is it to ensure compatibility between lavu and lavf compiled from
different svn revision ? (I thought this wasn't supported ?)
So a simple solution to solve the LIBAVUTIL_VERSION_INT issue would
be to just drop the legacy version of those declaration.
And if you don't want want to break API/ABI, another simple
solution would be to increment LIBAVUTIL_VERSION to 50.0.0 :-)
More information about the ffmpeg-devel