[FFmpeg-devel] [PATCH 2/3] ffprobe: report field order for video streams
Dave Rice
dave at dericed.com
Thu Oct 20 19:01:06 EEST 2016
> 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.
Dave Rice
More information about the ffmpeg-devel
mailing list