Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread Michael Niedermayer
On Mon, Dec 12, 2016 at 12:12:37PM +0100, Carl Eugen Hoyos wrote: > Hi! > > I saw only after writing this patch that a Google employee has sent > a more complicated variant of this patch in November (searching for > a description of the vendor field showed the patch). > > Please comment, Carl E

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread wm4
On Fri, 27 Jan 2017 12:35:25 +0100 Carl Eugen Hoyos wrote: > 2017-01-27 11:55 GMT+01:00 wm4 : > > On Fri, 27 Jan 2017 11:21:08 +0100 > > Carl Eugen Hoyos wrote: > > > >> 2017-01-27 10:42 GMT+01:00 wm4 : > >> > On Fri, 27 Jan 2017 10:38:23 +0100 > >> > Carl Eugen Hoyos wrote: > >> > > >> >

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread Carl Eugen Hoyos
2017-01-27 11:55 GMT+01:00 wm4 : > On Fri, 27 Jan 2017 11:21:08 +0100 > Carl Eugen Hoyos wrote: > >> 2017-01-27 10:42 GMT+01:00 wm4 : >> > On Fri, 27 Jan 2017 10:38:23 +0100 >> > Carl Eugen Hoyos wrote: >> > >> >> 2017-01-27 10:29 GMT+01:00 wm4 : >> >> > On Fri, 27 Jan 2017 10:19:32 +0100 >> >> >

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread wm4
On Fri, 27 Jan 2017 11:21:08 +0100 Carl Eugen Hoyos wrote: > 2017-01-27 10:42 GMT+01:00 wm4 : > > On Fri, 27 Jan 2017 10:38:23 +0100 > > Carl Eugen Hoyos wrote: > > > >> 2017-01-27 10:29 GMT+01:00 wm4 : > >> > On Fri, 27 Jan 2017 10:19:32 +0100 > >> > Carl Eugen Hoyos wrote: > >> > > >> >

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread Carl Eugen Hoyos
2017-01-27 10:42 GMT+01:00 wm4 : > On Fri, 27 Jan 2017 10:38:23 +0100 > Carl Eugen Hoyos wrote: > >> 2017-01-27 10:29 GMT+01:00 wm4 : >> > On Fri, 27 Jan 2017 10:19:32 +0100 >> > Carl Eugen Hoyos wrote: >> > >> >> 2017-01-27 10:09 GMT+01:00 wm4 : >> >> > On Fri, 27 Jan 2017 09:39:03 +0100 >> >> >

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread Paul B Mahol
On 12/12/16, Carl Eugen Hoyos wrote: > Hi! > > I saw only after writing this patch that a Google employee has sent > a more complicated variant of this patch in November (searching for > a description of the vendor field showed the patch). > > Please comment, Carl Eugen > -1 _

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread wm4
On Fri, 27 Jan 2017 10:38:23 +0100 Carl Eugen Hoyos wrote: > 2017-01-27 10:29 GMT+01:00 wm4 : > > On Fri, 27 Jan 2017 10:19:32 +0100 > > Carl Eugen Hoyos wrote: > > > >> 2017-01-27 10:09 GMT+01:00 wm4 : > >> > On Fri, 27 Jan 2017 09:39:03 +0100 > >> > Carl Eugen Hoyos wrote: > >> > > >> >

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread Carl Eugen Hoyos
2017-01-27 10:29 GMT+01:00 wm4 : > On Fri, 27 Jan 2017 10:19:32 +0100 > Carl Eugen Hoyos wrote: > >> 2017-01-27 10:09 GMT+01:00 wm4 : >> > On Fri, 27 Jan 2017 09:39:03 +0100 >> > Carl Eugen Hoyos wrote: >> > >> >> 2017-01-27 9:17 GMT+01:00 wm4 : >> >> >> >> > You're completely misunderstanding. >

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread wm4
On Fri, 27 Jan 2017 10:19:32 +0100 Carl Eugen Hoyos wrote: > 2017-01-27 10:09 GMT+01:00 wm4 : > > On Fri, 27 Jan 2017 09:39:03 +0100 > > Carl Eugen Hoyos wrote: > > > >> 2017-01-27 9:17 GMT+01:00 wm4 : > >> > >> > You're completely misunderstanding. > >> > >> Would you mind to elaborate?

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread Carl Eugen Hoyos
2017-01-27 10:09 GMT+01:00 wm4 : > On Fri, 27 Jan 2017 09:39:03 +0100 > Carl Eugen Hoyos wrote: > >> 2017-01-27 9:17 GMT+01:00 wm4 : >> >> > You're completely misunderstanding. >> >> Would you mind to elaborate? > > FFmpeg shouldn't mux codec-specific tags into a different > container. This is no

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread wm4
On Fri, 27 Jan 2017 09:39:03 +0100 Carl Eugen Hoyos wrote: > 2017-01-27 9:17 GMT+01:00 wm4 : > > > You're completely misunderstanding. > > Would you mind to elaborate? FFmpeg shouldn't mux codec-specific tags into a different container. Your ffmpeg.c patch works for transcoding only, not re

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread Carl Eugen Hoyos
2017-01-27 9:17 GMT+01:00 wm4 : > You're completely misunderstanding. Would you mind to elaborate? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread wm4
On Fri, 27 Jan 2017 09:05:15 +0100 Carl Eugen Hoyos wrote: > 2017-01-27 8:56 GMT+01:00 wm4 : > > On Fri, 27 Jan 2017 08:26:26 +0100 > > Carl Eugen Hoyos wrote: > > > >> 2017-01-27 7:04 GMT+01:00 wm4 : > >> > On Thu, 26 Jan 2017 18:06:39 +0100 > >> > Carl Eugen Hoyos wrote: > >> > > >> >>

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-27 Thread Carl Eugen Hoyos
2017-01-27 8:56 GMT+01:00 wm4 : > On Fri, 27 Jan 2017 08:26:26 +0100 > Carl Eugen Hoyos wrote: > >> 2017-01-27 7:04 GMT+01:00 wm4 : >> > On Thu, 26 Jan 2017 18:06:39 +0100 >> > Carl Eugen Hoyos wrote: >> > >> >> 2017-01-26 9:26 GMT+01:00 wm4 : >> >> > On Thu, 26 Jan 2017 09:16:00 +0100 >> >> > Ca

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-26 Thread wm4
On Fri, 27 Jan 2017 08:26:26 +0100 Carl Eugen Hoyos wrote: > 2017-01-27 7:04 GMT+01:00 wm4 : > > On Thu, 26 Jan 2017 18:06:39 +0100 > > Carl Eugen Hoyos wrote: > > > >> 2017-01-26 9:26 GMT+01:00 wm4 : > >> > On Thu, 26 Jan 2017 09:16:00 +0100 > >> > Carl Eugen Hoyos wrote: > >> > > >> >>

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-26 Thread Carl Eugen Hoyos
2017-01-27 7:04 GMT+01:00 wm4 : > On Thu, 26 Jan 2017 18:06:39 +0100 > Carl Eugen Hoyos wrote: > >> 2017-01-26 9:26 GMT+01:00 wm4 : >> > On Thu, 26 Jan 2017 09:16:00 +0100 >> > Carl Eugen Hoyos wrote: >> > >> >> 2017-01-26 9:07 GMT+01:00 wm4 : >> >> >> >> >> >> > Any metadata you export can and w

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-26 Thread wm4
On Thu, 26 Jan 2017 18:06:39 +0100 Carl Eugen Hoyos wrote: > 2017-01-26 9:26 GMT+01:00 wm4 : > > On Thu, 26 Jan 2017 09:16:00 +0100 > > Carl Eugen Hoyos wrote: > > > >> 2017-01-26 9:07 GMT+01:00 wm4 : > >> > >> >> >> > Any metadata you export can and will get copied to a new file when > >> >

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-26 Thread Carl Eugen Hoyos
2017-01-26 9:26 GMT+01:00 wm4 : > On Thu, 26 Jan 2017 09:16:00 +0100 > Carl Eugen Hoyos wrote: > >> 2017-01-26 9:07 GMT+01:00 wm4 : >> >> >> >> > Any metadata you export can and will get copied to a new file when >> >> >> > remuxing, therefor exporting arbitrary info that isn't actual stream >> >>

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-26 Thread wm4
On Thu, 26 Jan 2017 09:16:00 +0100 Carl Eugen Hoyos wrote: > 2017-01-26 9:07 GMT+01:00 wm4 : > > >> >> > Any metadata you export can and will get copied to a new file when > >> >> > remuxing, therefor exporting arbitrary info that isn't actual stream > >> >> > metadata tags in metadata is proble

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-26 Thread Carl Eugen Hoyos
2017-01-26 9:07 GMT+01:00 wm4 : >> >> > Any metadata you export can and will get copied to a new file when >> >> > remuxing, therefor exporting arbitrary info that isn't actual stream >> >> > metadata tags in metadata is problematic - it carries over to the >> >> > destination file, in which it wo

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-26 Thread wm4
On Thu, 26 Jan 2017 08:26:02 +0100 Carl Eugen Hoyos wrote: > 2017-01-26 6:24 GMT+01:00 wm4 : > > On Thu, 26 Jan 2017 00:35:17 +0100 > > Carl Eugen Hoyos wrote: > > > >> 2017-01-26 0:19 GMT+01:00 Hendrik Leppkes : > >> > On Thu, Jan 26, 2017 at 10:04 AM, Carl Eugen Hoyos > >> > wrote: > >

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-25 Thread Carl Eugen Hoyos
2017-01-26 6:24 GMT+01:00 wm4 : > On Thu, 26 Jan 2017 00:35:17 +0100 > Carl Eugen Hoyos wrote: > >> 2017-01-26 0:19 GMT+01:00 Hendrik Leppkes : >> > On Thu, Jan 26, 2017 at 10:04 AM, Carl Eugen Hoyos >> > wrote: >> >> 2017-01-25 14:22 GMT+01:00 wm4 : >> >>> On Mon, 12 Dec 2016 12:12:37 +0100 >>

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-25 Thread wm4
On Thu, 26 Jan 2017 00:35:17 +0100 Carl Eugen Hoyos wrote: > 2017-01-26 0:19 GMT+01:00 Hendrik Leppkes : > > On Thu, Jan 26, 2017 at 10:04 AM, Carl Eugen Hoyos > > wrote: > >> 2017-01-25 14:22 GMT+01:00 wm4 : > >>> On Mon, 12 Dec 2016 12:12:37 +0100 > >>> Carl Eugen Hoyos wrote: > >>> >

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-25 Thread Carl Eugen Hoyos
2017-01-26 0:19 GMT+01:00 Hendrik Leppkes : > On Thu, Jan 26, 2017 at 10:04 AM, Carl Eugen Hoyos wrote: >> 2017-01-25 14:22 GMT+01:00 wm4 : >>> On Mon, 12 Dec 2016 12:12:37 +0100 >>> Carl Eugen Hoyos wrote: >>> From 7c26220a8734fe7dc293efe6c13e3baf91defc7e Mon Sep 17 00:00:00 2001 From:

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-25 Thread Hendrik Leppkes
On Thu, Jan 26, 2017 at 10:04 AM, Carl Eugen Hoyos wrote: > 2017-01-25 14:22 GMT+01:00 wm4 : >> On Mon, 12 Dec 2016 12:12:37 +0100 >> Carl Eugen Hoyos wrote: >> >>> From 7c26220a8734fe7dc293efe6c13e3baf91defc7e Mon Sep 17 00:00:00 2001 >>> From: Carl Eugen Hoyos >>> Date: Mon, 12 Dec 2016 12:07:

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-25 Thread Carl Eugen Hoyos
2017-01-25 14:22 GMT+01:00 wm4 : > On Mon, 12 Dec 2016 12:12:37 +0100 > Carl Eugen Hoyos wrote: > >> From 7c26220a8734fe7dc293efe6c13e3baf91defc7e Mon Sep 17 00:00:00 2001 >> From: Carl Eugen Hoyos >> Date: Mon, 12 Dec 2016 12:07:27 +0100 >> Subject: [PATCH 2/2] lavf/mov: Export vendor metadata.

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-25 Thread wm4
On Mon, 12 Dec 2016 12:12:37 +0100 Carl Eugen Hoyos wrote: > From 7c26220a8734fe7dc293efe6c13e3baf91defc7e Mon Sep 17 00:00:00 2001 > From: Carl Eugen Hoyos > Date: Mon, 12 Dec 2016 12:07:27 +0100 > Subject: [PATCH 2/2] lavf/mov: Export vendor metadata. > > --- > libavformat/mov.c |

Re: [FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2017-01-24 Thread Carl Eugen Hoyos
2016-12-12 12:12 GMT+01:00 Carl Eugen Hoyos : > I saw only after writing this patch that a Google employee has sent > a more complicated variant of this patch in November (searching for > a description of the vendor field showed the patch). Ping for the ffmpeg patch. I will add audio vendor to t

[FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

2016-12-12 Thread Carl Eugen Hoyos
Hi! I saw only after writing this patch that a Google employee has sent a more complicated variant of this patch in November (searching for a description of the vendor field showed the patch). Please comment, Carl Eugen From 182ee7b3dca51a49a606c4d11758819ec29fc181 Mon Sep 17 00:00:00 2001 From