[FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

Signed-off-by: Michael Niedermayer --- tests/Makefile |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/Makefile b/tests/Makefile index 07557f9..2f095df 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -24,13 +24,16 @@ $(VREF): tests/videogen$(HOSTEXESUF) | tests

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Tue, 26 Aug 2014 13:41:42 +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > tests/Makefile |5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/tests/Makefile b/tests/Makefile > index 07557f9..2f095df 100644 > --- a/tests/Makefile > +++

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 26/08/14 8:41 AM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > tests/Makefile |5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/tests/Makefile b/tests/Makefile > index 07557f9..2f095df 100644 > --- a/tests/Makefile > +++ b/tests/Makefil

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Wed, Aug 27, 2014 at 05:21:18PM -0800, Lou Logan wrote: > On Tue, 26 Aug 2014 13:41:42 +0200, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > tests/Makefile |5 - > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/tests/Makefile b/

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 8/28/2014 2:25 AM, James Almer wrote: > wget is not available on a standard Msys installation. Probably the same with > Cygwin. bc isn't standard either and FATE uses it. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.or

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

Andreas Cadhalpun googlemail.com> writes: > But I'm fine with moving lena.pnm to the FATE samples, > which are downloaded with 'make fate-rsync'. > Attached patch removes tests/lena.pnm and adapts the > tests, assuming lena.pnm would be added to the top-level > of the SAMPLES directory. As sa

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 28.08.2014 14:01, Carl Eugen Hoyos wrote: Andreas Cadhalpun googlemail.com> writes: But I'm fine with moving lena.pnm to the FATE samples, which are downloaded with 'make fate-rsync'. Attached patch removes tests/lena.pnm and adapts the tests, assuming lena.pnm would be added to the top-lev

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 8/28/2014 12:28 PM, Andreas Cadhalpun wrote: > ffmpeg -f data -i http://samples.ffmpeg.org/image-samples/lena.pnm -c > copy -f data -map 0 -y lena.pnm >>> >>> From: >>> >> >> possible >> >> but would this make and

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 28.08.2014 14:29, Derek Buitenhuis wrote: On 8/28/2014 12:28 PM, Andreas Cadhalpun wrote: ffmpeg -f data -i http://samples.ffmpeg.org/image-samples/lena.pnm -c copy -f data -map 0 -y lena.pnm From: possible but wo

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 8/28/2014 1:42 PM, Andreas Cadhalpun wrote: > That's clear. But it doesn't work without internet connection. > So why not use fate-rsync, which is only run manually, when internet is > available? Probably because it entails reworking the vsynth build system stuff quite a bit to placate a singl

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

Andreas Cadhalpun googlemail.com> writes: > So why not use fate-rsync Because it downloads a few files more than Lena. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

Andreas Cadhalpun googlemail.com> writes: > But I'm not sure what copying it to tests/data would be good for? So people can still run the limited tests without downloading the whole fate suite. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@f

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 28.08.2014 15:16, Carl Eugen Hoyos wrote: Andreas Cadhalpun googlemail.com> writes: But I'm not sure what copying it to tests/data would be good for? So people can still run the limited tests without downloading the whole fate suite. With my patch that would still be possible, the limit

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Thu, Aug 28, 2014 at 02:12:14PM +0200, Andreas Cadhalpun wrote: > On 28.08.2014 14:01, Carl Eugen Hoyos wrote: > >Andreas Cadhalpun googlemail.com> writes: > > > >>But I'm fine with moving lena.pnm to the FATE samples, > >>which are downloaded with 'make fate-rsync'. > >>Attached patch removes

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 28.08.2014 17:53, Clément Bœsch wrote: ffplay http://lucy.pkh.me/lena.pnm File size and dimensions are the same, and we can of course keep the file name since the person looks like Lena a bit. Would that one be OK? According to http://goo.gl/0K3qyP it should be OK as long as the attribution

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

> On 28.08.2014 17:53, Clément Bœsch wrote: > >ffplay http://lucy.pkh.me/lena.pnm > >According to http://goo.gl/0K3qyP it should be OK as long as the > >attribution is kept, which can be done in the commit message. Clément, I believe this URL shortener will please you very much: http://www.shady

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 28.08.2014 19:37, Nicolas George wrote: On 28.08.2014 17:53, Clément Bœsch wrote: ffplay http://lucy.pkh.me/lena.pnm According to http://goo.gl/0K3qyP it should be OK as long as the attribution is kept, which can be done in the commit message. Clément, I believe this URL shortener will p

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Thu, Aug 28, 2014 at 02:42:45PM +0200, Andreas Cadhalpun wrote: > On 28.08.2014 14:29, Derek Buitenhuis wrote: > >On 8/28/2014 12:28 PM, Andreas Cadhalpun wrote: > >>ffmpeg -f data -i http://samples.ffmpeg.org/image-samples/lena.pnm -c > >>copy -f data -map 0 -y lena.pnm > > Fr

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Thu, Aug 28, 2014 at 07:47:11PM +0200, Andreas Cadhalpun wrote: > On 28.08.2014 19:37, Nicolas George wrote: > >>On 28.08.2014 17:53, Clément Bœsch wrote: > >>>ffplay http://lucy.pkh.me/lena.pnm > > > >>>According to http://goo.gl/0K3qyP it should be OK as long as the > >>>attribution is kept, w

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On 28.08.2014 19:49, Reimar Döffinger wrote: I completely lost track of the discussion, but this all so shouts bikeshed and overkill. If I may suggest another colour for the shed: Put some random, freely licensed image where the Lena image was, put the Lena one in the sample repository and run th

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

Le primidi 11 fructidor, an CCXXII, Andreas Cadhalpun a écrit : > I like this colour of the shed. ;) > * move lena.pnm to the FATE samples > * add e.g. mona_lisa.pnm and duplicate the vsynth2 tests (as vsynth4?) > > Has someone a problem with this solution? I find it very good. Just a small ni

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Thu, Aug 28, 2014 at 07:28:07PM +0200, Andreas Cadhalpun wrote: > On 28.08.2014 17:53, Clément Bœsch wrote: > >ffplay http://lucy.pkh.me/lena.pnm > > > >File size and dimensions are the same, and we can of course keep the file > >name since the person looks like Lena a bit. > > > >Would that one

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Thu, Aug 28, 2014 at 08:55:40PM +0200, Clément Bœsch wrote: > On Thu, Aug 28, 2014 at 07:28:07PM +0200, Andreas Cadhalpun wrote: > > On 28.08.2014 17:53, Clément Bœsch wrote: > > >ffplay http://lucy.pkh.me/lena.pnm > > > > > >File size and dimensions are the same, and we can of course keep the f

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Sun, Aug 31, 2014 at 02:35:40PM +0200, Andreas Cadhalpun wrote: > On 28.08.2014 20:06, Nicolas George wrote: > >Le primidi 11 fructidor, an CCXXII, Andreas Cadhalpun a écrit : > >>I like this colour of the shed. ;) > >> * move lena.pnm to the FATE samples > >> * add e.g. mona_lisa.pnm and dupl

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Wed, Sep 03, 2014 at 04:38:15PM +0200, Clément Bœsch wrote: > On Sun, Aug 31, 2014 at 02:35:40PM +0200, Andreas Cadhalpun wrote: > > On 28.08.2014 20:06, Nicolas George wrote: > > >Le primidi 11 fructidor, an CCXXII, Andreas Cadhalpun a écrit : > > >>I like this colour of the shed. ;) > > >> *

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Wed, Sep 03, 2014 at 04:47:21PM +0200, Clément Bœsch wrote: > On Wed, Sep 03, 2014 at 04:38:15PM +0200, Clément Bœsch wrote: > > On Sun, Aug 31, 2014 at 02:35:40PM +0200, Andreas Cadhalpun wrote: > > > On 28.08.2014 20:06, Nicolas George wrote: > > > >Le primidi 11 fructidor, an CCXXII, Andreas

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Wed, Sep 03, 2014 at 04:38:15PM +0200, Clément Bœsch wrote: > On Sun, Aug 31, 2014 at 02:35:40PM +0200, Andreas Cadhalpun wrote: > > On 28.08.2014 20:06, Nicolas George wrote: > > >Le primidi 11 fructidor, an CCXXII, Andreas Cadhalpun a écrit : > > >>I like this colour of the shed. ;) > > >> *

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Thu, Aug 28, 2014 at 01:28:18PM +0200, Andreas Cadhalpun wrote: > On 28.08.2014 04:36, Michael Niedermayer wrote: > >On Wed, Aug 27, 2014 at 05:21:18PM -0800, Lou Logan wrote: > >>On Tue, 26 Aug 2014 13:41:42 +0200, Michael Niedermayer wrote: > >> > >>>Signed-off-by: Michael Niedermayer > >>>--

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Wed, Sep 03, 2014 at 07:30:49PM +0200, Michael Niedermayer wrote: > On Thu, Aug 28, 2014 at 01:28:18PM +0200, Andreas Cadhalpun wrote: > > On 28.08.2014 04:36, Michael Niedermayer wrote: > > >On Wed, Aug 27, 2014 at 05:21:18PM -0800, Lou Logan wrote: > > >>On Tue, 26 Aug 2014 13:41:42 +0200, Mic

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

On Thu, Sep 04, 2014 at 03:36:03PM +0200, Michael Niedermayer wrote: > On Wed, Sep 03, 2014 at 07:30:49PM +0200, Michael Niedermayer wrote: > > On Thu, Aug 28, 2014 at 01:28:18PM +0200, Andreas Cadhalpun wrote: > > > On 28.08.2014 04:36, Michael Niedermayer wrote: > > > >On Wed, Aug 27, 2014 at 05: