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

Summary:	Build issue: soundkonverter FTBFS without --with-extra-includes
Product:	TDE	Reporter:	Darrell <darrella>
Component:	non-core programs	Assignee:	Timothy Pearson <kb9vqf>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bugwatch, darrella, slavek.banko
Priority:	P1
Version:	R14.0.0 [Trinity]
Hardware:	Other
OS:	Other
Compiler Version:		TDE Version String:
Application Version:		Application Name:
Attachments:	Patch to fix finding cdda.h Fix search for cdda includes

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.