[FFmpeg-devel] [PATCH 1/4] configure: add fftw3 detection

Reimar Döffinger Reimar.Doeffinger at gmx.de
Fri Mar 25 10:32:58 CET 2016


On Fri, Mar 25, 2016 at 10:05:09AM +0100, Paul B Mahol wrote:
> On 3/25/16, Ganesh Ajjanagadde <gajjanag at gmail.com> wrote:
> >                                 { require libfdk_aac fdk-aac/aacenc_lib.h
> > aacEncOpen -lfdk-aac &&
> >                                   warn "using libfdk without pkg-config"; }
> > }
> > +enabled libfftw3          && require2 libfftw3f "fftw3.h" fftwf_execute
> > -lfftw3f -lm
> >  flite_libs="-lflite_cmu_time_awb -lflite_cmu_us_awb -lflite_cmu_us_kal
> > -lflite_cmu_us_kal16 -lflite_cmu_us_rms -lflite_cmu_us_slt -lflite_usenglish
> > -lflite_cmulex -lflite"
> >  enabled libflite          && require2 libflite "flite/flite.h" flite_init
> > $flite_libs
> >  enabled fontconfig        && enable libfontconfig
> > --
> > 2.7.4
> 
> Big NO from me. Please refrain from doing such silly things.
> Either improve our implementation or do nothing.

I don't disagree with the basic objections, but
I'd appreciate more diplomacy and kindness (and
space for people to explain there motivations)
in the responses.
I am in the habit of liking to call things silly
myself, but honestly almost all patches here
exist for a good reason, even if including
them is not a good idea, and I don't want to
discourage people from sending in interesting
but "silly" things just because they might be derided.
Sorry for the long off-topic.

Thanks,
Reimar


More information about the ffmpeg-devel mailing list