Message ID | 20161201163739.23548-1-michael@niedermayer.cc |
---|---|
State | Accepted |
Commit | 89092fafdde894c6ba4d4f8e3cd1cce0d68bfc22 |
Headers | show |
Good to go On 12/01/2016 08:37 AM, Michael Niedermayer wrote: > They have changed due to 122190392b297b7cd9783641d880df887fd80d07 > > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> > --- > tests/ffserver.regression.ref | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/ffserver.regression.ref b/tests/ffserver.regression.ref > index 3106f85..398c285 100644 > --- a/tests/ffserver.regression.ref > +++ b/tests/ffserver.regression.ref > @@ -1,5 +1,5 @@ > -233020d119085ba47535d5f2faf73cc0 *ff-test_h.avi > -431b75d1f12cb039acebad61a3d39225 *ff-test_l.avi > +0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi > +e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi > a767dbdf5d1bded3450279f812f97b37 *ff-test.swf > dc16f607e13328a832e73801cd21ec98 *ff-test_h.asf > 69337d6c8cd7ac7e626338decdbf41d3 *ff-test_l.asf
On Thu, Dec 01, 2016 at 01:41:40PM -0800, Reynaldo H. Verdejo Pinochet wrote:
> Good to go
patch applied
thx
[...]
On 01.12.2016 17:37, Michael Niedermayer wrote: > They have changed due to 122190392b297b7cd9783641d880df887fd80d07 > > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> > --- > tests/ffserver.regression.ref | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/ffserver.regression.ref b/tests/ffserver.regression.ref > index 3106f85..398c285 100644 > --- a/tests/ffserver.regression.ref > +++ b/tests/ffserver.regression.ref > @@ -1,5 +1,5 @@ > -233020d119085ba47535d5f2faf73cc0 *ff-test_h.avi > -431b75d1f12cb039acebad61a3d39225 *ff-test_l.avi > +0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi > +e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi > a767dbdf5d1bded3450279f812f97b37 *ff-test.swf > dc16f607e13328a832e73801cd21ec98 *ff-test_h.asf > 69337d6c8cd7ac7e626338decdbf41d3 *ff-test_l.asf The test is unfortunately not very reproducible. Right now I'm getting varying values like on one run: -0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi -e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi -a767dbdf5d1bded3450279f812f97b37 *ff-test.swf +81c49d1bfcd43bcc43aa6aeed081126f *ff-test_h.avi +2d642fbb77276ed1d12bf235a6a6bd57 *ff-test_l.avi +daeee6c1418c5b2dcddded9b659e9fc2 *ff-test.swf And on the next: -0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi -e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi +5e7fc3d0b2f4866866ae4f268dc431ca *ff-test_h.avi +6cc80ab0caec9499d1b82cd01fe7a861 *ff-test_l.avi I've no idea what the underlying problem is. Best regards, Andreas
On Fri, Dec 02, 2016 at 12:59:41AM +0100, Andreas Cadhalpun wrote: > On 01.12.2016 17:37, Michael Niedermayer wrote: > > They have changed due to 122190392b297b7cd9783641d880df887fd80d07 > > > > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> > > --- > > tests/ffserver.regression.ref | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/tests/ffserver.regression.ref b/tests/ffserver.regression.ref > > index 3106f85..398c285 100644 > > --- a/tests/ffserver.regression.ref > > +++ b/tests/ffserver.regression.ref > > @@ -1,5 +1,5 @@ > > -233020d119085ba47535d5f2faf73cc0 *ff-test_h.avi > > -431b75d1f12cb039acebad61a3d39225 *ff-test_l.avi > > +0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi > > +e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi > > a767dbdf5d1bded3450279f812f97b37 *ff-test.swf > > dc16f607e13328a832e73801cd21ec98 *ff-test_h.asf > > 69337d6c8cd7ac7e626338decdbf41d3 *ff-test_l.asf > > The test is unfortunately not very reproducible. > Right now I'm getting varying values like on one run: > -0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi > -e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi > -a767dbdf5d1bded3450279f812f97b37 *ff-test.swf > +81c49d1bfcd43bcc43aa6aeed081126f *ff-test_h.avi > +2d642fbb77276ed1d12bf235a6a6bd57 *ff-test_l.avi > +daeee6c1418c5b2dcddded9b659e9fc2 *ff-test.swf > > And on the next: > -0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi > -e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi > +5e7fc3d0b2f4866866ae4f268dc431ca *ff-test_h.avi > +6cc80ab0caec9499d1b82cd01fe7a861 *ff-test_l.avi > > I've no idea what the underlying problem is. it works here almost always i remember just one failure but you could try to use diff the ffprobe output showing as much as ffprobe supports to see what differs. binary compare (if they match in size) or ffmpeg -c copy -f framecrc - also please upload some different files somewhere it also could be some timeout issue if you have a slow box and maybe run this under valgrind otherwise printing checksums on packets at various points might help (that is if previous stuff points at that changing) ... [...]
On 02.12.2016 04:27, Michael Niedermayer wrote: > On Fri, Dec 02, 2016 at 12:59:41AM +0100, Andreas Cadhalpun wrote: >> The test is unfortunately not very reproducible. >> Right now I'm getting varying values like on one run: >> -0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi >> -e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi >> -a767dbdf5d1bded3450279f812f97b37 *ff-test.swf >> +81c49d1bfcd43bcc43aa6aeed081126f *ff-test_h.avi >> +2d642fbb77276ed1d12bf235a6a6bd57 *ff-test_l.avi >> +daeee6c1418c5b2dcddded9b659e9fc2 *ff-test.swf These files are truncated, probably because ffserver hadn't fully started yet. Maybe the 'sleep 2' should be increased or a better way to determine ffserver's readiness can be found? >> And on the next: >> -0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi >> -e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi >> +5e7fc3d0b2f4866866ae4f268dc431ca *ff-test_h.avi >> +6cc80ab0caec9499d1b82cd01fe7a861 *ff-test_l.avi >> >> I've no idea what the underlying problem is. > > it works here almost always i remember just one failure The bigger problem is that I almost always get the 5e/6c files, but not once got the checksums you added. > but you could try to use diff the ffprobe output showing as much as > ffprobe supports to see what differs. > binary compare (if they match in size) or ffmpeg -c copy -f framecrc - > > also please upload some different files somewhere As they're quite small, I've just attached the 5e/6c files to this mail. Maybe you can figure out where the difference comes from, when you compare them with the files you get. > it also could be some timeout issue if you have a slow box and maybe > run this under valgrind It's not a particularly slow box, but when run under valgrind, the 2 seconds sleep is definitely not long enough, but with 5 seconds I also get the 5e/6c files. Best regards, Andreas
diff --git a/tests/ffserver.regression.ref b/tests/ffserver.regression.ref index 3106f85..398c285 100644 --- a/tests/ffserver.regression.ref +++ b/tests/ffserver.regression.ref @@ -1,5 +1,5 @@ -233020d119085ba47535d5f2faf73cc0 *ff-test_h.avi -431b75d1f12cb039acebad61a3d39225 *ff-test_l.avi +0c9639f09decbc54c9f091dcf1ca0e8f *ff-test_h.avi +e28ba75853caf975e06d92955c9f7f73 *ff-test_l.avi a767dbdf5d1bded3450279f812f97b37 *ff-test.swf dc16f607e13328a832e73801cd21ec98 *ff-test_h.asf 69337d6c8cd7ac7e626338decdbf41d3 *ff-test_l.asf
They have changed due to 122190392b297b7cd9783641d880df887fd80d07 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> --- tests/ffserver.regression.ref | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)