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 797286 - aom: Fails to build with AOM master
aom: Fails to build with AOM master
Status: RESOLVED DUPLICATE of bug 791674
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.14.1
Other Linux
: Normal normal
: 1.15.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-10-15 16:31 UTC by Xavier Claessens
Modified: 2018-10-15 17:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
aom: Fix build caused by API change in libaom (1.06 KB, patch)
2018-10-15 17:15 UTC, Xavier Claessens
none Details | Review

Description Xavier Claessens 2018-10-15 16:31:05 UTC
../subprojects/gst-plugins-bad/ext/aom/gstav1enc.c: In function ‘gst_av1_enc_process’:
../subprojects/gst-plugins-bad/ext/aom/gstav1enc.c:360:36: error: ‘AOM_FRAME_IS_INVISIBLE’ undeclared (first use in this function); did you mean ‘AOM_FRAME_IS_INTRAONLY’?
       if ((pkt->data.frame.flags & AOM_FRAME_IS_INVISIBLE) != 0)
                                    ^~~~~~~~~~~~~~~~~~~~~~
                                    AOM_FRAME_IS_INTRAONLY

It looks like AOM_FRAME_IS_INVISIBLE macro got removed from libaom with that commit: https://aomedia.googlesource.com/aom/+/22043ef429129c886c03c122ac3f603901882809

So they broke API, I don't know if that should be reported as a bug on their side, or if we can update our side to work with new API?
Comment 1 Xavier Claessens 2018-10-15 16:32:43 UTC
Adding Sean DuBois in CC because he's listed as author of that code.
Comment 2 Xavier Claessens 2018-10-15 17:15:13 UTC
Created attachment 373931 [details] [review]
aom: Fix build caused by API change in libaom

AOM_FRAME_IS_INVISIBLE got removed by this commit:
https://aomedia.googlesource.com/aom/+/22043ef429129c886c03c122ac3f603901882809
Comment 3 Olivier Crête 2018-10-15 17:49:41 UTC
This was already resolved with one of the patches from #791674

commit cc9d65a512db89238e9b319b12262449e94097bb
Author: Wonchul Lee <chul0812@gmail.com>
Date:   Wed Oct 3 17:13:32 2018 +0900

    av1enc: fix compliation with removed defines
    
    AOM_FRAME_IS_INVISIBLE is unused and remove in aom_encoder.h
    
    https://aomedia-review.googlesource.com/c/aom/+/70421
    
    https://bugzilla.gnome.org/show_bug.cgi?id=791674

*** This bug has been marked as a duplicate of bug 791674 ***