Re: [libav-devel] [PATCH] tests: Do not include stdout/stderr or diff if the test passed

2015-04-19 Thread Luca Barbato
On 18/04/15 23:52, Timothy Gu wrote: > On Wed, Apr 15, 2015 at 12:18 AM Luca Barbato wrote: > >> On 14/04/15 22:05, Timothy Gu wrote: >>> No. >>> >>> Consider three tests, the first one succeeds without any output, the >> second >>> one outputs some debug information, and the third fails. >>> >>>

Re: [libav-devel] [PATCH] tests: Do not include stdout/stderr or diff if the test passed

2015-04-18 Thread Timothy Gu
On Wed, Apr 15, 2015 at 12:18 AM Luca Barbato wrote: > On 14/04/15 22:05, Timothy Gu wrote: > > No. > > > > Consider three tests, the first one succeeds without any output, the > second > > one outputs some debug information, and the third fails. > > > > Before: > > > > test1:0:: > > test2:0:adsf

Re: [libav-devel] [PATCH] tests: Do not include stdout/stderr or diff if the test passed

2015-04-15 Thread Luca Barbato
On 14/04/15 22:05, Timothy Gu wrote: > No. > > Consider three tests, the first one succeeds without any output, the second > one outputs some debug information, and the third fails. > > Before: > > test1:0:: > test2:0:adsfasdfdsafdsafdsFADFadsf==: > test3:-2:asfasfwer34fsdadf:adsfrqwfdsaadsf >

Re: [libav-devel] [PATCH] tests: Do not include stdout/stderr or diff if the test passed

2015-04-14 Thread Timothy Gu
On Tue, Apr 14, 2015 at 3:43 AM Diego Biurrun wrote: > On Fri, Apr 10, 2015 at 10:36:03AM -0700, Timothy Gu wrote: > > FATE currently discards this information anyway, so why waste the > > disk space? > > --- a/tests/fate-run.sh > > +++ b/tests/fate-run.sh > > @@ -211,7 +211,13 @@ else > > > > -e

Re: [libav-devel] [PATCH] tests: Do not include stdout/stderr or diff if the test passed

2015-04-14 Thread Diego Biurrun
On Fri, Apr 10, 2015 at 10:36:03AM -0700, Timothy Gu wrote: > FATE currently discards this information anyway, so why waste the > disk space? > --- a/tests/fate-run.sh > +++ b/tests/fate-run.sh > @@ -211,7 +211,13 @@ else > > -echo "${test}:${sig:-$err}:$($base64 <$cmpfile):$($base64 <$errfile)"