By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1054 - Build issue: soundkonverter FTBFS without --with-extra-includes
Summary: Build issue: soundkonverter FTBFS without --with-extra-includes
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: non-core programs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Other
: P1 normal
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2012-06-20 09:08 CDT by Darrell
Modified: 2013-09-29 18:12 CDT (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Patch to fix finding cdda.h (598 bytes, patch)
2012-10-20 21:27 CDT, Darrell 		Details | Diff
Fix search for cdda includes (1.03 KB, patch)
2013-09-29 12:09 CDT, Slávek Banko 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darrell 2012-06-20 09:08:07 CDT 
I need to add --with-extra-includes=/usr/include/cdda.

With that configure option the package builds, but the configure output shows the following:

checking cdda_interface.h usability... no
checking cdda_interface.h presence... no
checking for cdda_interface.h... no

The fix probably is similar to bug report 788 and tdemultimedia, but I can't figure out where to add the fix. :-(
Comment 1 Darrell 2012-06-20 12:48:38 CDT 
To further expand this build error, the configure process does not stop the build process. As cdparanoia is a requirement for building soundkonverter, the build process should stop because of the following:

checking cdda_interface.h usability... no
checking cdda_interface.h presence... no
checking for cdda_interface.h... no

Without using --with-extra-includes, the build predictably fails like this:

In file included from cdmanager.cpp:3:
paranoia.h:25:28: error: cdda_interface.h: No such file or directory
paranoia.h:26:27: error: cdda_paranoia.h: No such file or directory

The fix then would be:

* End the configure process when the following occur:

checking cdda_interface.h usability... no
checking cdda_interface.h presence... no
checking for cdda_interface.h... no

* Add support to look in /usr/include/cdda.
Comment 2 Darrell 2012-06-20 13:00:40 CDT 
Changing the severity to Normal. I believe that although the configure output says the cdda headers are not found, the --with-extra-includes is finding them anyway and building the package.
Comment 3 Darrell 2012-10-20 21:27:06 CDT 
Created attachment 924 [details]
Patch to fix finding cdda.h
Comment 4 Slávek Banko 2013-09-29 12:09:32 CDT 
Created attachment 1527 [details]
Fix search for cdda includes

I prepared an alternative patch.
Please test it.
Comment 5 Darrell 2013-09-29 17:43:03 CDT 
Patch tested. No build failures. The final package is the same except for the expected changes in bin/soundkonverter.

Thank you!
Comment 6 Slávek Banko 2013-09-29 18:12:25 CDT 
Fixed in GIT hash 5077373c.
Thank you for testing.