[FFmpeg-devel] Revision 9087 regression test fail
Fri May 25 08:32:55 CEST 2007
On Wednesday 23 May 2007 13:06, Michael Niedermayer wrote:
> On Wed, May 23, 2007 at 08:40:35AM +0300, Siarhei Siamashka wrote:
> > On Wednesday 23 May 2007 04:45, Michael Niedermayer wrote:
> > > > Here is a starting part of the regression test report where errors
> > > > initially pop up (the rest of log is attached):
> > >
> > > the attached file is definitly not the complete output of "make test"
> > > i think its not even the complete output of make seektest as that
> > > should output a proper diff but the headers are missing ...
> > Yes, indeed, it only contained the last part of log with errors (the same
> > information which was enough for you in the report of Panagiotis
> > Issaris).
> > I'm interpreting your reply as a request for complete output of "make
> > test". Now attached the file with the output generated in 'maketest.log'
> > by invoking the following command from shell: "make test &> maketest.log"
> does it work with --enable-gpl? (it should)
Yes, it works, at least it reports that the tests succeeded. Thanks for
pinning down this issue. Though Benoit was faster to test it and propose
a patch , I felt like confirming it just in case (as I was the initial
reporter for this issue).
> if so the seek test should be disabled without that ...
On a related side, maybe 'Regression tests' part in ffmpeg documentation
should be updated to add an advice (or requirement) to run regression tests
with all possible features enabled (including --enable-gpl) for better code
coverage? Otherwise 'pass regression tests' step in patch submission
checklist is not well defined. Even a few more lines in the documentation can
help outsiders to get into ffmpeg development easier and make them provide
more useful feedback/bugreports reducing possibility of unintentionally doing
something in a wrong way and misunderstanding. Thank you again for your
continued ffmpeg support and fast resolution of problems.
More information about the ffmpeg-devel