Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] Chromium 110 and ffmpeg 5.1.2
@ 2023-02-22 20:41 Drew Abbott
  2023-02-22 22:38 ` Timo Rothenpieler
  2023-02-23  0:28 ` Hendrik Leppkes
  0 siblings, 2 replies; 4+ messages in thread
From: Drew Abbott @ 2023-02-22 20:41 UTC (permalink / raw)
  To: ffmpeg-devel

Hello,

I am trying to build Chromium 110 with ffmpeg version 5.1.2. One of
the errors I get while building has to do with the API for accessing
first_dts. It looks like previously it was just a member of AVStream,
but now it can only be accessed with av_stream_get_first_dts:
https://github.com/chromium/chromium/commit/b94755e4633045be96ab5e0bdde0db7e16a804bd
I can see now first_dts is a member of FFStream, but I don't know how
to go from an AVStream to an FFStream without using internals of
libavformat, if that is even possible. How was first_dts accessed in
5.1.2 from an AVStream?

Thanks,
Drew
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [FFmpeg-devel] Chromium 110 and ffmpeg 5.1.2
  2023-02-22 20:41 [FFmpeg-devel] Chromium 110 and ffmpeg 5.1.2 Drew Abbott
@ 2023-02-22 22:38 ` Timo Rothenpieler
  2023-02-23  0:28 ` Hendrik Leppkes
  1 sibling, 0 replies; 4+ messages in thread
From: Timo Rothenpieler @ 2023-02-22 22:38 UTC (permalink / raw)
  To: ffmpeg-devel

On 22.02.2023 21:41, Drew Abbott wrote:
> Hello,
> 
> I am trying to build Chromium 110 with ffmpeg version 5.1.2. One of
> the errors I get while building has to do with the API for accessing
> first_dts. It looks like previously it was just a member of AVStream,
> but now it can only be accessed with av_stream_get_first_dts:
> https://github.com/chromium/chromium/commit/b94755e4633045be96ab5e0bdde0db7e16a804bd
> I can see now first_dts is a member of FFStream, but I don't know how
> to go from an AVStream to an FFStream without using internals of
> libavformat, if that is even possible. How was first_dts accessed in
> 5.1.2 from an AVStream?

first_dts is a purely internal field of libavformat, you can't access it 
from outside of it.
av_stream_get_first_dts() is not a function I see anywhere in the ffmpeg 
codebase.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [FFmpeg-devel] Chromium 110 and ffmpeg 5.1.2
  2023-02-22 20:41 [FFmpeg-devel] Chromium 110 and ffmpeg 5.1.2 Drew Abbott
  2023-02-22 22:38 ` Timo Rothenpieler
@ 2023-02-23  0:28 ` Hendrik Leppkes
  2023-02-23  3:00   ` Neal Gompa
  1 sibling, 1 reply; 4+ messages in thread
From: Hendrik Leppkes @ 2023-02-23  0:28 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

On Wed, Feb 22, 2023 at 9:42 PM Drew Abbott <abbotta4@gmail.com> wrote:
>
> Hello,
>
> I am trying to build Chromium 110 with ffmpeg version 5.1.2. One of
> the errors I get while building has to do with the API for accessing
> first_dts. It looks like previously it was just a member of AVStream,
> but now it can only be accessed with av_stream_get_first_dts:
> https://github.com/chromium/chromium/commit/b94755e4633045be96ab5e0bdde0db7e16a804bd
> I can see now first_dts is a member of FFStream, but I don't know how
> to go from an AVStream to an FFStream without using internals of
> libavformat, if that is even possible. How was first_dts accessed in
> 5.1.2 from an AVStream?

Chrome hacks their ffmpeg rather then fixing their code. You can't
build it with anything but their canned version.

- Hendrik
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [FFmpeg-devel] Chromium 110 and ffmpeg 5.1.2
  2023-02-23  0:28 ` Hendrik Leppkes
@ 2023-02-23  3:00   ` Neal Gompa
  0 siblings, 0 replies; 4+ messages in thread
From: Neal Gompa @ 2023-02-23  3:00 UTC (permalink / raw)
  To: FFmpeg development discussions and patches; +Cc: Andreas Schneider

On Wed, Feb 22, 2023 at 7:28 PM Hendrik Leppkes <h.leppkes@gmail.com> wrote:
>
> On Wed, Feb 22, 2023 at 9:42 PM Drew Abbott <abbotta4@gmail.com> wrote:
> >
> > Hello,
> >
> > I am trying to build Chromium 110 with ffmpeg version 5.1.2. One of
> > the errors I get while building has to do with the API for accessing
> > first_dts. It looks like previously it was just a member of AVStream,
> > but now it can only be accessed with av_stream_get_first_dts:
> > https://github.com/chromium/chromium/commit/b94755e4633045be96ab5e0bdde0db7e16a804bd
> > I can see now first_dts is a member of FFStream, but I don't know how
> > to go from an AVStream to an FFStream without using internals of
> > libavformat, if that is even possible. How was first_dts accessed in
> > 5.1.2 from an AVStream?
>
> Chrome hacks their ffmpeg rather then fixing their code. You can't
> build it with anything but their canned version.
>

There's an upstream bug for it:
https://bugs.chromium.org/p/chromium/issues/detail?id=1306560

There was an attempt by Andreas Schneider to fix this, but the fix
wasn't complete enough for their testsuite:
https://chromium-review.googlesource.com/c/chromium/src/+/3525614

The Chromium people don't seem to engage with upstream projects (in
particular ffmpeg) based on this lovely gem:
https://bugs.chromium.org/p/chromium/issues/detail?id=1306560#c8

In Fedora, we're carrying the patch, as it seems to work for us. I
hope the fix lands upstream someday.




--
真実はいつも一つ!/ Always, there's only one truth!
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-02-23  3:01 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-22 20:41 [FFmpeg-devel] Chromium 110 and ffmpeg 5.1.2 Drew Abbott
2023-02-22 22:38 ` Timo Rothenpieler
2023-02-23  0:28 ` Hendrik Leppkes
2023-02-23  3:00   ` Neal Gompa

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git