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 582074 - [faad] Fails to build with faad 2.6 (misdetected as 2.7)
[faad] Fails to build with faad 2.6 (misdetected as 2.7)
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal blocker
: 0.10.12
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-05-10 14:50 UTC by Sebastian Dröge (slomo)
Modified: 2009-05-10 15:18 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Sebastian Dröge (slomo) 2009-05-10 14:50:04 UTC
Hi,
faad currently fails to build on x86. Used faad version is 2.6.1-3.1 as in Debian/unstable.

cc1: warnings being treated as errors
gstfaad.c: In function 'gst_faad_setcaps':
gstfaad.c:338: error: passing argument 4 of 'NeAACDecInit2' from incompatible pointer type
gstfaad.c: In function 'gst_faad_chain':
gstfaad.c:1310: error: passing argument 4 of 'NeAACDecInit' from incompatible pointer type
gstfaad.c:1317: error: passing argument 4 of 'NeAACDecInit2' from incompatible pointer type
make[3]: *** [libgstfaad_la-gstfaad.lo] Error 1
make[2]: *** [all-recursive] Error 1
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2
Comment 1 Sebastian Dröge (slomo) 2009-05-10 15:00:45 UTC
Also on x64_64. Problem is, that FAAD2_MINOR_VERSION == 7 but the header only says FAAD2_VERSION "2.6"...
Comment 2 Sebastian Dröge (slomo) 2009-05-10 15:13:55 UTC
Because the configure check is broken... it matches

cpp configure_test.c | grep 2.7
# 297 "/usr/include/features.h" 2 3 4

Fixing...
Comment 3 Sebastian Dröge (slomo) 2009-05-10 15:18:30 UTC
commit 9e2b9d18ac66da68ef574485a362cfc0550b5aa0
Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
Date:   Sun May 10 17:17:15 2009 +0200

    faad: Fix configure check for the FAAD version
    
    The previous version matched things like 297 for
    version 2.7, etc which could be added to the file
    by other headers.
    
    Fixes bug #582074.