[FFmpeg-cvslog] r26200 - trunk/configure

[Stefano Sabatini]

On date Monday 2011-01-03 11:25:25 +0000, Carl Eugen Hoyos wrote:
> Stefano Sabatini <stefano.sabatini-lala <at> poste.it> writes:
> 
> > > -ffprobe_deps="avcodec avformat"
> > > -ffserver_deps="avformat ffm_muxer rtp_protocol rtsp_demuxer"
> > > +ffprobe_deps="avcodec avformat swscale"
> > > +ffserver_deps="avformat ffm_muxer rtp_protocol rtsp_demuxer swscale"
> > >  ffserver_extralibs='$ldl'
> > 
> > This change is wrong, there is no point into making ffprobe and
> > ffserver depend on libswscale,
> 
> I don't disagree that there is no point, but currently it is the case.
> 
> > and the libavfilter dependency on
> > libswscale is already addressed in libavfilter/Makefile.
> 
> This is/was wrong.
> 
> > So I think this should just be reverted.
> 
> You mean it is preferable to let make fail deliberately than to have configure
> reflect the actual code necessities (no matter if they are useful or not)?

No I mean that we should fix the unnecessary dependency in cmdutils.c
as in the proposed patch, and *then* remove the configure dependency
(rather than simply add an unnecessary dependency in configure).

> Imo, it is easy to remove the dependencies from configure once the code has been
> fixed.