Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-26 Thread Reynaldo H. Verdejo Pinochet
Hi On 11/26/2014 07:35 PM, Lukasz Marek wrote: > [..] > maybe I should avoid commenting patches in other threads, but I removed Yeah ;) > this for now. I will repost it when it is finished. > [..] All good then. Please CC me when you do to make sure I don't miss it. Thanks a lot! Bests, --

Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-26 Thread Lukasz Marek
On 26.11.2014 23:27, Reynaldo H. Verdejo Pinochet wrote: Hi Lukasz. Are you sending an updated patch for this one or I missed it somehow? maybe I should avoid commenting patches in other threads, but I removed this for now. I will repost it when it is finished. I submitted it just to present t

Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-26 Thread Reynaldo H. Verdejo Pinochet
Hi Lukasz. Are you sending an updated patch for this one or I missed it somehow? -- Reynaldo H. Verdejo Pinochet Open Source Group Samsung Research America / Silicon Valley ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/

Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-20 Thread Lukasz Marek
On 21.11.2014 00:14, Alexander Strasser wrote: Hi Lukasz! On 2014-11-19 00:06 +0100, Lukasz Marek wrote: On 18.11.2014 22:20, Reynaldo H. Verdejo Pinochet wrote: Hi. Overall this looks good, thanks. I would move the test to tests/ though and integrate it with the build/test system as, say, the

Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-20 Thread Alexander Strasser
Hi Lukasz! On 2014-11-19 00:06 +0100, Lukasz Marek wrote: > On 18.11.2014 22:20, Reynaldo H. Verdejo Pinochet wrote: > >Hi. Overall this looks good, thanks. I would move the test to > >tests/ though and integrate it with the build/test system as, > >say, the way tiny_ssim is: > > > >grep -R tiny_s

Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-18 Thread Lukasz Marek
On 18.11.2014 22:20, Reynaldo H. Verdejo Pinochet wrote: Hi. Overall this looks good, thanks. I would move the test to tests/ though and integrate it with the build/test system as, say, the way tiny_ssim is: grep -R tiny_ssim * Makefile:TESTTOOLS = audiogen videogen rotozoom tiny_psnr tiny_ssi

Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-18 Thread Reynaldo H. Verdejo Pinochet
On 11/16/2014 11:00 PM, Lukasz Marek wrote: > [..] > Second thing, I don't know how to integrate this test into build system. > I put it into tool, but it requires ffserver_config.o and cmdutils.o > For now it have to be build with make && make tools/ffserver_config_test > as these two *.o are no

Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-18 Thread Reynaldo H. Verdejo Pinochet
Hi. Overall this looks good, thanks. I would move the test to tests/ though and integrate it with the build/test system as, say, the way tiny_ssim is: grep -R tiny_ssim * Makefile:TESTTOOLS = audiogen videogen rotozoom tiny_psnr tiny_ssim base64 tests/Makefile:FATE_UTILS = base64 tiny_psnr tiny_

Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-16 Thread Lukasz Marek
On 17.11.2014 02:46, Lukasz Marek wrote: Work in progress... Need support Signed-off-by: Lukasz Marek --- Makefile | 4 +- ffserver_config.c| 40 ++ ffserver_config.h| 7 ++ tools/ffserver_config_test.c | 178

[FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

2014-11-16 Thread Lukasz Marek
Work in progress... Need support Signed-off-by: Lukasz Marek --- Makefile | 4 +- ffserver_config.c| 40 ++ ffserver_config.h| 7 ++ tools/ffserver_config_test.c | 178 +++ 4 files changed, 228 inser