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 1159 - Build issue: kvirc fails to detect TDE directory
Summary: Build issue: kvirc fails to detect TDE directory
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: non-core programs (show other bugs)
Version: 3.5.13 [Trinity]
Hardware: All Linux
: P5 normal
Assignee: Slávek Banko
URL:
Depends on:
Blocks:
 
Reported: 2012-08-06 07:11 CDT by Francois Andriot
Modified: 2013-04-10 15:27 CDT (History)
4 users (show)

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

Attachments
Fix detection of TDE include directory (R14) (2.03 KB, patch)
2013-04-10 11:39 CDT, Slávek Banko 		Details | Diff
Fix detection of TDE include directory (v3.5.13-sru) (2.03 KB, patch)
2013-04-10 11:40 CDT, Slávek Banko 		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 Francois Andriot 2012-08-06 07:11:00 CDT 
During the "./configure" step of kvirc, the following step:
  checking for the general KDE directory ..

May fail. I found out that this step just checks for the existence of a specific file under different prefix to find out under which prefix TDE is installed.

The file it is looking for is "include/kwin.h".
It is defined in file 'configure.in' line 103, in the variable SS_KDE_GENERAL_FILE_TO_SEARCH .

Since I've installed my headers in a non standard location (not under /opt/trinity/include) , the check fails and kvirc does not install correctly (it still builds, but some files are installed in a wrong location).

We should use another test to find TDE's directory.
For example: kde-config --prefix
Comment 1 Darrell 2013-04-05 18:04:57 CDT 
Does this problem still exist?

The configure.in file was updated a while back with the kwin->twin renaming and now looks for include/twin.h. Seems the original suggestion still applies.
Comment 2 Timothy Pearson 2013-04-05 18:21:50 CDT 
Fixed in GIT hash 5a08fb9 by looking for tde-config instead of twin.h.

Thanks for reporting!
Comment 3 Slávek Banko 2013-04-09 20:44:26 CDT 
The patch is not sufficient. Setting the SS_KDE_GENERAL_FILE_TO_SEARCH to bin/tde-config is fine, but the setting SS_KDE_INCLUDE_FILE_TO_SEARCH to tde-config is wrong.
Comment 4 Slávek Banko 2013-04-10 11:39:56 CDT 
Created attachment 1147 [details]
Fix detection of TDE include directory (R14)

I changed the searched file to tdeversion.h and added call tde-config to get include path.

Please, try it.
Comment 5 Slávek Banko 2013-04-10 11:40:40 CDT 
Created attachment 1148 [details]
Fix detection of TDE include directory (v3.5.13-sru)
Comment 6 Timothy Pearson 2013-04-10 13:56:47 CDT 
Look good; if it builds for you then go ahead and push.
Comment 7 Slávek Banko 2013-04-10 15:27:41 CDT 
Fixed in GIT hash 1791f4e5.
And also in v3.5.13-sru branch.