[FFmpeg-devel] [PATCH 1/2] fate: Add fate-ts-opus-demux

Michael Niedermayer michael at niedermayer.cc
Sun Apr 3 00:20:59 CEST 2016


On Sat, Apr 02, 2016 at 11:14:29PM +0200, wm4 wrote:
> On Sat, 2 Apr 2016 18:01:47 -0300
> James Almer <jamrial at gmail.com> wrote:
> 
> > On 4/2/2016 5:53 PM, wm4 wrote:
> > > On Sat,  2 Apr 2016 22:05:35 +0200
> > > Michael Niedermayer <michael at niedermayer.cc> wrote:
> > >   
> > >> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > >> ---
> > >>  tests/fate/demux.mak         |    3 +
> > >>  tests/ref/fate/ts-opus-demux |  514 ++++++++++++++++++++++++++++++++++++++++++
> > >>  2 files changed, 517 insertions(+)
> > >>  create mode 100644 tests/ref/fate/ts-opus-demux
> > >>  
> > > 
> > > Maybe you should wait with all these new tests until the codecpar
> > > changes have been merged.  
> > 
> > Why? None of these new tests should generate conflicts when the
> > codecpar branch is rebased to current git head. And they may even
> > reveal new regressions once they are run afterwards.
> 
> I'm sure they will reveal regressions, because michaelni seems to be
> specifically hunting for them. When the branch gets merged, there
> should be no failing fate tests, so adding such tests would hold back
> the merge.
> 
> While it's normally a good thing to catch such regressions (and to fix
> them before a big thing is merged), the codecpar merge has been going
> on for weeks. It's holding back merging of other important features
> (like vaapi encoding to name a particularly popular one), and there's a
> limit to which extend we can load further work on daemon and nev.
> 
> So I suggest that such arguably less important tests/regressions are
> applied and fixed later, so that merging can go on and none of the devs
> have to die from stress.

We can add the tests now and then just disable them in the merge
if the people working on the merge belive that that is the best choice.
It should make it easier to keep track of issues

also, everyone please help derek and others

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160403/c6929563/attachment.sig>


More information about the ffmpeg-devel mailing list