Re: [FFmpeg-devel] Build Errors

2017-10-14 Thread Hendrik Leppkes
On Sun, Oct 15, 2017 at 8:17 AM, Yogender Gupta wrote: > 1. I need to comment out the following lines in configure while building in > mingw using msvc 2013. Else configure gets stuck, anything I need to take > care. Started happening recently. > > #for linkunit in $LIBRARY_LIST $PROGRAM_LIST $

[FFmpeg-devel] Build Errors

2017-10-14 Thread Yogender Gupta
1. I need to comment out the following lines in configure while building in mingw using msvc 2013. Else configure gets stuck, anything I need to take care. Started happening recently. #for linkunit in $LIBRARY_LIST $PROGRAM_LIST $EXTRALIBS_LIST; do #flatten_extralibs ${linkunit}_extralibs #

Re: [FFmpeg-devel] Configure: commit 6dfcbd80ad446ff163b47f2bf432bbf706436ea8 broke my builds

2017-10-14 Thread James Almer
On 10/14/2017 8:46 PM, hydra3...@gmail.com wrote: > Hello. Per ticket https://trac.ffmpeg.org/ticket/6731 I too am having > trouble cross-compiling ffmpeg. > > This is the configure being used >    ./configure --arch=x86 --target-os=mingw32 > --cross-prefix=/home/u/Desktop/ffmpeg-windows-build-hel

Re: [FFmpeg-devel] Configure: commit 6dfcbd80ad446ff163b47f2bf432bbf706436ea8 broke my builds

2017-10-14 Thread hydra3333
Hello. Per ticket https://trac.ffmpeg.org/ticket/6731 I too am having trouble cross-compiling ffmpeg. This is the configure being used ./configure --arch=x86 --target-os=mingw32 --cross-prefix=/home/u/Desktop/ffmpeg-windows-build-helpers-withOpenCL-master/sandbox/cross_compilers/mingw-w64-i68

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '7cb1d9e2dbbe5bf4652be5d78cdd68e956fa3d63'

2017-10-14 Thread Reino Wijnsma
On 14-10-2017 1:32, James Almer wrote: > On 10/13/2017 6:30 PM, Reino Wijnsma wrote: >> _Libilbc_ >> On 13-10-2017 20:09, James Almer wrote: >>> With this, the check will include the needed pthreads ldflags when >>> pkg-config is invoked with the --static flag. >> My 'libilbc.pc' doesn't include

Re: [FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

2017-10-14 Thread Michael Niedermayer
On Fri, Oct 13, 2017 at 09:19:04PM +0200, wm4 wrote: > On Fri, 13 Oct 2017 19:41:28 +0200 > Michael Niedermayer wrote: > > > On Fri, Oct 06, 2017 at 01:48:14AM +0200, wm4 wrote: > > > On Fri, 6 Oct 2017 00:01:30 +0200 > > > Michael Niedermayer wrote: > > > > > > > The opaque_ref wraping is a

[FFmpeg-devel] [PATCH] web/download: add Zeranoe macOS builds

2017-10-14 Thread Kyle Schwarz
--- src/download | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/download b/src/download index d2db75a..2dbc338 100644 --- a/src/download +++ b/src/download @@ -100,6 +100,9 @@ http://ffmpegmac.net/";> Builds 32-bit and 64-bit for OS X 10.5 and above

Re: [FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

2017-10-14 Thread Ronald S. Bultje
Hi Umair, On Sat, Oct 14, 2017 at 1:43 PM, Umair Khan wrote: > On Sat, Oct 14, 2017 at 8:02 PM, Ronald S. Bultje > wrote: > > Hi Umair, > > > > On Sat, Oct 14, 2017 at 9:59 AM, Umair Khan wrote: > > > >> I tested the file which Michael sent. The thing is that I'm getting > >> error in decoding

Re: [FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

2017-10-14 Thread Umair Khan
On Sat, Oct 14, 2017 at 8:02 PM, Ronald S. Bultje wrote: > Hi Umair, > > On Sat, Oct 14, 2017 at 9:59 AM, Umair Khan wrote: > >> I tested the file which Michael sent. The thing is that I'm getting >> error in decoding that file in both the cases, with or without the >> patch. I will begin debuggi

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/avio: retry_transfer_wrapper: don't treat 0 as EOF

2017-10-14 Thread Daniel Kučera
> > If one patch introduces a problem that gets fixed by a subsequent patch > then yes, both should be squashed into one to make sure the problem is > never introduced. > > The idea is that no patch/commit on its own should generate a fate > failure of any kind, as it would make bisecting unrelated

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/avio: retry_transfer_wrapper: don't treat 0 as EOF

2017-10-14 Thread James Almer
On 10/14/2017 2:22 PM, Daniel Kučera wrote: > 2017-10-14 19:16 GMT+02:00 Daniel Kučera : >> 2017-10-14 17:46 GMT+02:00 Michael Niedermayer : >>> On Sat, Oct 14, 2017 at 07:10:37AM +0200, Daniel Kučera wrote: breaks fate-indeo3-2 [...] -- You need to apply the whol

[FFmpeg-devel] [PATCH] libavformat/cache: don't treat 0 as EOF libavformat/aviobuf: don't treat 0 as EOF libavformat/avio: retry_transfer_wrapper: don't treat 0 as EOF

2017-10-14 Thread Daniel Kucera
transfer_func variable passed to retry_transfer_wrapper are h->prot->url_read and h->prot->url_write functions. These need to return EOF or other error properly. In case of returning >= 0, url_read/url_write is retried until error is returned. Signed-off-by: Daniel Kucera --- libavformat/avio.c

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/avio: retry_transfer_wrapper: don't treat 0 as EOF

2017-10-14 Thread Daniel Kučera
2017-10-14 19:16 GMT+02:00 Daniel Kučera : > 2017-10-14 17:46 GMT+02:00 Michael Niedermayer : >> On Sat, Oct 14, 2017 at 07:10:37AM +0200, Daniel Kučera wrote: >>> breaks fate-indeo3-2 >>> >>> [...] >>> -- >>> >>> >>> You need to apply the whole set. It passed all fate tests on my machine. >> >> I

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/avio: retry_transfer_wrapper: don't treat 0 as EOF

2017-10-14 Thread Daniel Kučera
2017-10-14 17:46 GMT+02:00 Michael Niedermayer : > On Sat, Oct 14, 2017 at 07:10:37AM +0200, Daniel Kučera wrote: >> breaks fate-indeo3-2 >> >> [...] >> -- >> >> >> You need to apply the whole set. It passed all fate tests on my machine. > > I applied the whole set and bisected to find which patch

[FFmpeg-devel] [PATCH] build: add global extralibs to pkg-config file generation

2017-10-14 Thread James Almer
Dependencies like libm and pthreads are now global again, so handle that while generating our pkg-config files. Signed-off-by: James Almer --- The pkg-config file generation nonetheless needs some more thought. Much like how we're dealing with external libs not properly listing all their dependen

[FFmpeg-devel] [PATCH] configure: add pthreads ldflags globally

2017-10-14 Thread James Almer
It's used by libavutil and by extension every other library, and by making it global we simplify a lot of checks. Signed-off-by: James Almer --- configure | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/configure b/configure index a0bfb269e8..09f8ed305b 100755 ---

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/dvbsubdec: Split best score computation out of loop in compute_default_clut()

2017-10-14 Thread Michael Niedermayer
On Sun, Oct 01, 2017 at 06:26:53PM +0200, Clément Bœsch wrote: > On Sun, Oct 01, 2017 at 06:10:36PM +0200, Michael Niedermayer wrote: > > 3% faster > > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/dvbsubdec.c | 9 ++--- > > 1 file changed, 6 insertions(+), 3 deletions(-) > >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/dvbsubdec: Factor a few expressions out of compute_default_clut()

2017-10-14 Thread Michael Niedermayer
On Sun, Oct 01, 2017 at 06:25:14PM +0200, Clément Bœsch wrote: > On Sun, Oct 01, 2017 at 06:10:35PM +0200, Michael Niedermayer wrote: > > 32% faster loop > > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/dvbsubdec.c | 16 +--- > > 1 file changed, 9 insertions(+), 7 de

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/pafvideo: Check for bitstream end in decode_0()

2017-10-14 Thread Michael Niedermayer
On Fri, Oct 13, 2017 at 03:06:53AM +0200, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: 3529/clusterfuzz-testcase-5057068371279872 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer > --- > libavc

[FFmpeg-devel] [PATCH] configure: add libm ldflags globally

2017-10-14 Thread James Almer
It's used by every library, and by making it global we simplify a lot of checks. Signed-off-by: James Almer --- configure | 18 +++--- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/configure b/configure index 22026ef832..a0bfb269e8 100755 --- a/configure +++ b/config

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/avio: retry_transfer_wrapper: don't treat 0 as EOF

2017-10-14 Thread Michael Niedermayer
On Sat, Oct 14, 2017 at 07:10:37AM +0200, Daniel Kučera wrote: > breaks fate-indeo3-2 > > [...] > -- > > > You need to apply the whole set. It passed all fate tests on my machine. I applied the whole set and bisected to find which patch caused the failure also the i-th patch of a set must pass

Re: [FFmpeg-devel] Configure: commit 6dfcbd80ad446ff163b47f2bf432bbf706436ea8 broke my builds

2017-10-14 Thread James Almer
On 10/14/2017 7:48 AM, Mark Himsley wrote: > Update: for me with my specific set of libraries, I have had to apply > this patch to configure in order to make configure accept my > configuration parameters: specifically changes to mp3lame, openjpeg > 1.5, x265 and xvid. > > diff --git a/configure b

Re: [FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

2017-10-14 Thread Ronald S. Bultje
Hi Umair, On Sat, Oct 14, 2017 at 9:59 AM, Umair Khan wrote: > I tested the file which Michael sent. The thing is that I'm getting > error in decoding that file in both the cases, with or without the > patch. I will begin debugging this issue, however I think the file > which Michael sent has go

Re: [FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

2017-10-14 Thread Umair Khan
On Sat, Oct 14, 2017 at 7:29 PM, Umair Khan wrote: > Hi, > > On Sat, Sep 23, 2017 at 8:45 PM, Michael Niedermayer > wrote: >> >> On Fri, Sep 22, 2017 at 01:54:27PM +0530, Umair Khan wrote: >> > On Thu, Sep 21, 2017 at 6:30 PM, Paul B Mahol wrote: >> > > On 9/21/17, Umair Khan wrote: >> > >> Hi

Re: [FFmpeg-devel] latest ffmpeg (git) and libvpx_vp9

2017-10-14 Thread ZikZak
Ah sorry, as I used the git version I thought it would be better here. My bad. On 14 October 2017 at 14:50, Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 14, 2017 at 8:28 AM, ZikZak wrote: > > > I freshly recompiled libvpx (for both VP8 and VP9) then FFMPEG and > > strangely the result only all

Re: [FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

2017-10-14 Thread Umair Khan
Hi, On Sat, Sep 23, 2017 at 8:45 PM, Michael Niedermayer wrote: > > On Fri, Sep 22, 2017 at 01:54:27PM +0530, Umair Khan wrote: > > On Thu, Sep 21, 2017 at 6:30 PM, Paul B Mahol wrote: > > > On 9/21/17, Umair Khan wrote: > > >> Hi Ronald, > > >> > > >> On Mon, Sep 11, 2017 at 10:27 PM, Ronald S

Re: [FFmpeg-devel] latest ffmpeg (git) and libvpx_vp9

2017-10-14 Thread Ronald S. Bultje
Hi, On Sat, Oct 14, 2017 at 8:28 AM, ZikZak wrote: > I freshly recompiled libvpx (for both VP8 and VP9) then FFMPEG and > strangely the result only allows me libvpx_vp8 as encoder despite a > configure with enable-encoder=libvpx_vp8 and enable-encoder=libvpx_vp9 > Please send these types of que

[FFmpeg-devel] latest ffmpeg (git) and libvpx_vp9

2017-10-14 Thread ZikZak
Hello, I freshly recompiled libvpx (for both VP8 and VP9) then FFMPEG and strangely the result only allows me libvpx_vp8 as encoder despite a configure with enable-encoder=libvpx_vp8 and enable-encoder=libvpx_vp9 vpxenc's help lists VP9 as a possible encoder Configuration for libvpx: ./configure

Re: [FFmpeg-devel] Configure: commit 6dfcbd80ad446ff163b47f2bf432bbf706436ea8 broke my builds

2017-10-14 Thread Mark Himsley
Update: for me with my specific set of libraries, I have had to apply this patch to configure in order to make configure accept my configuration parameters: specifically changes to mp3lame, openjpeg 1.5, x265 and xvid. diff --git a/configure b/configure index dfbc251..67c012d 100755 --- a/configur

[FFmpeg-devel] Configure: commit 6dfcbd80ad446ff163b47f2bf432bbf706436ea8 broke my builds

2017-10-14 Thread Mark Himsley
I build FFmpeg and all the libraries I need every night. Since 08:00 BST on 12-Oct-2017 that build has not worked, the previous build on 11-Oct-2017 worked. My configure command for building 64bit Linux that is run on an Ubuntu 16.04 system, which worked on Wednesday but not on Thursday, is: ./co

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '7cb1d9e2dbbe5bf4652be5d78cdd68e956fa3d63'

2017-10-14 Thread Jörn Heusipp
Hello On 10/14/2017 01:32 AM, James Almer wrote: On 10/13/2017 6:30 PM, Reino Wijnsma wrote: On 13-10-2017 18:48, James Almer wrote: Are "-lshlwapi -lpthread -lcrypt32" extralibs you added manually, or were they derived from dependencies like libmpg123, vorbis, etc by pkg-config? My configu