Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-24 Thread Diego Biurrun
On Tue, Feb 23, 2016 at 03:26:18PM +0100, Luca Barbato wrote: > On 23/02/16 08:14, Luca Barbato wrote: > > Not ok, the problem is quite interesting, the linker might not be > > completely at fault here. > > I was late for a meeting so I didn't get you more context, some > explanations: > > f_j_re

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-23 Thread Luca Barbato
On 23/02/16 08:14, Luca Barbato wrote: > Not ok, the problem is quite interesting, the linker might not be > completely at fault here. > I was late for a meeting so I didn't get you more context, some explanations: f_j_rev_dct is used by the test the function lives in a file that also implement

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-22 Thread Luca Barbato
On 22/02/16 18:20, Vittorio Giovara wrote: > On Sun, Feb 21, 2016 at 5:51 AM, Luca Barbato wrote: >> On 21/02/16 02:49, Vittorio Giovara wrote: >>> On Sat, Feb 20, 2016 at 8:22 PM, Luca Barbato wrote: On 20/02/16 23:40, Diego Biurrun wrote: > Are you on Git HEAD and did you distclean? T

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-22 Thread Vittorio Giovara
On Sun, Feb 21, 2016 at 5:51 AM, Luca Barbato wrote: > On 21/02/16 02:49, Vittorio Giovara wrote: >> On Sat, Feb 20, 2016 at 8:22 PM, Luca Barbato wrote: >>> On 20/02/16 23:40, Diego Biurrun wrote: Are you on Git HEAD and did you distclean? The wmv2 decoder depends on >>> >>> ar -t libavcod

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-21 Thread Luca Barbato
On 21/02/16 02:49, Vittorio Giovara wrote: > On Sat, Feb 20, 2016 at 8:22 PM, Luca Barbato wrote: >> On 20/02/16 23:40, Diego Biurrun wrote: >>> Are you on Git HEAD and did you distclean? The wmv2 decoder depends on >> >> ar -t libavcodec/libavcodec.a | grep idctdsp.o >> idctdsp.o >> >> Looks st

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
On Sat, Feb 20, 2016 at 4:12 PM, Derek Buitenhuis wrote: > On 2/20/2016 7:36 PM, Vittorio Giovara wrote: >> I am able to reproduce with that exact same line, on OSX, with clang and gcc > > A real (ports) GCC or the llvm-gcc that apple ships? > > Could be the same DCE bug if so? > Probably, I trie

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
On Sat, Feb 20, 2016 at 8:22 PM, Luca Barbato wrote: > On 20/02/16 23:40, Diego Biurrun wrote: >> Are you on Git HEAD and did you distclean? The wmv2 decoder depends on > > ar -t libavcodec/libavcodec.a | grep idctdsp.o > idctdsp.o > > Looks strange, the object is there... > Lu, are you able to

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Luca Barbato
On 20/02/16 23:40, Diego Biurrun wrote: > Are you on Git HEAD and did you distclean? The wmv2 decoder depends on ar -t libavcodec/libavcodec.a | grep idctdsp.o idctdsp.o Looks strange, the object is there... lu ___ libav-devel mailing list libav-deve

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Luca Barbato
On 20/02/16 22:12, Derek Buitenhuis wrote: > On 2/20/2016 7:36 PM, Vittorio Giovara wrote: >> I am able to reproduce with that exact same line, on OSX, with clang and gcc > > A real (ports) GCC or the llvm-gcc that apple ships? real gcc. > Could be the same DCE bug if so? seems more a linking i

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Diego Biurrun
On Sat, Feb 20, 2016 at 02:36:08PM -0500, Vittorio Giovara wrote: > On Sat, Feb 20, 2016 at 12:57 PM, Diego Biurrun wrote: > > On Sat, Feb 20, 2016 at 11:22:18AM -0500, Vittorio Giovara wrote: > >> On Sat, Feb 20, 2016 at 6:46 AM, Diego Biurrun wrote: > >> > On Sat, Feb 20, 2016 at 04:22:37AM -05

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Derek Buitenhuis
On 2/20/2016 7:36 PM, Vittorio Giovara wrote: > I am able to reproduce with that exact same line, on OSX, with clang and gcc A real (ports) GCC or the llvm-gcc that apple ships? Could be the same DCE bug if so? - Derek ___ libav-devel mailing list liba

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
On Sat, Feb 20, 2016 at 12:57 PM, Diego Biurrun wrote: > On Sat, Feb 20, 2016 at 11:22:18AM -0500, Vittorio Giovara wrote: >> On Sat, Feb 20, 2016 at 6:46 AM, Diego Biurrun wrote: >> > On Sat, Feb 20, 2016 at 04:22:37AM -0500, Vittorio Giovara wrote: >> >> The test program fails to build with `co

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Hendrik Leppkes
On Sat, Feb 20, 2016 at 5:22 PM, Vittorio Giovara wrote: > On Sat, Feb 20, 2016 at 6:46 AM, Diego Biurrun wrote: >> On Sat, Feb 20, 2016 at 04:22:37AM -0500, Vittorio Giovara wrote: >>> The test program fails to build with `configure --disable-decoders`. >> >> I cannot reproduce. > > my full conf

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Diego Biurrun
On Sat, Feb 20, 2016 at 11:22:18AM -0500, Vittorio Giovara wrote: > On Sat, Feb 20, 2016 at 6:46 AM, Diego Biurrun wrote: > > On Sat, Feb 20, 2016 at 04:22:37AM -0500, Vittorio Giovara wrote: > >> The test program fails to build with `configure --disable-decoders`. > > > > I cannot reproduce. > >

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
On Sat, Feb 20, 2016 at 6:46 AM, Diego Biurrun wrote: > On Sat, Feb 20, 2016 at 04:22:37AM -0500, Vittorio Giovara wrote: >> The test program fails to build with `configure --disable-decoders`. > > I cannot reproduce. my full config line is ./configure --enable-debug --cc=gcc-5 --enable-nonfree

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Diego Biurrun
On Sat, Feb 20, 2016 at 04:22:37AM -0500, Vittorio Giovara wrote: > The test program fails to build with `configure --disable-decoders`. I cannot reproduce. > --- a/libavcodec/Makefile > +++ b/libavcodec/Makefile > @@ -797,7 +797,7 @@ HOSTPROGS = aac_tablegen >

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Luca Barbato
On 20/02/16 10:22, Vittorio Giovara wrote: > The test program fails to build with `configure --disable-decoders`. > --- > libavcodec/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/Makefile b/libavcodec/Makefile > index 256dee3..3c57539 100644 > --- a/

[libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
The test program fails to build with `configure --disable-decoders`. --- libavcodec/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 256dee3..3c57539 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -797,7 +79