Bluray M2TS file with separate PCR pid

[hashbord]

Summary of the bug:

When generating one M2TS file, the PCR marks are included in the Video pid. However, from the Blu-ray specification these marks require to be in one alone pid with number 0x1001.

After commit 422f0a61367f068fb8b1b9ab8cc2eee1808c803c the M2TS files generated by FFmpeg are starting to be Blu-ray conformant. However, still some errors are not targeted.

Reading ticket #7842 you can check that the "HDMV" descriptor and the Standard Pids problem are solved. However, the PCR in the pid 0x1001 is missing. Specifically the label M2TS_PCR_PID is defined, but it's never used!

Please, can you fix this error?
Thank you.

[Carl Eugen Hoyos]

It would be great if you - instead of analysing the issue - could explain how it can be reproduced.

[Marton Balint]

I think this can be reproduced by writing any m2ts file.

I have a patch series pending which fixes this, but I am not yet satisfied with it, that is why I have not posted it in ffmpeg-devel. But you can use it in the meantime:

​https://github.com/cus/ffmpeg/tree/mpegts2

[hashbord]

Replying to cehoyos:

It would be great if you - instead of analysing the issue - could explain how it can be reproduced.

Hi cehoyos,

I'm not a developer, so I can't provide a patch. However, I think the problem is well described: when generating M2TS files the result is not conformant with the Bluray specification because the PCR marks are included in the video PID, instead of inside an alone PCR pid.

How to reproduce it? Easy, create one m2ts file: "ffmpeg -i test.mp4 -bsf h264_mp4toannexb test.m2ts". Then you will see that the PCR marks are incrustated in the video pid 0x1011, and not in the expected pid 0x1001.

Futhermore, cus has provided a patch (Thank you man!): ​https://github.com/cus/ffmpeg/commit/224fac9bc11d9b5879878640c28726f97153e05a

Please, check it.

[hashbord]

Please cus, can do you publish a patch in the mailinglist?

[hashbord]

Hi cus and cehoyos,

As the solution it's already implemented, I suggest to prepare a patch and publish it in the dev mailinglist. I appreciate the effort done by CUS and I hope his work will be merged.

Regards.

[DreckSoft]

Is someone still working on this? I tried the version Marton Balint posted and the patch (need some manual work) on the regular version but both produce a result unplayable in my Blu-Ray player.

[Balling]

both produce a result unplayable in my Blu-Ray player.

What? This is not tsmuxer.

[DreckSoft]

txmuxer produces a playable result but newer versions have issues with EAC3, so no audio. That's why I tried ffmpeg in the first place.

[Balling]

I saw your ​https://github.com/justdan96/tsMuxer/issues/682 yeah.