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 581807 - During configure gimp-gap tests only for FAAD1. FAAD2 is not properly detected.
During configure gimp-gap tests only for FAAD1. FAAD2 is not properly detected.
Status: RESOLVED FIXED
Product: gimp-gap
Classification: Other
Component: general
Current SVN
Other All
: Normal normal
: ---
Assigned To: hof
hof
Depends on:
Blocks:
 
 
Reported: 2009-05-08 00:24 UTC by David Evans
Modified: 2009-05-11 17:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch for configure.in to recognize libfaad2 (only) (525 bytes, patch)
2009-05-08 00:27 UTC, David Evans
none Details | Review

Description David Evans 2009-05-08 00:24:50 UTC
Please describe the problem:
During configure, gimp-gap attempts to test for the presence of libfaad but
the test employed only tests for faad version 1.  MacPorts provides faad version 2 which is not recognized even though the underlying ffmpeg configure script
tests for either one and ffmpeg 0.5 will build with either.

Steps to reproduce:
Attempt to configure with only libfaad2 installed


Actual results:
libfaad is not detected

Expected results:
Since ffmpeg accepts either version so should gimp-gap

Does this happen every time?
Yes

Other information:
Attached is the patch that MacPorts uses to change the libfaad test in configure.in from version 1 to version 2.  This allows libfaad to be
recognized when using MacPorts.  A better approach to use in general would be to test first for version 2 and if that is not available fall back to version 1.  ffmpeg detects either in its configuration script.
Comment 1 David Evans 2009-05-08 00:27:10 UTC
Created attachment 134233 [details] [review]
Patch for configure.in to recognize libfaad2 (only)
Comment 2 hof 2009-05-11 05:38:47 UTC
Your patch was applied to the GIT repository 2009-05-10 14:18:43 (GMT)
http://git.gnome.org/cgit/gimp-gap/commit/?id=073457de56c2c07ec0e2c698cac773bf7093de9e
Comment 3 David Evans 2009-05-11 17:30:42 UTC
This works for MacPorts where only libfaad2 exists but you might want to consider
adding back an additional test for libfaad1 if this one fails.  ffmpeg will
accept either.  You might refer to their configure file for the logic
they use.