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 790 - Build issue: tdemultimedia: audiofile.h configure errors
Summary: Build issue: tdemultimedia: audiofile.h configure errors
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdemultimedia (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Other
: P1 critical
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2012-01-12 12:27 CST by Darrell
Modified: 2012-10-19 15:25 CDT (History)
2 users (show)

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

Attachments
Fix audiofile.h build error (1.27 KB, patch)
2012-02-21 19:39 CST, Darrell 		Details | Diff
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-01-12 12:27:45 CST 
When building kdemultimedia, the configure output shows the following:

=============================================================
checking audiofile.h usability... yes
checking audiofile.h presence... no
configure: WARNING: audiofile.h: accepted by the compiler, rejected by the preprocessor!
configure: WARNING: audiofile.h: proceeding with the compiler's result
checking for audiofile.h... yes
=============================================================

Checking the configure log revealed the problem for this error was:

"tqt.h No such file or directory."

I was able to stop this particular error with this patch:

=============================================================
diff -urN kdemultimedia/admin/acinclude.m4.in kdemultimedia.new/admin/acinclude.m4.in
--- kdemultimedia/admin/acinclude.m4.in 2011-10-23 17:53:19.000000000 -0500
+++ kdemultimedia.new/admin/acinclude.m4.in 2012-01-10 23:02:01.000000000 -0600
@@ -1236,7 +1236,7 @@
   CXXFLAGS="-I/usr/include/tqt"
   AC_TRY_COMPILE([
   #define TQT_VERSION_ONLY
-  #include <tqt.h>
+  #include <tqt/tqt.h>
   #undef TQT_VERSION_ONLY
   #ifdef USE_QT4
   choke me
@@ -2846,7 +2846,7 @@
   QT_LDFLAGS="$QT_LDFLAGS $LIBTQT_LDFLAGS"
   AC_SUBST(QT_LDFLAGS)
   qtlib="$qtlib $LIBTQT_LDFLAGS"
-  LIBTQT_CXXFLAGS="-include tqt.h"
+  LIBTQT_CXXFLAGS="-include tqt/tqt.h"
   all_includes="$all_includes $LIBTQT_CXXFLAGS"
   QT_INCLUDES="$QT_INCLUDES $LIBTQT_CXXFLAGS"

diff -urN kdemultimedia/cmake/modules/TDEMacros.cmake kdemultimedia.new/cmake/modules/TDEMacros.cmake
--- kdemultimedia/cmake/modules/TDEMacros.cmake 2011-10-27 11:11:44.000000000 -0500
+++ kdemultimedia.new/cmake/modules/TDEMacros.cmake 2012-01-10 23:02:35.000000000 -0600
@@ -1120,7 +1120,7 @@

 macro( tde_include_tqt )
   foreach( _cpp ${ARGN} )
-    set_source_files_properties( ${_cpp} PROPERTIES COMPILE_FLAGS "-include tqt.h" )
+    set_source_files_properties( ${_cpp} PROPERTIES COMPILE_FLAGS "-include tqt/tqt.h" )
   endforeach()
 endmacro( )

=============================================================

The tqt header files are installed at /usr/include/tqt.

I believe the patch is a stop gap. I believe the correct solution is to update the configure process to find the tqt subdirectory where the tqt.h header file is located.
Comment 1 Darrell 2012-02-21 19:39:56 CST 
Created attachment 404 [details]
Fix audiofile.h build error

Long ago I posted the patch as part of my original description. I'm attaching the actual patch.

I'm bumping this report to Blocker level because this bug is related to building.
Comment 2 Darrell 2012-04-16 16:09:58 CDT 
The package builds with work-arounds. Bumping to Critical.
Comment 3 Timothy Pearson 2012-06-09 18:09:08 CDT 
The proposed patch doesn't make sense.  The root cause was actually in tdemultimedia, and has been fixed in GIT hash dea8e5c.

Thanks for reporting!
Comment 4 Darrell 2012-06-09 20:06:49 CDT 
Patch works:

checking audiofile.h usability... yes
checking audiofile.h presence... yes
checking for audiofile.h... yes

Package builds without error. Even builds with parallel processing! Thank you!