Re: [libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-21 Thread Diego Biurrun
On Thu, Jan 21, 2016 at 11:41:19AM -0500, Vittorio Giovara wrote: > On Wed, Jan 20, 2016 at 3:44 AM, Diego Biurrun wrote: > > On Tue, Jan 19, 2016 at 02:27:13PM -0500, Vittorio Giovara wrote: > >> On Tue, Jan 19, 2016 at 1:36 PM, Diego Biurrun wrote: > >> > Leave these out or create proper fine-g

Re: [libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-21 Thread Vittorio Giovara
On Wed, Jan 20, 2016 at 3:44 AM, Diego Biurrun wrote: > On Tue, Jan 19, 2016 at 02:27:13PM -0500, Vittorio Giovara wrote: >> On Tue, Jan 19, 2016 at 1:36 PM, Diego Biurrun wrote: >> > Leave these out or create proper fine-grained dependencies. Bloating >> > the build for the other codecs is not

Re: [libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-20 Thread Diego Biurrun
On Tue, Jan 19, 2016 at 02:27:13PM -0500, Vittorio Giovara wrote: > On Tue, Jan 19, 2016 at 1:36 PM, Diego Biurrun wrote: > > Leave these out or create proper fine-grained dependencies. Bloating > > the build for the other codecs is not acceptable. I have a very similar > > patch in a branch I n

Re: [libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-19 Thread Vittorio Giovara
On Tue, Jan 19, 2016 at 1:36 PM, Diego Biurrun wrote: > Leave these out or create proper fine-grained dependencies. Bloating > the build for the other codecs is not acceptable. I have a very similar > patch in a branch I need to revisit some day, waiting for me to address > the same issues. ok

Re: [libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-19 Thread Diego Biurrun
build: Factor out CELP module It's not just configure. On Tue, Jan 19, 2016 at 01:12:11PM -0500, Vittorio Giovara wrote: > --- a/configure > +++ b/configure > @@ -1646,6 +1646,7 @@ CONFIG_EXTRA=" > blockdsp > bswapdsp > cabac > +celp_math > dirac_parse > dvprofile >

[libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-19 Thread Vittorio Giovara
--- This might compile a couple of unnecessary files, but they are relatively small and it will simplify more entries in this way. Vittorio configure | 13 - libavcodec/Makefile | 32 ++-- 2 files changed, 18 insertions(+), 27 deletions(-) diff --