[FFmpeg-devel] [PATCH 2/3] ffprobe: report field order for video streams
t.rapp at noa-archive.com
Tue Oct 25 10:20:39 EEST 2016
On 20.10.2016 18:01, Dave Rice wrote:
>> On Oct 20, 2016, at 3:28 AM, Tobias Rapp <t.rapp at noa-archive.com> wrote:
>> On 19.10.2016 23:15, Michael Niedermayer wrote:
>>> On Wed, Oct 19, 2016 at 02:35:21PM +0200, Tobias Rapp wrote:
>>>> From: Rodger Combs <rodger.combs at gmail.com>
>>>> Reviewed-by: Tobias Rapp <t.rapp at noa-archive.com>
>>>> doc/ffprobe.xsd | 1 +
>>>> ffprobe.c | 13 +++++++++++++
>>>> tests/ref/fate/concat-demuxer-extended-lavf-mxf | 2 +-
>>>> tests/ref/fate/concat-demuxer-extended-lavf-mxf_d10 | 2 +-
>>>> tests/ref/fate/concat-demuxer-simple1-lavf-mxf | 2 +-
>>>> tests/ref/fate/concat-demuxer-simple1-lavf-mxf_d10 | 2 +-
>>>> tests/ref/fate/concat-demuxer-simple2-lavf-ts | 2 +-
>>>> tests/ref/fate/ffprobe_compact | 4 ++--
>>>> tests/ref/fate/ffprobe_csv | 4 ++--
>>>> tests/ref/fate/ffprobe_default | 2 ++
>>>> tests/ref/fate/ffprobe_flat | 2 ++
>>>> tests/ref/fate/ffprobe_ini | 2 ++
>>>> 12 files changed, 29 insertions(+), 9 deletions(-)
>>> This too doesnt apply:
>>> Applying: ffprobe: report field order for video streams
>>> Using index info to reconstruct a base tree...
>>> error: patch failed: tests/ref/fate/ffprobe_compact:27
>>> error: tests/ref/fate/ffprobe_compact: patch does not apply
>>> error: Did you hand edit your patch?
>>> It does not apply to blobs recorded in its index.
>> Could be due to encoding problems or long lines. Have rebased the patch onto current git HEAD and attached it as a file.
> I tested this and the new version applied for me. The output of xml=q=1 validated against the revision. LGTM. Having this data in ffprobe is really helpful.
Apparently this patch has been pushed by Rodger in the meanwhile.
More information about the ffmpeg-devel