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 1041 - Build issue: tdesdk/kbugbuster FTBFS
Summary: Build issue: tdesdk/kbugbuster FTBFS
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdesdk (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Other
: P1 critical
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2012-06-17 13:30 CDT by Darrell
Modified: 2012-11-07 20:52 CST (History)
3 users (show)

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

Attachments
Initial patch (1011 bytes, patch)
2012-11-04 20:41 CST, Slávek Banko 		Details | Diff
Fix cmake build of kbugbuster with kcal (2.02 KB, patch)
2012-11-05 18:11 CST, 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-17 13:30:39 CDT 
Latest GIT. I am unable to build kbugbuster in tdesdk. I realize kbugbuster is more or less broken from a usability perspective, but we should still be able to build.

I can build tdesdk in one of two ways:

* -DBUILD_KBUGBUSTER=OFF

* Reversing GIT patches b04ab117 and 629e2283, both from 03-27-2012.

The failure is being unable to find the libkcal headers installed by tdepim. Those headers are installed in $PREFIX/include/libkcal, as seen in tdepim/libkcal/CMakeLists.txt: DESTINATION ${INCLUDE_INSTALL_DIR}/libkcal.

As seen from the two commits, the build process does not look in ${INCLUDE_INSTALL_DIR}/libkcal to find the headers.

Reversing the two patches in GIT seems one way to fix this.
Comment 1 Slávek Banko 2012-11-04 20:41:27 CST 
Created attachment 956 [details]
Initial patch

There is currently no testing for required libraries and headers. So far the patch is not pretty. The purpose is to test whether this is the right way.

I tried to build with:
 -DBUILD_KBUGBUSTER="ON"
 -DWITH_KCAL = "ON"
Comment 2 Darrell 2012-11-04 22:08:18 CST 
I ran a quick test of the patch in attachment 956 [details]. I used -DWITH_KCAL=ON and -DBUILD_KBUGBUSTER=ON. I did not reverse commits 629e2283 and b04ab117.

The patch seems to work. In my build log there are many instances of -I/opt/trinity/include/libkcal. I do not see any errors or warnings. The cmake output shows:

//Enable kcal support (kbugbuster)
WITH_KCAL:BOOL=ON

There is a kcal_bugzilla.la and kcal_bugzilla.so in the final package.
Comment 3 Slávek Banko 2012-11-05 18:11:01 CST 
Created attachment 957 [details]
Fix cmake build of kbugbuster with kcal

Since the initial patch was succeeding, I have prepared a better patch. Was added test for the presence kcal.
Comment 4 Slávek Banko 2012-11-07 20:52:20 CST 
Fixed in GIT hash 59a2f3d9.