After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 757498 - Fix compilation with ffmpeg 2.9
Fix compilation with ffmpeg 2.9
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other Linux
: Normal normal
: 1.7.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-11-02 21:26 UTC by Sebastian Dröge (slomo)
Modified: 2016-01-16 15:26 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
avdeinterlace: Port non-deprecated AVFilter API (7.19 KB, patch)
2015-11-04 20:28 UTC, Sebastian Dröge (slomo)
committed Details | Review
avviddec: Stop using CODEC_FLAG_EMU_EDGE and avcodec_get_edge_width() (1.75 KB, patch)
2015-11-04 20:28 UTC, Sebastian Dröge (slomo)
rejected Details | Review
libav: Remove usage of deprecated API (12.58 KB, patch)
2015-11-04 20:28 UTC, Sebastian Dröge (slomo)
committed Details | Review
libav: Bitrate field changed from int to int64_t, fix compiler warnings (1.91 KB, patch)
2016-01-16 14:50 UTC, Sebastian Dröge (slomo)
committed Details | Review
avdeinterlace: Chain up to parent class' dispose() (840 bytes, patch)
2016-01-16 14:50 UTC, Sebastian Dröge (slomo)
committed Details | Review

Description Sebastian Dröge (slomo) 2015-11-02 21:26:21 UTC
A patch is included in the Debian bug report: https://bugs.debian.org/803822

Needs to be reviewed and possibly fixed up.
Comment 1 Sebastian Dröge (slomo) 2015-11-04 20:28:20 UTC
Created attachment 314853 [details] [review]
avdeinterlace: Port non-deprecated AVFilter API
Comment 2 Sebastian Dröge (slomo) 2015-11-04 20:28:34 UTC
Created attachment 314854 [details] [review]
avviddec: Stop using CODEC_FLAG_EMU_EDGE and avcodec_get_edge_width()

It's deprecated now and not needed anymore:
https://anonscm.debian.org/cgit/pkg-multimedia/ffmpeg.git/tree/doc/APIchanges#n558
Comment 3 Sebastian Dröge (slomo) 2015-11-04 20:28:41 UTC
Created attachment 314855 [details] [review]
libav: Remove usage of deprecated API
Comment 4 Sebastian Dröge (slomo) 2015-11-04 20:30:05 UTC
This is also all relevant for 2.8 already, just that it still compiles without then.
Comment 5 Sebastian Dröge (slomo) 2015-11-04 20:30:32 UTC
(In reply to Sebastian Dröge (slomo) from comment #2)
> Created attachment 314854 [details] [review] [review]
> avviddec: Stop using CODEC_FLAG_EMU_EDGE and avcodec_get_edge_width()
> 
> It's deprecated now and not needed anymore:
> https://anonscm.debian.org/cgit/pkg-multimedia/ffmpeg.git/tree/doc/
> APIchanges#n558

This one should probably just remove the usage of GstVideoAlignment completely then?
Comment 6 Nicolas Dufresne (ndufresne) 2015-11-04 20:32:38 UTC
Need more research imho, could have been replaced.
Comment 7 Sebastian Dröge (slomo) 2015-11-04 20:49:36 UTC
Comment on attachment 314855 [details] [review]
libav: Remove usage of deprecated API

This one seems fine
Comment 8 Sebastian Dröge (slomo) 2015-12-28 10:36:37 UTC
Nicolas, the edge API also gives deprecation warnings now. So I guess it's indeed unneeded nowadays.
Comment 9 Nicolas Dufresne (ndufresne) 2015-12-28 20:52:44 UTC
Ok, one will need to check the new behaviour. Did they removed the left/top padding ? Note, we still need some video alignment code, so we keep some best effort negotiation of the stride (even if direct rendering is next to near impossible to obtain now). (note, I'm just giving feedback, I'll be back hacking next week)
Comment 10 Sebastian Dröge (slomo) 2015-12-29 08:26:40 UTC
I don't know yet, I just saw all the deprecation warnings during compilation and didn't look closer yet :)
Comment 11 Sebastian Dröge (slomo) 2016-01-16 14:50:23 UTC
Created attachment 319175 [details] [review]
libav: Bitrate field changed from int to int64_t, fix compiler warnings
Comment 12 Sebastian Dröge (slomo) 2016-01-16 14:50:28 UTC
Created attachment 319176 [details] [review]
avdeinterlace: Chain up to parent class' dispose()
Comment 13 Sebastian Dröge (slomo) 2016-01-16 15:20:46 UTC
Comment on attachment 314854 [details] [review]
avviddec: Stop using CODEC_FLAG_EMU_EDGE and avcodec_get_edge_width()

This one is not mergeable yet, it seems like EMU_EDGE is still used in some places in current master although it's marked as deprecated. It looks like it's going to be removed with API version 58, but master is still at 57.
Comment 14 Sebastian Dröge (slomo) 2016-01-16 15:25:39 UTC
Attachment 314853 [details] pushed as ddec3a2 - avdeinterlace: Port non-deprecated AVFilter API
Attachment 314855 [details] pushed as 6235a04 - libav: Remove usage of deprecated API
Attachment 319175 [details] pushed as 46fb2e9 - libav: Bitrate field changed from int to int64_t, fix compiler warnings
Attachment 319176 [details] pushed as e04bcf0 - avdeinterlace: Chain up to parent class' dispose()
Comment 15 Sebastian Dröge (slomo) 2016-01-16 15:26:36 UTC
This now builds with 2.8 and git master.