[FFmpeg-devel] [PATCH 2/3] ffprobe: report field order for video streams

[Tobias Rapp]

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.

Thanks,
Tobias